Commit Graph

6 Commits (66e48f5eb4993baa74d125d182002e891ff65f33)

Author SHA1 Message Date
David Shrewsbury 827fd9acc2 Fix allowed_address_pairs and extra_dhcp_opts in os_port
Commit f71542c set the incorrect type for these two parameters to
dict when they are actually list of dicts.

Also, the extra_dhcp_opts was incorrectly named (without the terminal
's') and NEVER worked, so this was corrected.

Fixes #3301
9 years ago
David Shrewsbury f71542ccf2 Fix parameters where type is assumed.
A change is coming to Ansible where module params will default to str.
Many of our modules were taking advantage of this by not being explicit
about the type, so they will break when that change merges. This hopefully
catches those cases.
9 years ago
David Shrewsbury e8abea25c7 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.
9 years ago
Lars Kellogg-Stedman a2fe8dba68 allow os_port to accept a list of security groups
with this commit, the `security_groups` attribute for `os_port` will
accept either a common-delimited string or ` YAML list.  That is, either
this:

    - os_port:
        [...]
        security_groups: group1,group2

Or this:

    - os_port:
        [...]
        security_groups:
          - group1
          - group2
9 years ago
Toshio Kuratomi 3af0d509c5 Some docs fixes 9 years ago
dagnello 3365dad0d8 Adding os_port Module to openstack ansible modules 9 years ago