diff options
author | Monty Taylor <mordred@inaugust.com> | 2015-06-19 17:17:46 -0400 |
---|---|---|
committer | Monty Taylor <mordred@inaugust.com> | 2015-07-06 18:50:03 -0400 |
commit | 9f03302b68b4038fa664230dcbb66920325dbd1f (patch) | |
tree | e9ac2735fa1b8ed9934603fb5eca39c903d1ee28 | |
parent | 2e8daa23309ada2bfba8415ea6ec5d764b565f05 (diff) | |
download | ansible-modules-core-9f03302b68b4038fa664230dcbb66920325dbd1f.tar.gz |
Use int in the parameter list instead of casting
-rw-r--r-- | cloud/openstack/os_security_group_rule.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/cloud/openstack/os_security_group_rule.py b/cloud/openstack/os_security_group_rule.py index fc928394..7d86408b 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() |