From 3b9cf8270f04ac794c974688707887919b76ea8a Mon Sep 17 00:00:00 2001 From: lwade Date: Mon, 18 Feb 2013 10:14:09 +0000 Subject: [PATCH] Correct ec2_vol doc errors --- library/ec2_vol | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/library/ec2_vol b/library/ec2_vol index b5b9335ce0b..c25cff49559 100644 --- a/library/ec2_vol +++ b/library/ec2_vol @@ -49,18 +49,24 @@ options: examples: - code: 'local_action: ec2_vol instance=XXXXXX volume_size=5 device_name=sdd' description: "Simple playbook example" - - code: '- name: Launch instances + - code: | + - name: Launch instances local_action: ec2 keypair=$keypair image=$image wait=true count=3 register: ec2 - name: Create volumes and attach local_action: ec2_vol instance=${item.id} volume_size=5 with_items: ${ec2.instances} - register: ec2_vol' - description: "Advanced playbook example" + register: ec2_vol + description: "Advanced - attaching multiple volumes to multiple instances" requirements: [ "boto" ] author: Lester Wade ''' +# Note: this module needs to be made idempotent. Possible solution is to use resource tags with the volumes. +# if state=present and it doesn't exist, create, tag and attach. +# Check for state by looking for volume attachment with tag (and against block device mapping?). +# Would personally like to revisit this in May when Eucalyptus also has tagging support (3.3). + import sys import time