Renamed instance_tags to resource_tags based on community feedback.
This commit is contained in:
parent
3c4b14523b
commit
e56cffe3a5
1 changed files with 6 additions and 6 deletions
|
@ -46,7 +46,7 @@ options:
|
||||||
choices: [ "yes", "no" ]
|
choices: [ "yes", "no" ]
|
||||||
subnets:
|
subnets:
|
||||||
description:
|
description:
|
||||||
- "A dictionary array of subnets to add of the form: { cidr: ..., az: ... , instance_tags: ... }. Where az is the desired availability zone of the subnet, but it is not required. Tags (i.e.: instance_tags) is also optional and use dictionary form: { "Environment":"Dev", "Tier":"Web", ...}. All VPC subnets not in this list will be removed."
|
- "A dictionary array of subnets to add of the form: { cidr: ..., az: ... , resource_tags: ... }. Where az is the desired availability zone of the subnet, but it is not required. Tags (i.e.: resource_tags) is also optional and use dictionary form: { "Environment":"Dev", "Tier":"Web", ...}. All VPC subnets not in this list will be removed."
|
||||||
required: false
|
required: false
|
||||||
default: null
|
default: null
|
||||||
aliases: []
|
aliases: []
|
||||||
|
@ -137,13 +137,13 @@ EXAMPLES = '''
|
||||||
subnets:
|
subnets:
|
||||||
- cidr: 172.22.1.0/24
|
- cidr: 172.22.1.0/24
|
||||||
az: us-west-2c
|
az: us-west-2c
|
||||||
instance_tags: { "Environment":"Dev", "Tier" : "Web" }
|
resource_tags: { "Environment":"Dev", "Tier" : "Web" }
|
||||||
- cidr: 172.22.2.0/24
|
- cidr: 172.22.2.0/24
|
||||||
az: us-west-2b
|
az: us-west-2b
|
||||||
instance_tags: { "Environment":"Dev", "Tier" : "App" }
|
resource_tags: { "Environment":"Dev", "Tier" : "App" }
|
||||||
- cidr: 172.22.3.0/24
|
- cidr: 172.22.3.0/24
|
||||||
az: us-west-2a
|
az: us-west-2a
|
||||||
instance_tags: { "Environment":"Dev", "Tier" : "DB" }
|
resource_tags: { "Environment":"Dev", "Tier" : "DB" }
|
||||||
internet_gateway: True
|
internet_gateway: True
|
||||||
route_tables:
|
route_tables:
|
||||||
- subnets:
|
- subnets:
|
||||||
|
@ -281,7 +281,7 @@ def create_vpc(module, vpc_conn):
|
||||||
if add_subnet:
|
if add_subnet:
|
||||||
try:
|
try:
|
||||||
new_subnet = vpc_conn.create_subnet(vpc.id, subnet['cidr'], subnet.get('az', None))
|
new_subnet = vpc_conn.create_subnet(vpc.id, subnet['cidr'], subnet.get('az', None))
|
||||||
new_subnet_tags = subnet.get('instance_tags', None)
|
new_subnet_tags = subnet.get('resource_tags', None)
|
||||||
if new_subnet_tags:
|
if new_subnet_tags:
|
||||||
# Sometimes AWS takes its time to create a subnet and so using new subnets's id
|
# Sometimes AWS takes its time to create a subnet and so using new subnets's id
|
||||||
# to create tags results in exception.
|
# to create tags results in exception.
|
||||||
|
@ -422,7 +422,7 @@ def create_vpc(module, vpc_conn):
|
||||||
|
|
||||||
for sn in current_subnets:
|
for sn in current_subnets:
|
||||||
returned_subnets.append({
|
returned_subnets.append({
|
||||||
'instance_tags': dict((t.name, t.value) for t in vpc_conn.get_all_tags(filters={'resource-id': sn.id})),
|
'resource_tags': dict((t.name, t.value) for t in vpc_conn.get_all_tags(filters={'resource-id': sn.id})),
|
||||||
'cidr': sn.cidr_block,
|
'cidr': sn.cidr_block,
|
||||||
'az': sn.availability_zone,
|
'az': sn.availability_zone,
|
||||||
'id': sn.id,
|
'id': sn.id,
|
||||||
|
|
Loading…
Reference in a new issue