From 9f03302b68b4038fa664230dcbb66920325dbd1f Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Fri, 19 Jun 2015 17:17:46 -0400 Subject: [PATCH] Use int in the parameter list instead of casting --- cloud/openstack/os_security_group_rule.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/cloud/openstack/os_security_group_rule.py b/cloud/openstack/os_security_group_rule.py index fc9283940c0..7d86408b379 100644 --- a/cloud/openstack/os_security_group_rule.py +++ b/cloud/openstack/os_security_group_rule.py @@ -94,8 +94,8 @@ def _find_matching_rule(module, secgroup): :returns: The matching rule dict, or None if no matches. """ protocol = module.params['protocol'] - port_range_min = int(module.params['port_range_min']) - port_range_max = int(module.params['port_range_max']) + port_range_min = module.params['port_range_min'] + port_range_max = module.params['port_range_max'] remote_ip_prefix = module.params['remote_ip_prefix'] ethertype = module.params['ethertype'] direction = module.params['direction'] @@ -142,8 +142,8 @@ def main(): security_group = dict(required=True), protocol = dict(default='tcp', choices=['tcp', 'udp', 'icmp']), - port_range_min = dict(required=True), - port_range_max = dict(required=True), + port_range_min = dict(required=True, type='int'), + port_range_max = dict(required=True, type='int'), remote_ip_prefix = dict(required=False, default=None), # TODO(mordred): Make remote_group handle name and id remote_group = dict(required=False, default=None), @@ -211,4 +211,4 @@ def main(): from ansible.module_utils.basic import * from ansible.module_utils.openstack import * -main() \ No newline at end of file +main()