]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fixes Cinder REST API /volumes issue
authorFei Long Wang <flwang@cn.ibm.com>
Thu, 14 Mar 2013 06:35:10 +0000 (14:35 +0800)
committerFei Long Wang <flwang@cn.ibm.com>
Thu, 21 Mar 2013 05:31:58 +0000 (13:31 +0800)
commit7c760581d80b2ee5cd1e01a1d0a007770c9aa807
treeeae0036db72dde259ef1ac085e802bacdfc9c2bb
parent65e291054681e45d3160d995c5648b8311bc1a69
Fixes Cinder REST API /volumes issue

Issue #1
Once GET variable 'offset' is specified, the API /volume will get an empty
output.

Issue #2
Should validate the GET variable 'limit' before query database to get a
consistent message with Cinder REST API v1.
By current implement, error message is as below if the variable 'limit'
is invalid:
--------------------------------------------------------------------------
{"computeFault": {"message": "The server has either erred or is incapable
of performing the requested operation.", "code": 500}}

After this change, the new message is as below:
--------------------------------------------------------------------------
{"badRequest": {"message": "limit param must be an integer", "code": 400}}

Fixes Bug: 1154454

Change-Id: Ifb155477bae0ea3e39877737ee9019e7d8a104a7
cinder/api/v2/volumes.py
cinder/tests/api/v2/test_volumes.py
cinder/volume/api.py