extra 'u' in "cinder show"

Bug #1615921 reported by Cao ShuFeng
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-cinderclient
Fix Released
Undecided
Michael Dovgal

Bug Description

There was a patch[1] that remove extra 'u' in cli output.
But there is still an omission.
cinder show a8538f0b-6903-4c7c-b18a-391992fde022

+--------------------------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| Property | Value |
+--------------------------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| attachments | [{u'server_id': u'fe2080d3-e393-42cd-b6a6-1eebc8ba65a3', u'attachment_id': u'c2fede56-430c-4c19-aef9-ff8ce72689be', u'attached_at': u'2016-08-23T05:07:13.000000', u'host_name': None, u'volume_id': u'a8538f0b-6903-4c7c-b18a-391992fde022', u'device': u'/dev/vdb', u'id': u'a8538f0b-6903-4c7c-b18a-391992fde022'}] |
| availability_zone | nova |
| bootable | false |
| consistencygroup_id | None |
| created_at | 2016-08-23T05:06:27.000000 |
| description | None |
| encrypted | False |
| id | a8538f0b-6903-4c7c-b18a-391992fde022 |
| metadata | {'readonly': 'False', 'attached_mode': 'rw'} |
| migration_status | None |
| multiattach | False |
| name | None |
| os-vol-host-attr:host | ubuntu@lvmdriver-1#lvmdriver-1 |
| os-vol-mig-status-attr:migstat | None |
| os-vol-mig-status-attr:name_id | None |
| os-vol-tenant-attr:tenant_id | 375994b21556490b8aa5209b95e9a8f5 |
| replication_status | disabled |
| size | 1 |
| snapshot_id | None |
| source_volid | None |
| status | in-use |
| updated_at | 2016-08-23T05:07:13.000000 |
| user_id | 0913c2085a804641920f8c64dc046596 |
| volume_image_metadata | {'a': 'b'} |
| volume_type | lvmdriver-1 |
+--------------------------------+------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+

There is a related bug[2].

[1]: https://review.openstack.org/#/c/281475
[2]: https://bugs.launchpad.net/python-cinderclient/+bug/1606904

Cao ShuFeng (caosf-fnst)
Changed in python-cinderclient:
assignee: nobody → Cao ShuFeng (caosf-fnst)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-cinderclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/358995

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

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/358995
Reason: Merged changes with https://review.openstack.org/#/c/347804/

Changed in python-cinderclient:
assignee: Cao ShuFeng (caosf-fnst) → Michael Dovgal (mdovgal)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

Reviewed: https://review.openstack.org/347804
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=64c8f74a7c238694d473884682a763f6a8a5b02b
Submitter: Jenkins
Branch: master

commit 64c8f74a7c238694d473884682a763f6a8a5b02b
Author: Cao ShuFeng <email address hidden>
Date: Wed Jul 27 10:13:36 2016 -0400

    Remove extra 'u' from cli output

    The unicode_key_value_to_string() function is designed to remove
    extra 'u' in cinderclient cli output.
    However this patch[1] bring the extra 'u' back.
    Let's remove the extra 'u' again.

    Closes-bug: #1615921
    Closes-bug: #1606904

    [1] https://review.openstack.org/#/c/342734/
    Change-Id: I26f0ad7149f57e935953c2398ba90b7b3585e201

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.10.0

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

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.