dockerhub seems to have recently started to increase their rate limits further, beyond what is documented, even to the extent where pulling the same image twice at the same time results in a ban. See #31797 (comment)
Fix all issues by just using another mirror, as documented in https://cloud.google.com/artifact-registry/docs/pull-cached-dockerhub-images
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/31906.
See the guideline for information on the review process.
| Type | Reviewers |
|---|---|
| ACK | 0xB10C |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
Reviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
An alternative would probably be to configure the docker deamons of the self-hosted runners to use the mirror?
in e.g. /etc/docker/daemon.json
0{
1 "registry-mirrors": ["https://mirror.gcr.io"]
2}
or in /etc/containers/registries.conf.d/gcr.conf for podman
0[[registry]]
1location = "https://mirror.gcr.io"
2prefix = "docker.io"
5@@ -6,7 +6,7 @@
6
7 export LC_ALL=C.UTF-8
8
9-export CI_IMAGE_NAME_TAG="docker.io/ubuntu:24.04"
10+export CI_IMAGE_NAME_TAG="mirror.gcr.io/ubuntu:24.04"
An alternative would probably be to configure the docker deamons of the self-hosted runners to use the mirror?
I haven’t tested this and it seems like more hassle along with the risk of reintroducing the bug fixed in #28330. I guess the bug is fixed in the meantime via CI_IMAGE_PLATFORM, but I haven’t tried it.
Though, I don’t mind either way. I am happy to close this pull and review a different one, if someone creates one.
An alternative would probably be to configure the docker deamons of the self-hosted runners to use the mirror?
I haven’t tested this
I tested it and your suggestion failed locally for me:
0# podman pull docker.io/ubuntu/24.04
1Error: loading drop-in registries configuration "/etc/containers/registries.conf.d/gcr.conf": invalid location 'https://mirror.gcr.io': URI schemes are not supported
2# cat /etc/containers/registries.conf.d/gcr.conf
3[[registry]]
4location = "https://mirror.gcr.io"
5prefix = "docker.io"
(then removing https, it will still fail due to pulling from docker.io)
Also, I don’t see any benefit over this pull request? This pull request is a single (hopefully) one-time change, fixing the problem for all CI workers, CI machines, and other current (and future) CI deployments.
Agree that this is the better than having to modify all CI machines and deployments. I did some light testing on my runners and didn’t see any problems in fetching the images. The CI passing here indicate the same. While switching the GitHub Actions tasks to the new images probably isn’t necessary due to being exempt from the rate-limiting (https://github.com/bitcoin/bitcoin/pull/31906#discussion_r1961960848), I don’t think it hurts.
ACK fa8de4706a01322fd8ab8e491d297b97f001ecff