make all str conversions utf8 in ceph backup driver

Bug #1206044 reported by Edward Hope-Morley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Edward Hope-Morley

Bug Description

A patch was applied to make string non-unicode conversion use string.utf8() as opposed to str() but was only applied to a subset of use cases. This patch will apply it universally throughout the driver.

Changed in cinder:
assignee: nobody → Edward Hope-Morley (hopem)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/39050
Committed: http://github.com/openstack/cinder/commit/31238c748e1c0cffb9ba412b3bf55b87f991714a
Submitter: Jenkins
Branch: master

commit 31238c748e1c0cffb9ba412b3bf55b87f991714a
Author: Edward Hope-Morley <email address hidden>
Date: Mon Jul 29 09:41:35 2013 +0100

    Make unicode-to-utf8 conversion universal in ceph backup driver.

    librbd does not like unicode strings. A patch was
    recently applied (c98cf2d8) to make all config
    values utf8 but there are other areas of the ceph
    backup driver where this also applies. This patch
    adds a new method which converts strings to utf8
    and replaces all relevant uses of str() with this
    new conversion.

    Change-Id: I8c3b01d1ea00b601c599396e3a662f172ffb9757
    Fixes: bug #1206044

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-3 → 2013.2
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.