-
Notifications
You must be signed in to change notification settings - Fork 13k
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
CI: use aws ECR insteand of Docker Hub #134135
base: master
Are you sure you want to change the base?
Conversation
I expect the authentication to fail, let's see. Probably we need to add the permissions |
This comment has been minimized.
This comment has been minimized.
After the authentication works, we should also check whether it is still possible to build these Dockerfiles locally after these changes. |
It's possible, I tried, you are subject to the unauthenticated rate limit, as dockerhub. This is discussed in the linked issue 👍 |
I see, ok :) Awesome. |
A job failed! Check out the build log: (web) (plain) Click to see the possible cause of the failure (guessed by this bot)
|
it might be that since I'm in a fork we can't use the secret to authenticate with aws? |
☔ The latest upstream changes (presumably #133809) made this pull request unmergeable. Please resolve the merge conflicts. |
☔ The latest upstream changes (presumably #134614) made this pull request unmergeable. Please resolve the merge conflicts. |
Close rust-lang/infra-team#176