Skip to content

Commit 9ddc00a

Browse files
authored
fix(cloudwatch-actions): lambda permission use unique prefix id (#34269)
### Issue # (if applicable) Closes #33958 ### Reason for this change Duplicate id of alarm of multiple StepScalingPolicy ### Description of changes Update Lambda permission prefix to use `Names.uniqueId` Considering this requires resource destruction, please let me know if i need a feature flag for the fix. As there was another feature flag in the exact same place. ### Description of how you validated changes Unit + Integ ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 5ee092a commit 9ddc00a

11 files changed

+2231
-3
lines changed

packages/@aws-cdk-testing/framework-integ/test/aws-cloudwatch-actions/test/integ.lambda-alarm-multiple-stepscalingpolicy.js.snapshot/cdk.out

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-cloudwatch-actions/test/integ.lambda-alarm-multiple-stepscalingpolicy.js.snapshot/integ.json

Lines changed: 13 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-cloudwatch-actions/test/integ.lambda-alarm-multiple-stepscalingpolicy.js.snapshot/lambda-alarm-multiple-stepscalingpolicy.assets.json

Lines changed: 20 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)