]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Validate bool value using strutils.bool_from_string method
authorRajesh Tailor <rajesh.tailor@nttdata.com>
Fri, 29 May 2015 10:26:22 +0000 (03:26 -0700)
committerRajesh Tailor <rajesh.tailor@nttdata.com>
Thu, 18 Jun 2015 13:29:17 +0000 (06:29 -0700)
commit24fd26fc98acf1fbc07f6922ace5de14296d8f0e
treefa101671fcb85e9238be919ca98285d2902dd5c0
parentde42caeb2e3f90e7ab80e9a5a41e85c1be17888a
Validate bool value using strutils.bool_from_string method

bool value is validated with different logic scattered all over the places
1. isinstance of six.string_type or bool
2. is_valid_boolstr method

Made changes to use strutils.bool_from_string method to validate bool
value at all places as this method does all things expected above.
Changes are not made in v1 api as it is going to be deprecated in
Liberty release.

Example error message:
Invalid value for force_host_copy: Unrecognized value 'xyz', acceptable
values are: '0', '1', 'f', 'false', 'n', 'no', 'off', 'on', 't', 'true',
'y', 'yes'

ApiImpact: 400 BadRequest response will be returned in case invalid force
value is provided during volume upload to image, earlier it was ignored.

Closes-Bug: 1460575
Change-Id: Ie64d429f5860a950cda9b363a65c5282f128f10b
cinder/api/contrib/admin_actions.py
cinder/api/contrib/volume_actions.py
cinder/api/v2/snapshots.py
cinder/tests/unit/api/contrib/test_admin_actions.py