diff options
| author | Mauro S. M. Rodrigues <maurosr@linux.vnet.ibm.com> | 2013-03-04 22:55:34 -0500 |
|---|---|---|
| committer | Mauro S. M. Rodrigues <maurosr@linux.vnet.ibm.com> | 2013-03-05 08:00:54 -0500 |
| commit | 6b3bba9141c7fdc4b00025f7e6ee8d980e41ec9a (patch) | |
| tree | 4b4e83c52e34c6797f72f39bd042b28a49cb43e9 /nova/api | |
| parent | 71047567ee25bacf664f9b6387d9a5c53c63622e (diff) | |
Fix 'to integer' conversion of max and min count values
Until now when you used values non-integer to max_count and/or min_count the
coercion worked like: min_count = int(2.5) so min_count received 2 instead of
raise bad request.
Convert to string before coerce to int solves the problem since int coercion
cannot convert from a string which is not a perfect int.
Change-Id: Ifaa9910e13614554d1769e71e8eba9587ec5a13b
Diffstat (limited to 'nova/api')
| -rw-r--r-- | nova/api/openstack/compute/servers.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/nova/api/openstack/compute/servers.py b/nova/api/openstack/compute/servers.py index 05aa7b238..aec082bad 100644 --- a/nova/api/openstack/compute/servers.py +++ b/nova/api/openstack/compute/servers.py @@ -830,7 +830,7 @@ class Controller(wsgi.Controller): max_count = server_dict.get('max_count', min_count) try: - min_count = int(min_count) + min_count = int(str(min_count)) except ValueError: msg = _('min_count must be an integer value') raise exc.HTTPBadRequest(explanation=msg) @@ -839,7 +839,7 @@ class Controller(wsgi.Controller): raise exc.HTTPBadRequest(explanation=msg) try: - max_count = int(max_count) + max_count = int(str(max_count)) except ValueError: msg = _('max_count must be an integer value') raise exc.HTTPBadRequest(explanation=msg) |