[proxy] memcache backend - keys must be bytes, not text

Bug #1230556 reported by Allele Dev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Critical
Allele Dev

Bug Description

From: tests.unit.proxy.test_catalog.CatalogTest.test_simple (using memcached oslo.cache backend)

```
MemcachedStringEncodingError: Keys must be str()'s, not unicode. Convert your unicode strings using mystring.encode(charset)!
```

Allele Dev (alleledev)
Changed in marconi:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → Alejandro Cabrera (cpp-cabrera)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to marconi (master)

Fix proposed to branch: master
Review: https://review.openstack.org/49388

Changed in marconi:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/49388
Committed: http://github.com/openstack/marconi/commit/92fad17cc72007abdda46a3b6a98ec26e486f9cc
Submitter: Jenkins
Branch: master

commit 92fad17cc72007abdda46a3b6a98ec26e486f9cc
Author: Alejandro Cabrera <email address hidden>
Date: Wed Oct 2 11:25:22 2013 -0400

    fix: encode keys before caching

    This patch addresses an error in the caching layer that prevented it
    from playing nice with backends like memcached that expect keys to be
    bytestrings rather than unicode strings. The change is made in
    oslo.cache to make its use as transparent as possible to users.

    Change-Id: Iee48e16393321ff7908807a9f0f7fa20480ec5f4
    Closes-Bug: 1230556

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in marconi:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.