VersionedObject.__repr__ should return encoded string
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Invalid
|
Undecided
|
Liang Chen | ||
oslo.versionedobjects |
Fix Released
|
Undecided
|
Liang Chen | ||
cinder (Ubuntu) |
Fix Released
|
Medium
|
Liang Chen |
Bug Description
[Impact]
* Cinder snapshot display-description cannot contain non-ascii characters.
[Test Case]
* cinder create 1
* cinder snapshot-create --display-
[Regression Potential]
* None
The following error is reported when creating a volume snapshot with non-ascii display-
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
2015-11-09 05:55:50.995 29937 ERROR oslo_messaging.
Root cause is that profiler tries to get a string representation of the arguments (cinder.
[1] trying to encode the returned string when it's an unicode object
https:/
[2] if encoding arg is left null, default encoding will be used
https:/
Related branches
- Marc Deslauriers: Disapprove
-
Diff: 75 lines (+55/-0)3 files modifieddebian/changelog (+7/-0)
debian/patches/encode-repr-return.patch (+47/-0)
debian/patches/series (+1/-0)
Changed in oslo.versionedobjects: | |
assignee: | nobody → Liang Chen (cbjchen) |
description: | updated |
Changed in oslo.versionedobjects: | |
status: | In Progress → Fix Committed |
tags: | added: sts |
Changed in oslo.versionedobjects: | |
milestone: | none → 0.13.0 |
status: | Fix Committed → Fix Released |
Changed in cinder: | |
assignee: | nobody → Liang Chen (cbjchen) |
status: | New → In Progress |
description: | updated |
Changed in cinder (Ubuntu): | |
status: | New → In Progress |
assignee: | nobody → Liang Chen (cbjchen) |
Changed in cinder (Ubuntu): | |
importance: | Undecided → Medium |
Changed in cinder: | |
status: | In Progress → Invalid |
Fix proposed at https:/ /review. openstack. org/#/c/ 242964/