Skip to content

Commit d8bbc1c

Browse files
authored
feat(codepipeline): add usePipelineRoleForActions field support in L2 (#33961)
### Issue # 27605 #27605 ### Reason for this change Lots of customer reported they don't want such many new roles generated in cdk level. Current cdk library will help customer to generate a action role if no role configured. But in Codepipeline backend logic, the pipeline service role will be used as default action role, which the cdk library doesn't align. This pr will support customer to have more alternatives and won't break anything ### Description of changes add `usePipelineRoleForActions` field support in L2 construct, and modify the function `AddAction` and `ProduceAction` to make sure this flag covers everything needed ### Describe any new or updated permissions being added No ### Description of how you validated changes Unit test/Integ test/local deployment ### Checklist - [Y] 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 d02e64a commit d8bbc1c

File tree

13 files changed

+32181
-4
lines changed

13 files changed

+32181
-4
lines changed

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.pipeline-use-pipeline-role-for-actions.js.snapshot/asset.97484721f29e34bf38d7a459804dd2d2a8dea6f8c27d7531e215bf4274fbc895.bundle/index.js

+30,674
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-codepipeline-actions/test/integ.pipeline-use-pipeline-role-for-actions.js.snapshot/aws-cdk-integ-test-service-roles-for-actions.assets.json

+20
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)