terraform-aws-ecr
terraform-aws-ecr copied to clipboard
feat: add organizations as readonly access
what
- Add the ability to have organizations as trustees (read-only) for the ECR repository
why
- As described in #82, it's sometimes useful to allow an entire organization to consume images from a centralized repository
references
- closes #82
Although functionally the PR is ready I want to update the readme too. Is this done automatically by the pipeline? Sorry, I couldn't find instructions on how to do it automatically.
This pull request is now in conflict. Could you fix it @dragosmc? 🙏
Any update on this? @dragosmc
Apologies, didn't get a chance to revisit and address the comments. I expect to have an updated PR over the weekend.
I believe this is ready. Unfortunately, I cannot run the tests locally due to an issue with bats on MacOS
This pull request is now in conflict. Could you fix it @dragosmc? 🙏
The feature introduced by this PR is very useful. @dragosmc, please could you rebase the PR?
@dragosmc friendly ping on rebasing this as it does seem like people want this. If you can get this updated and conflicts resolved, then I'll review and we'll get this merged. Feel free to ping me when it's ready. Thanks!
Hi @Gowiem - I'll get this rebased and changed over the weekend. 👍
I renamed my fork's branch and this PR was closed automatically. A new one is open here https://github.com/cloudposse/terraform-aws-ecr/pull/106