-
Notifications
You must be signed in to change notification settings - Fork 167
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
Public ECR #116
Comments
The |
IIUC there's code change required to use aws.ECRPUBLIC service to get login credentials. Anyway, I'd love to see the confirmation from maintainers how we can publish to ECR Gallery with this awesome action! |
In case anyone has the same problem and is looking for a solution: https://github.com/docker/login-action can be used to login into public ECR. |
This action does not support IAM roles out of the box, does it? |
@danielpacak @RickyCook @allisaurus |
Any updates on this pls? |
This requires a code change, this actions at the moment only supports Private ECRs https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/ECRPUBLIC.html I believe the |
See [this issue](aws-actions/amazon-ecr-login#116)
See [this issue](aws-actions/amazon-ecr-login#116)
Would love for this to be implemented soon. Having to use two discrete actions to login to each ECR registry (public and private) is not great. |
Hi everyone! This has been here a while, we're going to get some things done around this very soon. Would you prefer that support for ECR Public is added to this action, or should we create a second action for ECR Public login and keep this one focused on ECR? Thanks! |
It would be great if this action supports both private and public repositories. The name is intuitive, and developers have already tried to use it for both cases. One similar action that supports different types of repositories is |
I prefer to add ECR Public in this action. |
When giving a public ECR like this (to push an image):
The step fails and gives the error:
The text was updated successfully, but these errors were encountered: