]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix a problem with 'volume list' when 'all_tenants=0'
authorIan Govett <igovett@gmail.com>
Tue, 16 Sep 2014 11:32:49 +0000 (07:32 -0400)
committerIan Govett <igovett@gmail.com>
Tue, 16 Sep 2014 11:39:36 +0000 (07:39 -0400)
commited2f3ad04410e1236eba95ef6d9025aa901ddbf3
tree9c71a0d8f28615d7003c76e8827e9a58358c91e0
parent58eda5d1f41082a7e1ffef66239be30b8ac1321a
Fix a problem with 'volume list' when 'all_tenants=0'

This change checks the value of 'all_tenants' and returns a list of volumes
for all tenants when 'all_tenants' is 1 (or True). A tenant-specific volume
list is returned when 'all_tenants' is 0 (or False).

An InvalidInput exception is thrown if the 'all_tenants' value is not 0, 1,
True, or False (case insensitive).

Provided new unit tests for the get_all api which check the all_tenants, and
limits parameters.

Fixed pep8 compliance test fails with code being improperly indented, and
changed calls using str() to use six.text_type()

Change-Id: If0b8c343ee7e4e05c9aec89241458dbdf2e4734b
Closes-Bug: #1342046
cinder/tests/test_volume.py
cinder/volume/api.py