]> 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)
committerjohn-griffith <john.griffith@solidfire.com>
Sun, 24 Mar 2013 20:11:13 +0000 (14:11 -0600)
commit18e2c3542a6db98b1a71e50ed77b337ab179a0ba
tree10381538626172fc544169da0b4074ffb3509ca0
parent19f4688f3640bf9243a297101cf4d9ca92bc6013
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
(cherry picked from commit 7c760581d80b2ee5cd1e01a1d0a007770c9aa807)
cinder/api/v2/volumes.py
cinder/tests/api/v2/test_volumes.py
cinder/volume/api.py