cinder volume create from image with non-ASCII name fails

Bug #1352739 reported by Sharif Olorin
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Undecided
Masaki Kanno

Bug Description

This applies to the SQLAlchemy Cinder backend. Reproduced in icehouse, but looks like it's present in master as well.

Reproduce:

 - start an instance
 - take a snapshot, give it a non-ASCII name (mine was "ceilometer-dev-β")
 - create a volume from the resulting image

Observed behaviour: volume transitions from "Downloading" to "Error"; checking the cinder log results in the attached stacktrace. The same process works fine without non-ASCII characters in the image name. I infer that this is not expected from this[0] wiki page.

Looking in cinder/db/sqlalchemy/api.py, I see this: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/api.py#L2488

which looks like it could benefit from using the safe encoding functions in openstack.common.strutils, but these aren't used elsewhere in the module and I'm unsure if there's a reason for that or not (rather new to Openstack at the moment). Happy to submit a patch if I haven't misunderstood anything.

[0] https://wiki.openstack.org/wiki/Encoding

Revision history for this message
Sharif Olorin (fractalcat) wrote :
Changed in cinder:
status: New → Confirmed
Changed in cinder:
assignee: nobody → Masaki Kanno (kanno-masaki)
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/129112

Changed in cinder:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (master)

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/129112
Reason: over a month old with no update.

Revision history for this message
chenying (ying-chen) wrote :

This bug has been fixed.

Changed in cinder:
status: In Progress → Invalid
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.