wbrawner/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

VPC peer module unhandled exception on bad peering ID (#45306)

Browse source 
* Improve error handling for VPC peer module when a bad peering-id is specified

* changelog
This commit is contained in:
Ryan Brown 2018-11-29 08:54:16 -05:00 committed by Sloane Hertel
parent 7f7c059a04
commit af6427bf40
2 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/ec2_vpc_peer_describe_peer_with_exception_handling.yaml Normal file
View file

@ -0,0 +1,2 @@
bugfixes:
- Handle ClientError exceptions when describing VPC peering connections.

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

@ -220,9 +220,11 @@ except ImportError:
pass # caught by imported HAS_BOTO3
import distutils.version
import traceback
from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.ec2 import boto3_conn, ec2_argument_spec, get_aws_connection_info, HAS_BOTO3
from ansible.module_utils.aws.core import is_boto3_error_code
def tags_changed(pcx_id, client, module):
@ -328,8 +330,13 @@ def remove_peer_connection(client, module):
def peer_status(client, module):
params = dict()
params['VpcPeeringConnectionIds'] = [module.params.get('peering_id')]
vpc_peering_connection = client.describe_vpc_peering_connections(**params)
return vpc_peering_connection['VpcPeeringConnections'][0]['Status']['Code']
try:
vpc_peering_connection = client.describe_vpc_peering_connections(**params)
return vpc_peering_connection['VpcPeeringConnections'][0]['Status']['Code']
except is_boto3_error_code('InvalidVpcPeeringConnectionId.Malformed') as e: # pylint: disable=duplicate-except
module.fail_json(msg='Malformed connection ID: {0}'.format(e), traceback=traceback.format_exc())
except botocore.exceptions.ClientError as e: # pylint: disable=duplicate-except
module.fail_json(msg='Error while describing peering connection by peering_id: {0}'.format(e), traceback=traceback.format_exc())
def accept_reject(state, client, module):