]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Verify all quotas before updating the database
authorankitagrawal <ankit11.agrawal@nttdata.com>
Tue, 17 Mar 2015 09:08:12 +0000 (02:08 -0700)
committerDoug Hellmann <doug@doughellmann.com>
Wed, 15 Apr 2015 19:49:45 +0000 (19:49 +0000)
commit0c467970f12cc7212cc43ee210e6386d083b5ef9
tree37306a6da8d391af3f47d2af062afbc7aa1a967b
parent672a1d4362268cd614adb2b424c982d6555d1629
Verify all quotas before updating the database

Update quota multi-value in one request is not an atomic operation, so
even if a quota is invalid (e.g., volumes has a non-integer value),
some values may change but the response code is 400.

This patch will make sure the quota limits are of integer type for all
the requested quota keys before updating the database.

Closes-Bug: 1432972
Change-Id: If83af0a80489887098e52b03b9c4012e3ef061b5
(cherry picked from commit 197299985a52535ad995dbd111491b58462f140a)
cinder/api/contrib/quotas.py
cinder/tests/api/contrib/test_quotas.py