cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Remove unused vars and pass params dict to describe_peering_connections

Browse source
This commit is contained in:
Mike Mochan 2016-01-09 20:20:32 +10:00 committed by Matt Clay
parent deaf40d942
commit 439c4aa979
1 changed files with 4 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
lib/ansible/modules/extras/cloud/amazon/ec2_vpc_peer.py
View file

@ -161,10 +161,10 @@ except ImportError:
HAS_BOTO3 = False
def describe_peering_connections(vpc_id, peer_vpc_id, client):
def describe_peering_connections(params, client):
result = client.describe_vpc_peering_connections(Filters=[
{'Name': 'requester-vpc-info.vpc-id', 'Values': [vpc_id]},
{'Name': 'accepter-vpc-info.vpc-id', 'Values': [peer_vpc_id]}
{'Name': 'requester-vpc-info.vpc-id', 'Values': [params['VpcId']]},
{'Name': 'accepter-vpc-info.vpc-id', 'Values': [params['PeerVpcId']]}
])
if result['VpcPeeringConnections'] == []:
result = client.describe_vpc_peering_connections(Filters=[
@ -190,11 +190,7 @@ def create_peer_connection(client, module):
if module.params.get('peer_owner_id'):
params['PeerOwnerId'] = str(module.params.get('peer_owner_id'))
params['DryRun'] = module.check_mode
vpc_id = module.params.get('vpc_id')
peer_vpc_id = module.params.get('peer_vpc_id')
peer_owner_id = module.params.get('peer_owner_id', False)
peering_conns = describe_peering_connections(vpc_id, peer_vpc_id, client)
peering_conns = describe_peering_connections(params, client)
for peering_conn in peering_conns['VpcPeeringConnections']:
if is_active(peering_conn):
return (changed, peering_conn['VpcPeeringConnectionId'])