I18N issue: some log/messages are not wrapped with _()
Bug #1081498 reported by
Ying Chun Guo
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Cinder |
Fix Released
|
Medium
|
Ying Chun Guo | ||
| oslo-incubator |
Fix Released
|
Low
|
Davanum Srinivas (DIMS) | ||
| Grizzly |
Fix Released
|
Low
|
Davanum Srinivas (DIMS) | ||
Bug Description
Hi, all
In cinder source code, some LOG messages or printing messages are wrapped with _(), some are not. I used a program to scan all the source code. There are around 1500+ messages wrapped with _() and 10 messages without _(). I saved all the LOG messages and printing messages without _() into a file.
I suggest we fix this issue and provide i18n to those messages without _().
Daisy
Revision history for this message
| Ying Chun Guo (daisy-ycguo) wrote | #1 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to cinder (master) | #2 |
| Changed in cinder: | |
| assignee: | nobody → Ying Chun Guo (daisy-ycguo) |
| status: | New → In Progress |
| Changed in cinder: | |
| status: | In Progress → Triaged |
| status: | Triaged → In Progress |
| Changed in oslo: | |
| status: | New → Confirmed |
| importance: | Undecided → Low |
Revision history for this message
| Davanum Srinivas (DIMS) (dims-v) wrote | #3 |
| Changed in oslo: | |
| assignee: | nobody → Davanum Srinivas (dims-v) |
| status: | Confirmed → In Progress |
| Changed in cinder: | |
| importance: | Undecided → Medium |
| milestone: | none → grizzly-2 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to cinder (master) | #4 |
| Changed in cinder: | |
| status: | In Progress → Fix Committed |
| Changed in oslo: | |
| status: | In Progress → Fix Committed |
| Changed in oslo: | |
| milestone: | none → grizzly-1 |
| status: | Fix Committed → Fix Released |
| Changed in cinder: | |
| status: | Fix Committed → Fix Released |
| Changed in cinder: | |
| milestone: | grizzly-2 → 2013.1 |
To post a comment you must log in.
Fix proposed to branch: master
Review: https:/