extra 'u' in "cinder type-show" command

Bug #1589461 reported by Cao ShuFeng
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-cinderclient
Fix Released
Undecided
aohuanxuan

Bug Description

There was a patch[1] that remove extra 'u' in cli output.
But there is still an omission.
cinder type-show

$ cinder type-show d5aabe2a-13a8-432b-b471-d3a80b66ef00
+---------------------------------+------------------------------------------------------+
| Property | Value |
+---------------------------------+------------------------------------------------------+
| description | None |
| extra_specs | {u'a': u'c', u'volume_backend_name': u'lvmdriver-1'} |
| id | d5aabe2a-13a8-432b-b471-d3a80b66ef00 |
| is_public | True |
| name | lvmdriver-1 |
| os-volume-type-access:is_public | True |
| qos_specs_id | None |
+---------------------------------+------------------------------------------------------+

https://review.openstack.org/#/c/281475

Cao ShuFeng (caosf-fnst)
Changed in python-cinderclient:
assignee: nobody → Cao ShuFeng (caosf-fnst)
description: updated
Cao ShuFeng (caosf-fnst)
Changed in python-cinderclient:
assignee: Cao ShuFeng (caosf-fnst) → nobody
assignee: nobody → Cao ShuFeng (caosf-fnst)
assignee: Cao ShuFeng (caosf-fnst) → nobody
aohuanxuan (huanxuan-ao)
Changed in python-cinderclient:
assignee: nobody → aohuanxuan (huanxuan-ao)
Changed in python-cinderclient:
status: New → In Progress
Revision history for this message
Duncan Thomas (duncan-thomas) wrote :

https://review.openstack.org/#/c/281475 is merged. Can this bug now be closed?

Revision history for this message
Cao ShuFeng (caosf-fnst) wrote :

Hi, Duncan:
    It seems we need to wait this:
    https://review.openstack.org/#/c/325739/

Thanks

Revision history for this message
aohuanxuan (huanxuan-ao) wrote :

Actually we can say this bug is a missing problem of this patch https://review.openstack.org/#/c/281475
It still need to be solved

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

Reviewed: https://review.openstack.org/325739
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=724901e3740fd6429a851f4dbceecbc117750c44
Submitter: Jenkins
Branch: master

commit 724901e3740fd6429a851f4dbceecbc117750c44
Author: Huanxuan Ao <email address hidden>
Date: Mon Jun 6 14:15:38 2016 +0800

    Fix output error for type-show command

    Before this patch, the output of "extra_specs"
    in type-show command is this:
    | extra_specs | {u'Alpha': u'a'} |
    Becuase the type of "extra_specs" is not a string
    and need to use unicode_key_value_to_string()
    method to change it to string.

    This patch add formatters and call
    unicode_key_value_to_string() mothod for
    "extra_specs" and change the output to this:
    | extra_specs | {'Alpha': 'a'} |

    Change-Id: Ia2e2633068ce5b5e6622474b145a1d4f074fd551
    Closes-Bug: #1589461

Changed in python-cinderclient:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-cinderclient 1.9.0

This issue was fixed in the openstack/python-cinderclient 1.9.0 release.

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.