]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove the useless next link for volumes, transfers and backups
authorVincent Hou <sbhou@cn.ibm.com>
Wed, 29 Oct 2014 08:40:31 +0000 (01:40 -0700)
committerVincent Hou <sbhou@cn.ibm.com>
Thu, 12 Mar 2015 02:14:30 +0000 (02:14 +0000)
commit5d8cf41bc2122d271af4ba7988fc3fd39f02e987
treec76212dfc3aa1c7236f9dda564834dfe56dc8040
parent5f0fe65cc492c57e62212d288b627ea955d4510c
Remove the useless next link for volumes, transfers and backups

Two cases this patch will resolve:
1) Currently if the number of items equals the osapi_max_limit or
the last page of items equals the osapi_max_limit without the
parameter limit set in the user request, a next link is generated
in the response, though this next link will return empty volume
list. In fact it is unnecessary to generate the next link in
this case.

2) If the number of items equals the osapi_max_limit and limit is
greater than osapi_max_limit, a next link is generated. Actually,
the next link does not need to be generated, because it is certain
that there is no more volumes left in the database.

The method _get_collection_links has been called in volumes,
volume_transfers and backups. The patch can only affect the next
link generation for three of them. However, other lists like
consistency groups, qos specs, cgsnapshots have not implemented the
generation for the next link. Potentially this can be a wishlist
item for them.

Change-Id: I0f1f449c73d51675281497a095d869c1e72c889f
closes-bug: #1350558
cinder/api/common.py
cinder/api/contrib/backups.py
cinder/api/contrib/volume_transfer.py
cinder/api/v2/views/volumes.py
cinder/api/v2/volumes.py
cinder/api/views/backups.py
cinder/api/views/transfers.py
cinder/tests/api/test_common.py
cinder/tests/api/v2/test_volumes.py