Updated tag behavior.
This commit is contained in:
parent
fe3f6b3253
commit
e36ef4defb
1 changed files with 36 additions and 14 deletions
|
@ -69,7 +69,7 @@ options:
|
||||||
default: None
|
default: None
|
||||||
tags:
|
tags:
|
||||||
description:
|
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
|
required: false
|
||||||
default: None
|
default: None
|
||||||
version_added: "1.7"
|
version_added: "1.7"
|
||||||
|
@ -98,6 +98,11 @@ EXAMPLES = '''
|
||||||
max_size: 10
|
max_size: 10
|
||||||
desired_capacity: 5
|
desired_capacity: 5
|
||||||
vpc_zone_identifier: 'subnet-abcd1234,subnet-1a2b3c4d'
|
vpc_zone_identifier: 'subnet-abcd1234,subnet-1a2b3c4d'
|
||||||
|
tags:
|
||||||
|
- environment: production
|
||||||
|
propagate_at_launch: no
|
||||||
|
|
||||||
|
deprecated method of expressing tags:
|
||||||
tags:
|
tags:
|
||||||
- key: environment
|
- key: environment
|
||||||
value: production
|
value: production
|
||||||
|
@ -167,10 +172,18 @@ def create_autoscaling_group(connection, module):
|
||||||
|
|
||||||
asg_tags = []
|
asg_tags = []
|
||||||
for tag in set_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 as_groups:
|
||||||
if not vpc_zone_identifier and not availability_zones:
|
if not vpc_zone_identifier and not availability_zones:
|
||||||
|
@ -209,15 +222,24 @@ def create_autoscaling_group(connection, module):
|
||||||
existing_tags = as_group.tags
|
existing_tags = as_group.tags
|
||||||
existing_tag_map = dict((tag.key, tag) for tag in existing_tags)
|
existing_tag_map = dict((tag.key, tag) for tag in existing_tags)
|
||||||
for tag in set_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:
|
if changed:
|
||||||
connection.create_or_update_tags(asg_tags)
|
connection.create_or_update_tags(asg_tags)
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue