cinder transfer-list missing the --all-tenants option

Bug #1351084 reported by Gary W. Smith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Liyingjun
python-cinderclient
Fix Released
Medium
Liyingjun

Bug Description

While developing the code for (https://blueprints.launchpad.net/horizon/+spec/cinder-volume-transfer), it became apparent that cinder transfer-list is missing the --all-tenants option, which makes it inconsistent with 'cinder list'. Without this fix, when running as admin horizon will need to make multiple queries to cinder (one for each tenant present in the volume list) in order to display the transfers that exist for the volumes being shown.

It appears that cinder already supports this functionality, and it is just a matter of fixing the python-cinderclient.

Changed in python-cinderclient:
importance: Undecided → Medium
status: New → Confirmed
Liyingjun (liyingjun)
Changed in python-cinderclient:
assignee: nobody → Liyingjun (liyingjun)
Revision history for this message
Liyingjun (liyingjun) wrote :
Changed in cinder:
assignee: nobody → Liyingjun (liyingjun)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
status: New → In Progress
Changed in python-cinderclient:
status: Confirmed → In Progress
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/114176

Mike Perez (thingee)
Changed in cinder:
milestone: none → juno-rc1
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/114175
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=f523e6e59326cabb7fc98ca7c6becb0f0ccb6759
Submitter: Jenkins
Branch: master

commit f523e6e59326cabb7fc98ca7c6becb0f0ccb6759
Author: liyingjun <email address hidden>
Date: Thu Aug 7 05:58:17 2014 +0800

    Add filter to volume transfer REST api

    The transfer api supports filter param,
    https://github.com/openstack/cinder/blob/master/cinder/transfer/api.py#L72
    But the transfer REST api didn't pass the filter params when calling get_all.

    Change-Id: I1e29614a3368296571378b4a80a5cfa29b16754e
    Closes-bug: 1351084

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: juno-rc1 → 2014.2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

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

commit c01529fa538d28af3a9f0f7cdc1a96a9df5ed221
Author: liyingjun <email address hidden>
Date: Thu Aug 7 06:13:25 2014 +0800

    Add missing all-tenants option to transfer-list

    cinder transfer-list is missing the --all-tenants option, which makes
    it inconsistent with 'cinder list'. Without this fix, when running as
    admin horizon will need to make multiple queries to cinder (one for
    each tenant present in the volume list) in order to display the transfers
    that exist for the volumes being shown.

    Change-Id: Ic4d72a54c450876ff24c64efd83678062c93a24e
    Closes-bug: 1351084

Changed in python-cinderclient:
status: In Progress → Fix Committed
Changed in python-cinderclient:
milestone: none → 1.3.1
status: Fix Committed → Fix Released
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.