-
-
Notifications
You must be signed in to change notification settings - Fork 114
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
[docker] Failed to authorize in new workflow #1255
Comments
Actually, I'm not sure if a personal access token is being used as a secret for the DOCKER_PASSWORD variable. Looking at containerd/nerdctl#3378, that seems to be the proposed solution, and it matches my own working setup. |
I've managed to reproduce the issue. Indeed, using the regular password instead of an access token while 2FA is enabled causes the 401 Unauthorized error. I've also made sure that switching the password in the repo secrets to a read/write personal access token solves the issue. |
I thought that it was already using a PAT, but maybe I'm thinking of all my other projects, and this one predates them... shall give that a go /o/ |
Still failing with a fresh token D: |
Looks like all the different things that I tried for DOCKER_REGISTRY were incorrect, and the default seems to be correct (but the default isn't documented anywhere so I don't know what that value actually is) If somebody wants to be able to push to a different registry we can re-add that flag, if somebody tells me what the correct value is 😅 |
Huh, that's weird! Never would've guessed that was the reason. But now that it builds I tried pulling the image to my Raspberry Pi, and it works! 🎉 |
Follow-up of #1251
The latest builds on main seem to be failing due to an unauthorized error related to a missing token.
The exact error is:
This is not a login issue, as that step succeeds in the workflow. Nor is this a permission issue, as a read+write token is all that's necessary.
The text was updated successfully, but these errors were encountered: