]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 33b8853) | patch
Verify all quotas before updating the database
authorankitagrawal <ankit11.agrawal@nttdata.com>
Tue, 17 Mar 2015 09:08:12 +0000 (02:08 -0700)
committerankitagrawal <ankit11.agrawal@nttdata.com>
Tue, 17 Mar 2015 12:46:29 +0000 (05:46 -0700)
commit197299985a52535ad995dbd111491b58462f140a
tree193ff9479bfba3962f27b359d74bba7816bfec19tree | snapshot
parent33b885381640861cc1fa3af65e95fc0cf619c9f0commit | diff
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
cinder/api/contrib/quotas.py diff | blob | history
cinder/tests/api/contrib/test_quotas.py diff | blob | history