Skip to content
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

feat: Add BatchGet* to IAM read only policy #540

Merged
merged 3 commits into from
Dec 26, 2024

Conversation

moleus
Copy link
Contributor

@moleus moleus commented Dec 17, 2024

Description

docker pull does API request to ECR and requires BatchGetImage. With Read Only policy we expect that we can pull image from the registry.

This PR adds BatchGet* action to address this problem.

Motivation and Context

Breaking Changes

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

@bryantbiggs bryantbiggs changed the title fix: add missing BatchGet* action to pull from ECR feat: Add BatchGet* to IAM read only policy Dec 26, 2024
@bryantbiggs bryantbiggs merged commit d886f11 into terraform-aws-modules:master Dec 26, 2024
34 checks passed
antonbabenko pushed a commit that referenced this pull request Dec 26, 2024
## [5.51.0](v5.50.0...v5.51.0) (2024-12-26)

### Features

* Add `BatchGet*` to IAM read only policy ([#540](#540)) ([d886f11](d886f11))
@antonbabenko
Copy link
Member

This PR is included in version 5.51.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants