Commit Graph

23 Commits (c37fa5ddce75b15838d16c0ef57835f5b6f7e411)

Author SHA1 Message Date
James Tanner ad837709bd Addresses #5023 Fix import comments 11 years ago
James Tanner 6000d636b3 Fixes #5023 Convert all modules to use new snippet import pattern 11 years ago
Michael DeHaan 84ff24d92c Merge pull request #4346 from dmkoch/digitalocean-droplet-options
Add "virtio" and "private_networking" options for DigitalOcean droplets
11 years ago
Michael DeHaan 119b6d73dd Remove examples of module antipattern of setting the choices array for boolean types. 11 years ago
Daniel Koch cdc5e04d7b Add check to make sure dopy is at least version 0.2.2 for digital_ocean module 11 years ago
Daniel Koch 8c0e8354bf Add "virtio" and "private_networking" options for DigitalOcean droplets
The dopy manager defaults to False for virtio, but the DigitalOcean web interface defaults to True, which is a pretty safe default and sensible considering the performance gains.

Private networks are a new feature currently available only in the NYC2 region as of Oct 2, 2013.

NOTE: dopy >= 0.2.2 is required for these options to work.
11 years ago
James Cammarata 484297cfa8 Added a version_added string to the new digital_ocean module param 11 years ago
Ralph Bean a9a2a96218 Add a new unique_name param to the digital_ocean module for idempotence.
As it stands now, it is difficult to write idempotent tasks for digital
ocean droplets.  Digital ocean assigns new nodes a random id when they
are provisioned and that id is the only key that can be used to identify
it in subsequent runs of that play.

The workflow previously involved manual intervention:

- write a play defining a new node with no specified id
- run it, collect the randomly assigned id by hand
- modify the play to add the id by hand so future runs don't create
  duplicate nodes
- perform future re-runs that check if the node exists (by its id)
  - if it does exist then do nothing.
  - if it does not exist, then create it and return a *new random id*
  - collect the new random id by hand, modify the playbook file, and
    start all over.

Its a huge pain.

The modifications in this commit allow you to use the 'hostname' as a
primary key for idempotence with digital ocean.  By default, digital
ocean will let you create as many hosts with the same hostname as you
like.  Here, we provide an option to constrain the user to using only
unique hostnames.

The workflow will now look like:

- write a play defining a new node with a specified hostname and
  "unique_name: true""
- run it, create the new node and move on.
- re-run it, notice that a node with that hostname is already created
  and move on.
11 years ago
Ralph Bean 6ecf41530b Fix an example with a yaml syntax error. 11 years ago
Ralph Bean d782d91b80 Fix some "make pep8" errors in the digital_ocean module. 11 years ago
Ralph Bean a1ea15141e Remove trailing whitespace in digital ocean module. 11 years ago
Ralph Bean 3eab663f2e Remove duplicate docs in digital ocean module.
The EXAMPLES block here has two copies of the same docs,
one nicely formatted, the other less so.

It looks like a pass was made to clean up the docs but the old
cruftier ones were never removed.
11 years ago
Bryan Larsen cea812c3a7 update_attr should wait for the full wait timeout before checking for a missing IP addres 12 years ago
Bryan Larsen 848b38fc7f digital_ocean doc update 12 years ago
Bryan Larsen bdbc963736 digital_ocean: don't run update_attr if wait=no
Sometimes when using digital_ocean with wait=no I get the error "No ip is found".  But with wait=no I wouldn't expect there to be any IP, that gets allocated later.   However, looking at the code, it turns out that with even with wait=no it waits up to 10 seconds for an IP to be allocated.  We could wait longer, but with wait=no that seems like the wrong choice; it's easy enough to grab an IP later with a wait=yes command.

To make this change I removed the call to update_attr in @classmethod add.  An add is always followed by an ensure_powered_on which will do the update_attr if wait=yes.   It would be possible to instead do a call to update_attr with no retries and ignore the errors but I figured it would be better to be consistently not return an IP than to sometimes return it and sometimes not.   Inconsistent behaviour makes debugging deployment scripts very difficult.
12 years ago
Benjamin Schwarze 31bc25063e fix wrong argument_spec in digital_ocean module
as 'required' and 'default' are mutually exclusive removed 'required' in preference of 'default'
12 years ago
Michael DeHaan 995ef374db Fixup versions of newly added modules. 12 years ago
Michael DeHaan 3f8aa8aec5 Fix up some module docs formatting errors. 12 years ago
Michael DeHaan 231d9b101d YAML fix 12 years ago
Michael DeHaan faaa90ed03 Merge branch 'ocean' into devel
Conflicts:
	library/cloud/digital_ocean
12 years ago
Michael DeHaan c9c45ac597 Reformat examples 12 years ago
Vincent Viallet cc1cb01540 Rename digital ocean module to digital_ocean, add documentation. 12 years ago
Vincent Viallet d4b5122ad9 Rename digital ocean module to digital_ocean, add documentation. 12 years ago