-
Notifications
You must be signed in to change notification settings - Fork 2
feat: Restrict ECR repository resource-based policy based on pipeline mode (DBTP-3170) #1546
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
Merged
Merged
Changes from 3 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
b0d3a05
feat: Update IAM policy document to limit ECR permissions (DBTP-3170)
BenFielder 551a9f4
Add role names for github actions and paketo
BenFielder 2031ea4
Merge branch 'main' into rewrite-ecr-permissions
BenFielder 00b0f0b
allow image build role to delete
DeveloperConnor ab3102f
merge main
DeveloperConnor 7621040
Update policy to have conditional sections based on pipeline_mode
BenFielder 415a71c
Change name to match original file
BenFielder e83b4f0
Merge branch 'main' into rewrite-ecr-permissions
BenFielder e188d4a
Updates based on PR feedback
BenFielder f5d0aa9
Run tf fmt
BenFielder 6a7feaf
Update terraform/codebase-pipelines/ecr.tf
dragos441 35dc9da
Separate out pull permissions
BenFielder f480a3c
Move policy statements into no condition area
BenFielder cddda65
TF fmt
BenFielder 67c3a0e
Update tests for new ECR policy
BenFielder 26166eb
Change test condition star expected
BenFielder 7421106
Update tests to read correct part of values
BenFielder 8c05632
Fix issue with tests
BenFielder 9090f33
tf fmt tests
BenFielder 9a48388
fmt and fix test
BenFielder 3db1cb3
try to recreate local var
BenFielder 94fc9e3
fmt and add new test cases
BenFielder e1b23b3
Add spacing
BenFielder 74da15f
Add override data to fix uneeded part on iam file
BenFielder 963f1f9
Merge branch 'main' into rewrite-ecr-permissions
BenFielder File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.