diff options
| author | Sloane Hertel <shertel@redhat.com> | 2017-09-21 08:29:39 -0400 |
|---|---|---|
| committer | Ryan Brown <sb@ryansb.com> | 2017-09-21 08:29:39 -0400 |
| commit | 2fca9d5530e325303cc88fad66e68a35c8662669 (patch) | |
| tree | edf62dc3d969abb3429e9a117a6c918eb9a125f9 | |
| parent | 64f54fa6e2d4ffa0147fa276223afb812bdae506 (diff) | |
| download | ansible-2fca9d5530e325303cc88fad66e68a35c8662669.tar.gz | |
[cloud] Fix broken LimitExceeded check in iam_managed_policy (#30537) (#30626)
When policy versions exceed 5, we hit LimitExceeded. However,
the exception checking should use `e.response['Error']['Code']`
| -rw-r--r-- | lib/ansible/modules/cloud/amazon/iam_managed_policy.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ansible/modules/cloud/amazon/iam_managed_policy.py b/lib/ansible/modules/cloud/amazon/iam_managed_policy.py index 84bbc1ccce..ce4e1d1397 100644 --- a/lib/ansible/modules/cloud/amazon/iam_managed_policy.py +++ b/lib/ansible/modules/cloud/amazon/iam_managed_policy.py @@ -188,7 +188,7 @@ def get_or_create_policy_version(module, iam, policy, policy_document): version = iam.create_policy_version(PolicyArn=policy['Arn'], PolicyDocument=policy_document)['PolicyVersion'] return version, True except botocore.exceptions.ClientError as e: - if e['Error']['Code'] == 'LimitExceeded': + if e.response['Error']['Code'] == 'LimitExceeded': delete_oldest_non_default_version(module, iam, policy) try: version = iam.create_policy_version(PolicyArn=policy['Arn'], PolicyDocument=policy_document)['PolicyVersion'] |