Description of changes:
Apparently this exception would cause the handlers to continuously retry and would emit internal failure metrics even thought this is more of a user error. Changing this to throw CfnInvalidRequestException instead
Testingcfn submit and observed the new error message:
Domain must contain 0 repositories before deletion (Service: Codeartifact, Status Code: 409, Request ID: 2d4eee90-712e-4eb5-b6f9-928c7145431b, Extended Request ID: null)
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Issue #, if available:
Description of changes: Apparently this exception would cause the handlers to continuously retry and would emit internal failure metrics even thought this is more of a user error. Changing this to throw
CfnInvalidRequestException
insteadTesting
cfn submit
and observed the new error message:By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.