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/