Merge branch 'asg_update' of https://github.com/jsmartin/ansible into jsmartin-asg_update

cloud/ec2_asg

@@ -69,7 +69,7 @@ options:
     default: None
   tags:
     description:
-      - List of tag dictionaries to use. Required keys are 'key', 'value'. Optional key is 'propagate_at_launch', which defaults to true.
+      - A list of tags to add to the Auto Scale Group. Optional key is 'propagate_at_launch', which defaults to true.
     required: false
     default: None
     version_added: "1.7"
@@ -99,6 +99,11 @@ EXAMPLES = '''
     max_size: 10
     desired_capacity: 5
     vpc_zone_identifier: 'subnet-abcd1234,subnet-1a2b3c4d'
+    tags:
+      - environment: production
+        propagate_at_launch: no
+
+deprecated method of expressing tags:
     tags:
       - key: environment
         value: production
@@ -169,10 +174,18 @@ def create_autoscaling_group(connection, module):
 
     asg_tags = []
     for tag in set_tags:
-        asg_tags.append(Tag(key=tag.get('key'),
+        if tag.has_key('key') and tag.has_key('value'):  # this block is to support depricated form
-             value=tag.get('value'),
+            asg_tags.append(Tag(key=tag.get('key'),
-             propagate_at_launch=bool(tag.get('propagate_at_launch', True)),
+                value=tag.get('value'),
-             resource_id=group_name))
+                propagate_at_launch=bool(tag.get('propagate_at_launch', True)),
+                resource_id=group_name))
+        else:
+            for k,v in tag.iteritems():
+                if k !='propagate_at_launch':
+                    asg_tags.append(Tag(key=k,
+                         value=v,
+                         propagate_at_launch=bool(tag.get('propagate_at_launch', True)),
+                         resource_id=group_name))
 
     if not as_groups:
         if not vpc_zone_identifier and not availability_zones:
@@ -211,15 +224,24 @@ def create_autoscaling_group(connection, module):
             existing_tags = as_group.tags
             existing_tag_map = dict((tag.key, tag) for tag in existing_tags)
             for tag in set_tags:
-                if 'key' not in tag:
+                if tag.has_key('key') and tag.has_key('value'):  # this is to support deprecated method
-                    continue
+                    if 'key' not in tag:
-                if ( not tag['key'] in existing_tag_map or
+                        continue
-                     existing_tag_map[tag['key']].value != tag['value'] or
+                    if ( not tag['key'] in existing_tag_map or
-                     ('propagate_at_launch' in tag and
+                        existing_tag_map[tag['key']].value != tag['value'] or
-                     existing_tag_map[tag['key']].propagate_at_launch != tag['propagate_at_launch']) ):
+                        ('propagate_at_launch' in tag and
-
+                        existing_tag_map[tag['key']].propagate_at_launch != tag['propagate_at_launch']) ):
-                    changed = True
+                        changed = True
-                    continue
+                        continue
+                else:
+                    for k,v in tag.iteritems():
+                        if k !='propagate_at_launch':
+                            if ( not k in existing_tag_map or
+                                existing_tag_map[k].value != v or
+                                ('propagate_at_launch' in tag and
+                                existing_tag_map[k].propagate_at_launch != tag['propagate_at_launch']) ):
+                                changed = True
+                                continue
             if changed:
                 connection.create_or_update_tags(asg_tags)