aed44e8886
Fix OpenStack module failure output
...
The exception message, when shade fails, will contain much more
specific information about the failure if the exception is treated
as a string. The 'message' attribute alone is usually not helpful.
2016-12-08 11:24:00 -05:00
45012ea723
allow empty description attribute for os_security_group
...
The `os_security_group` module would fail if there was no `description:`
attribute:
localhost | FAILED! => {
"changed": false,
"failed": true,
"msg": "Error creating security group larstest: Invalid input for
description. Reason: 'None' is not a valid string."
}
This commit makes the default description `''` rather than `None`.
2016-12-08 11:23:43 -05:00
186fbd7645
Remove duplicate shade requirement
2016-12-08 11:23:10 -05:00
73cb529d2c
Make sure we're always returning objects too
2016-12-08 11:23:10 -05:00
072110c757
Resolving secgroup.id issue in this module
...
secgroup['id'] was not being returned in all cases where the
specified security group exists.
2016-12-08 11:23:10 -05:00
8ff71a3fa8
Actually use changed variable
2016-12-08 11:23:09 -05:00
4125b576e9
Return the secgroup for all present states
2016-12-08 11:23:09 -05:00
89f95471f8
Return secgroup, not just id
2016-12-08 11:23:09 -05:00
6527f895a1
Remove 'result' value
...
This value is pretty much useless, and a holdover from the old
module code. Let's remove it.
2016-12-08 11:23:08 -05:00
7f0da6d851
Update the docstring for os_security_group
...
Indicate that idempotence is on security group names, and give
an example for updating a security group description.
2016-12-08 11:23:08 -05:00
640dbee643
Update for latest shade API
...
Shade version 0.7.0 introduces new API methods for creating,
deleting, and updating security groups. Let's use those and
clean up the module.
2016-12-08 11:23:08 -05:00
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
David Shrewsbury
Lars Kellogg-Stedman
Monty Taylor
dagnello