-
Notifications
You must be signed in to change notification settings - Fork 558
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
Build X fails to push to AWS ECR registry #576
Comments
@vbyrd That seems pretty clear to me: You're trying to push In this case - name: Get Tags for Image
id: metadata
uses: docker/metadata-action@v3
with:
images: ${{ steps.ecr-login.outputs.registry }}/${{ github.event.repository.name }}
tags: |
type=raw,value=latest
type=raw,value=main
type=sha |
Ah I see. I will try this shortly. I assumed that the metadata action step would use the registry authenticated in the login action step by default (much like the login action step uses the aws-credentials step automatically). It's a bit counter intuitive but I get why it was done (allow for multiple registry logins, ect.) Perhaps "docker/login-action@v1" could have an output of registry much like "amazon-ecr-login@v1" so that it is clear we are using "steps.docker-login.outputs.registry" in this metadata "images" step. Seems like it would still allow for multiple logins and metadata. |
I confirmed that it is working as expected. |
Troubleshooting
Before submitting a bug report, please read the Troubleshooting doc. DONE ✅
Behavior
Build fails when trying to push image to AWS ECR
Steps to reproduce this issue
Expected behavior
Build X should push images to AWS Private ECR successfully
Actual behavior
Build X debug logs show that is trying to push the image to Docker Registry instead of AWS ECR and thus failing
Configuration
Logs
Docker Login Log:
Build X Debug Log of Failure:
Docker build/tag/push calling buildx Log:
The text was updated successfully, but these errors were encountered: