-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(CDK Pipelines): allow passing a Role to CodePipeline
#18549
Comments
You can pass a |
Yep, that's what I recommended the customer do, but it's pretty non-trivial to figure out. A property in the construct props would make it more discoverable. |
Still don't really agree. We should control better for policy size, or injecting, instead. |
For reference: I just deployed a fix following the advice by @skinny85 which essentially (besides a narrower policy) looks like the code below: The stack redeployed and kept its resources. I had to pull out the cross account property tough.
Update: Just found the docs for that: Update 2: I somehow broke stuff, so not sure if it actually works |
Hello @rix0rrr , I am trying to solve for the same problem; i.e to pass a role to the Code Snippet.
Error
Could you kindly help on the same? Thanks, |
@ac259 what version of CDK are you using? |
@skinny85 I was using an older 2.x version but I upgraded to the 2.47.0 and tested again.
|
I'm asking about the libraries, though; not the CDK CLI version. You'll find their version in the It was released in version |
@skinny85 thank you so much - I was on an earlier version of the library. Was able to get it to work! |
Description
Because of #16244, customers have to sometimes create a Role manually and control its permissions.
We should make that easy in
CodePipeline
.Use Case
See above.
Proposed Solution
Add a new property
role
to theCodePipeline
construct.Other information
No response
Acknowledge
The text was updated successfully, but these errors were encountered: