cinder manageable-list does not support marker and raise 500 error

Bug #1657992 reported by Prerna Dembla
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
wangxiyuan
python-cinderclient
Fix Released
Undecided
wangxiyuan

Bug Description

cinder manageable-list is used to list all manageable volumes on a cinder host. Marker can be specified with this by using --marker<volume id> as a argument. It should return volumes that appear later in the volume list than that represented by this volume id but Cinder return HTTP 500 error.
---------------------------------------
usage: cinder manageable-list [--detailed <detailed>] [--marker <marker>]
                              [--limit <limit>] [--offset <offset>]
                              [--sort <key>[:<direction>]]
                              <host>

Lists all manageable volumes.

Positional arguments:
  <host> Cinder host on which to list manageable volumes; takes
                        the form: host@backend-name#pool

Optional arguments:
  --detailed <detailed>
                        Returned detailed information (default true).
  --marker <marker> Begin returning volumes that appear later in the
                        volume list than that represented by this volume id.
                        Default=None.
  --limit <limit> Maximum number of volumes to return. Default=None.
  --offset <offset> Number of volumes to skip after marker. Default=None.
  --sort <key>[:<direction>]
                        Comma-separated list of sort keys and directions in
                        the form of <key>[:<asc|desc>]. Valid keys: id,
                        status, size, availability_zone, name, bootable,
                        created_at, reference. Default=None.
------------------------------------------
cinder manageable-list controller@lvm#lvm --sort size --marker bc545d26-ce13-4e47-a6e5-bae6c74f8dae
ERROR: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-d1606041-bf07-4871-aac3-2e14a4f97dcd)

Tags: bugsmash
Changed in cinder:
assignee: nobody → Prerna Dembla (prerna.dembla)
wangxiyuan (wangxiyuan)
Changed in cinder:
status: New → Confirmed
Ivan Kolodyazhny (e0ne)
tags: added: bugsmash
Changed in cinder:
importance: Undecided → Medium
Revision history for this message
wangxiyuan (wangxiyuan) wrote :

Test with master branch, the 500 error doesn't raise now. But the marker in cinder is "reference", not volume_id. And the reference in cinder is a dict. But the maker which users pass is always string. So seems that it doesn't work.

Revision history for this message
wangxiyuan (wangxiyuan) wrote :
wangxiyuan (wangxiyuan)
Changed in cinder:
assignee: Prerna Dembla (prerna.dembla) → wangxiyuan (wangxiyuan)
wangxiyuan (wangxiyuan)
Changed in python-cinderclient:
assignee: nobody → wangxiyuan (wangxiyuan)
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/466147

Changed in cinder:
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/466150

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

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

commit 990527ea79b17110aad48d5600cb6dde397b30c1
Author: wangxiyuan <email address hidden>
Date: Fri May 19 10:12:13 2017 +0800

    Fix the wrong help message of marker

    The marker for list manageable resources should be the resources'
    "reference" property, not "volume id"

    Change-Id: Icfe10b9a041daa6a9333f71fae15eba86d330b7b
    Closes-bug: #1657992

Changed in python-cinderclient:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

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

commit 8a736209edaebd9402133f8e522eb8f453343b12
Author: wangxiyuan <email address hidden>
Date: Fri May 19 10:00:46 2017 +0800

    Convert marker format for listing manageable resources

    The marker for listing manageable resources is the "reference"
    property which should be dict.

    We should convert and check its format before using.

    Change-Id: I190ff8506e0056d685ce429c2fa275b097f9e635
    Closes-bug: #1657992

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 11.0.0.0b2

This issue was fixed in the openstack/cinder 11.0.0.0b2 development milestone.

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.