ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-07-23 05:10:22 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Merge pull request #20953 from surfuga/fix-20951-ec2_vpc_net-multi_ok

Browse source 
Fixes #20951 fixed multi_ok in ec2_vpc_net.py (var multi usage in vpc_exists function)
This commit is contained in:
s-hertel 2017-02-09 14:24:51 -05:00 • committed by GitHub
commit 0e4f48d684
1 changed files with 6 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

11
lib/ansible/modules/cloud/amazon/ec2_vpc_net.py
View file

@ -118,9 +118,9 @@ def boto_exception(err):
return error return error
def vpc_exists(module, vpc, name, cidr_block, multi): def vpc_exists(module, vpc, name, cidr_block, multi):
"""Returns True or False in regards to the existence of a VPC. When supplied """Returns None or a vpc object depending on the existence of a VPC. When supplied
with a CIDR, it will check for matching tags to determine if it is a match with a CIDR, it will check for matching tags to determine if it is a match
otherwise it will assume the VPC does not exist and thus return false. otherwise it will assume the VPC does not exist and thus return None.
""" """
matched_vpc = None matched_vpc = None
@ -130,11 +130,12 @@ def vpc_exists(module, vpc, name, cidr_block, multi):
e_msg=boto_exception(e) e_msg=boto_exception(e)
module.fail_json(msg=e_msg) module.fail_json(msg=e_msg)
if len(matching_vpcs) == 1: if multi:
return None
elif len(matching_vpcs) == 1:
matched_vpc = matching_vpcs[0] matched_vpc = matching_vpcs[0]
elif len(matching_vpcs) > 1: elif len(matching_vpcs) > 1:
if multi: module.fail_json(msg='Currently there are %d VPCs that have the same name and '
module.fail_json(msg='Currently there are %d VPCs that have the same name and '
'CIDR block you specified. If you would like to create ' 'CIDR block you specified. If you would like to create '
'the VPC anyway please pass True to the multi_ok param.' % len(matching_vpcs)) 'the VPC anyway please pass True to the multi_ok param.' % len(matching_vpcs))