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

Bug #1230556 reported by Allele Dev on 2013-09-25
This bug affects 1 person
Affects Status Importance Assigned to Milestone
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) on 2013-10-01
Changed in marconi:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → Alejandro Cabrera (cpp-cabrera)

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

Changed in marconi:
status: Confirmed → In Progress

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) on 2013-12-19
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in marconi:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers