]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
get volumes API does not handle limit=0
authorSteven Kaufer <kaufer@us.ibm.com>
Fri, 7 Mar 2014 04:54:50 +0000 (04:54 +0000)
committerSteven Kaufer <kaufer@us.ibm.com>
Sat, 8 Mar 2014 02:11:27 +0000 (02:11 +0000)
commit5cee4f41078ff692a37bb4c553f8faa68c782253
treec670072f6975a294607e3c032dbcb86f400f47e8
parentae9c73f8bfe1d0cce931ef5974cf1fa9ee536dc1
get volumes API does not handle limit=0

This bug fixes a regression from bug 1288429, where the "next"
link is added when the number of volumes returned is the
maximum limit (even if the "limit" param is not specified).

The regression is hit when a "limit" of 0 is specified; in this
case the logic to create the "next" link is still executed and an
exception is thrown. The fix is to add back in the "if" check that
implictly checks if the max_items is non-0.

Test code was also created to verify that a limit of 0 is handled
correctly.

Change-Id: I92f9afb9b0e2c627d2c77f67fa026b731903384f
Closes-bug: 1289124
cinder/api/common.py
cinder/tests/api/v2/test_volumes.py