-
Notifications
You must be signed in to change notification settings - Fork 4k
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
(CdkPipeline): CdkPipeline source stage fails in a cross account setup #14156
Comments
Hey @ganeshnj , thanks for opening the issue. I'm trying to reproduce it right now. Before I do, I'm wondering whether the problem might be that you're creating a completely new CodeCommit repository here, and it's empty. Can you try pushing some commits to the repository, and see if that changes anything? Thanks, |
Repo with commits on main branch also has the same behavior. I kept the example simple so that during reproduction minimum manual steps are required. |
OK, I've managed to reproduce the error in my own setup. Digging in to what's causing it. |
OK. The problem seems to be the @ganeshnj can you let me know if it fixes it for you? Looks like the CodeCommit source Action needs those permissions in the cross-account case. I'll prepare a PR adding them back in. |
…ross-account Apparently, when removing the s3:PutObject* permissions in aws#12391, we broke the CodeCommitSourceAction when it's cross-account. Not entirely sure why is that permission required only when the action is cross-account, but I have confirmed this fixes the problem, so add an explicit call to `Bucket.grantPutAcl()` when the actions is cross-account. Fixes aws#14156
…ross-account (#14260) Apparently, when removing the s3:PutObject* permissions in #12391, we broke the CodeCommitSourceAction when it's cross-account. Not entirely sure why is that permission required only when the action is cross-account, but I have confirmed this fixes the problem, so add an explicit call to `Bucket.grantPutAcl()` when the actions is cross-account. Fixes #14156 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
Apologies, I couldn't get in time. I confirm, |
…ross-account (aws#14260) Apparently, when removing the s3:PutObject* permissions in aws#12391, we broke the CodeCommitSourceAction when it's cross-account. Not entirely sure why is that permission required only when the action is cross-account, but I have confirmed this fixes the problem, so add an explicit call to `Bucket.grantPutAcl()` when the actions is cross-account. Fixes aws#14156 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…ross-account (aws#14260) Apparently, when removing the s3:PutObject* permissions in aws#12391, we broke the CodeCommitSourceAction when it's cross-account. Not entirely sure why is that permission required only when the action is cross-account, but I have confirmed this fixes the problem, so add an explicit call to `Bucket.grantPutAcl()` when the actions is cross-account. Fixes aws#14156 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
CDKPipeline fails to clone repository that exists in a different AWS Account.
Reproduction Steps
Sample: https://github.com/ganeshnj/cdk-pipeline-cross-account-sample
Go to the source AWS account and create a
main
branch.Try again source stage.
Permissions issue
What did you expect to happen?
Source stage should succeed
What actually happened?
Source stage failed with permission error.
Environment
Other
This is 🐛 Bug Report
The text was updated successfully, but these errors were encountered: