Commit Graph

225 Commits (1a298f794812c1a7e1ebc9d76cc10bbc57499572)

Author SHA1 Message Date
Brian Coca fd59dccdd7 Merge pull request #2681 from timrchavez/pass_network_argument
Pass network argument to add_ips_to_server
9 years ago
Dmitry Labutin 35d32c9378 Fix issue #2721: os_object parameters and docs don't align (file/filename) 9 years ago
Brian Coca 907b7f7cf4 fix bad linebreak 9 years ago
Michael Scherer 90f8228fb1 Fix #2629, replace choices=BOOLEANS by type='bool' 9 years ago
Sam Thursfield 6e9adc4687 os_server: Add some error checking for the 'nics' parameter
If this parameter was not of the right type, the module would fail with a
traceback, with a "AttributeError: 'str' object has no attribute 'get'"
exception.

It now gives a proper error message on type errors.
9 years ago
Alberto Gireud 9cc67e45a6 Update root_volume variable 9 years ago
Toshio Kuratomi f04cd88d22 Fix os_server docs build 9 years ago
Donovan Jones cf061dd93a fix typo in os_server security_groups description 9 years ago
Monty Taylor 187a4bd505 Add support for network, boot_from_volume and volumes
nics is a great flexible parameter, but it's wordy. Shade now supports
a simple parameter too, which is just "network" and takes a name or id.
Add passthrough support.

In addition to supporting booting from a pre-existing volume, nova and
shade both support the concept of booting from volume based on an image.
Pass the parameters through.

Shade supports boot-time attachment of additional volumes for OpenStack
instances. Pass through the parameter so that ansible users can also
take advantage of this.
9 years ago
Timothy R. Chavez 05e16c950d Pass network argument to add_ips_to_server
The `network` argument needs to be passed through `add_ips_to_server`
so that the default value can be properly overridden from ansible.
9 years ago
Timothy R. Chavez b9fe8166fd Get new server object after adding floating IP
We need a new server object once we add the floating ip, otherwise we
will be operating with the older server object pre-floating-ip
assignment.
9 years ago
Markus Suonto 32edc2f56f fixed quantum_ modules to work with minimum access rights if greater access rights are not needed 9 years ago
David Shrewsbury d13741314e Bug fix for os_image and min_disk/min_ram
The min_disk and min_ram parameters were not being passed to
the shade API. They also need to be integer values. Also
updated the description of these parameters for better
clarification.
9 years ago
David Shrewsbury 2a306c6b8c Fix os_user_group module
This module had a couple of errors in it. Also added check mode
support.
9 years ago
Monty Taylor ccb39767cf Use add_ips_to_server API
This module is still using an old pre-release API and needs to update
to use the current API.

Co-Authored-By: Marton Kiss <marton.kiss@gmail.com>
9 years ago
Brian Coca 1f9c204178 Merge pull request #2379 from larsks/feature/os_server_facts_returns_list
os_server_facts returns facts about multiple servers
9 years ago
Greg DeKoenigsberg 57dcf2c9dd @lorin stepping down as maintainer 9 years ago
Lars Kellogg-Stedman e70002d2f9 os_server_facts returns facts about multiple servers
have `os_server_facts` call `list_servers` rather than `get_server`, and
treat the `server` parameter as a wildcard pattern.  This permits one to
get facts on a single server:

    - os_server:
        server: webserver1

On mutiple servers:

    - os_server:
        server: webserver*

Or on all servers:

    - os_server:

Introduces a `detailed` parameter to request additional server details
at the cost of additional API calls.
9 years ago
Brian Coca 30951e2811 Merge pull request #2345 from larsks/bug/port-allow-secgroup-list
allow os_port to accept a list of security groups
9 years ago
Brian Coca 7c1fcd8e13 Merge pull request #2349 from larsks/bug/router-should-return-id
make os_router return a top level 'id' key
9 years ago
Brian Coca ad8fd86467 Merge pull request #2338 from larsks/bug/flavor-should-return-id
make os_flavor return a top-level 'id' key
9 years ago
Brian Coca 0e0c28bfb1 Merge pull request #2344 from larsks/bug/allow-secgroup-list
allow os_server to accept a list of security groups
9 years ago
Brian Coca 3199a73927 Merge pull request #2339 from larsks/bug/keypair-should-return-id
make os_keypair return a top level 'id' key
9 years ago
Brian Coca 59f11aa973 Merge pull request #2336 from larsks/feature/os-subnet-returns-something
return information about created subnet
9 years ago
Lars Kellogg-Stedman d82460a372 make os_router return a top level 'id' key
make os_router return a top-level 'id' key, much like other
os_* resources.
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
Brian Coca 12b0582f9e Merge pull request #2343 from larsks/bug/secgroup-allow-empty-description
allow empty description attribute for os_security_group
9 years ago
Brian Coca 49a66eb8d2 Merge pull request #2337 from larsks/bug/os-network-always-unchanged
make os_network correctly report changed status
9 years ago
Lars Kellogg-Stedman b0c10a7d31 allow os_server to accept a list of security groups
This commit allows the `security_groups` parameter of the `os_server`
module to be either a YAML list or a common-delimited string (much like
the `nics` attribute).  E.g., this:

    - os_nova_server:
        [...]
        security_groups:
          - default
          - webserver

Or this:

    - os_nova_server:
        [...]
        security_groups: default,webserver
9 years ago
Lars Kellogg-Stedman da0fbfc564 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`.
9 years ago
Lars Kellogg-Stedman 377811dac2 make os_keypair return a top level 'id' key
make os_keypair return a top-level 'id' key, much like other os_*
resources.
9 years ago
Lars Kellogg-Stedman 8368da4297 make os_flavor return a top-level 'id' key
make os_flavor return a top-level 'id' key, much like other os_*
resources.
9 years ago
Lars Kellogg-Stedman c54c5c8234 make os_network correctly report changed status
The `os_network` module was incorrectly returning changed=False whether
or not the network was created.  This commit makes the changed return
value useful.
9 years ago
Lars Kellogg-Stedman d73f5a4adb return information about created subnet
make os_subnet behave like os_network in terms of returning information
about the created resource.  With this commit, os_subnet will return the
created subnet in `subnet` and the subnet id in `id`.
9 years ago
David Shrewsbury 5d73a9a4c5 Fix for routers without external interfaces 9 years ago
Matt Martz 2e49d89be7 Merge pull request #2328 from emonty/bug/is-public
Actually pass in is_public to create_image
9 years ago
Monty Taylor e7bdf7f7f6 Actually pass in is_public to create_image
Fixes #2325
9 years ago
Brian Coca 88e7dcf174 fixed mispelled description 9 years ago
Matt Martz dc1f0c6f9f Merge pull request #2319 from Shrews/os_user
Add OpenStack Keystone User module
9 years ago
David Shrewsbury fde149cbe8 Clarify password requirement and add return docs. 9 years ago
David Shrewsbury cdf7117f7e Add OpenStack Keystone User module
This is a replacement for PR #1598 and fixes #283
9 years ago
Brian Coca 0f308e1e07 Merge pull request #2275 from larsks/bug/2253
fix handling of nics argument
9 years ago
Brian Coca b73ffa3b6f Merge pull request #2310 from Shrews/os_router
Fix os_router to accept internal interfaces
9 years ago
David Shrewsbury 55e9dc73f5 Fix os_router to accept internal interfaces
Allow the 'interfaces' attribute to represent internal router
interfaces, composed of subnet names, and the 'external_fixed_ips'
attribute to represent external interface subnet/IP.
9 years ago
Lars Kellogg-Stedman 06098a9d69 fix handling of nics argument
The existing code was receiving a list of strings and erroneously
assuming it was being given a list of dictionaries, leading it to fail
with:

    AttributeError: 'str' object has no attribute 'get'

This commit corrects the list handling code to check the type of each
item and handle it appropriately.   Also, based on bcoca's comment
in #2253, thie code removes the special case for a string-only argument.

By transforming string arguments into dicts and then handling them like
any other dict argument, this also permits arguments of the form:

    nics: net-name=mynet

Or:

    nics: port-name=mynet

Previous versions of this code only supported `net-id` and `port-id` in
string specifications.
9 years ago
Monty Taylor 4590ba4e59 Make the auto ip parameter auto_ip
There was a parameter in the docs called 'public_ip' that didn't
actually exist. Additionally, auto_floating_ip is not consistent with
the underlying parameter which is auto_ip - for no good reason.

Add auto_ip as the real parameter, and then make public_ip and
auto_floating_ip as aliases for it for backwards compatability.

Fixes #2301
9 years ago
Brian Coca 5da7cf696c fixed results docs 9 years ago
Toshio Kuratomi 3af0d509c5 Some docs fixes 9 years ago
Brian Coca 7b21bc7144 Merge pull request #2238 from dagnello/server_meta_string
Adding string support to metadata argument
9 years ago
Brian Coca 6e54a5edc5 Merge pull request #2077 from dagnello/subnet_list_state
Adding os_subnets_facts module
9 years ago