Shobhit Srivastava
316ccb0e65
checking remote_group_id while comparing os_security_group_rule
2016-12-08 11:23:31 -05:00
Monty Taylor
530f12035b
Update docstring to show port ranges as optional
2016-12-08 11:23:18 -05:00
David Shrewsbury
789c1fcbe7
Correct port matching logic
...
Port matching logic did not take into account recent shade change
to equate (None, None) to (1, 65535) when Nova is the backend.
Also, this encapsulates the port matching logic into a single function
and heavily documents the logic.
2016-12-08 11:23:18 -05:00
David Shrewsbury
387fe5b0e7
Change required parameters for rules module
...
The ports and protocol are no longer required (and now depends on
a new version of shade).
2016-12-08 11:23:18 -05:00
Monty Taylor
4f2028439f
Use int in the parameter list instead of casting
2016-12-08 11:23:18 -05:00
dagnello
2c3fc61357
Resolving issues in rule comparison algorithm
...
Port range min/max values are at times represented as string and
compared to int equivalents. This fix explicitly ensures all
port range values are ints for proper comparisons.
2016-12-08 11:23:18 -05:00
David Shrewsbury
e7dd93c5c1
Compare ports as strings
...
Ports as returned from shade are ints. They are strings as they come
in to the module.
2016-12-08 11:23:18 -05:00
David Shrewsbury
14212c37e2
Fix syntax error
2016-12-08 11:23:18 -05:00
Monty Taylor
60102e36e2
Return rule object
2016-12-08 11:23:18 -05:00
David Shrewsbury
174254a423
Update secgroup rules module for latest shade
...
This allows the rules module to work against either nova or neutron
for handling security groups. New parameters for 'direction' and
'ethertype' are added.
Check mode is supported with this version.
2016-12-08 11:23:18 -05:00
David Shrewsbury
fcc1d643f6
Recongnize None and -1 port equivalency
...
shade 0.7.0 represents disabled min/max ports as None (in the
neutron style) rather than -1. Recognize this as the same as -1.
2016-12-08 11:23:17 -05:00
David Shrewsbury
b42a9d56e6
Fix id value reference
2016-12-08 11:23:17 -05:00
David Shrewsbury
f02ec15c17
Update rules mode for latest shade
...
Shade 0.7.0 normalized the security group data that is returned,
when using nova, to look more like neutron security group data.
This adjusts for that change.
2016-12-08 11:23:17 -05:00
dagnello
ecef2aadba
Minor fixes for os_security_group_rule module
...
Was not able to use this module as it was. The changes submitted
resolved the issues I ran into in order to get it working.
2016-12-08 11:23:17 -05:00
Davide Guerri
2d7dbc5d0c
Fix invalid syntax in openstack_module_kwargs call
2016-12-08 11:23:17 -05:00
Monty Taylor
9acc891477
Fix example code syntax
2016-12-08 11:23:17 -05:00
Monty Taylor
8d5e43c14f
Add OpenStack Security Group Rule module
2016-12-08 11:23:17 -05:00
David Shrewsbury
9cbd7b2ece
Remove os_security_group_rule module
...
The rules module will have it's own branch.
2016-12-08 11:23:08 -05:00
Monty Taylor
234a6c93c7
Add OpenStack Security Group support
...
Two modules - one for security groups and one to manage rules in a
security group.
2016-12-08 11:23:07 -05:00