openstack/trove
Code Issues Proposed changes

Remove Min/Max for Configuration Group Booleans

Browse Source 
boolean configuration-group parameters at the moment require a
min/max value of 0/1 to work around some faulty logic that is
meant to validate integer values, but instead validates integer
and boolean values.

Closes-Bug: #1288507
Change-Id: I2d7556621e891b22291c7ef9e9c523e1ce98a047
This commit is contained in:
Khyati Sheth 2014-03-05 18:29:20 -08:00 committed by amcrn
parent ee1fc921a1
commit 481018d354
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
trove/configuration/service.py
View File

@ -198,7 +198,7 @@ class ConfigurationsController(wsgi.Controller):
raise exception.UnprocessableEntity(message=msg)
# integer min/max checking
if isinstance(v, int):
if isinstance(v, int) and not isinstance(v, bool):
try:
min_value = int(rule.get('min'))
except ValueError: