Restore the Google Cloud mirror for CI resolution - #12700
Conversation
Reverts b47bcec. That commit commented the mirror out because the Google Cloud mirror of Central lagged a fresh release: apache-maven-4.0.0-rc-6 was still a 404 there days after Central served it. The mirror has since caught up, and resolving through it is measurably cheaper for CI than hitting Central directly on every job. The lag is a release-day concern, not a steady-state one, so the mirror is the better default between releases.
gnodet
left a comment
gnodet
left a comment
There was a problem hiding this comment.
Clean restoration of the Google Cloud mirror for CI. The mirror URL (https://maven-central.storage-download.googleapis.com/maven2/) is unchanged from the original addition by Tamas Cservenak in PR #11622, and this is a straightforward revert of the temporary disabling in commit b47bcec during the rc-6 sync lag.
The PR body honestly documents the accepted trade-off (mirror may lag on future RC drops, easily revertible). No security concerns — this is Google's well-known read-only CDN mirror of Maven Central.
Minor: the file still lacks a trailing newline (pre-existing since the original addition, not introduced here).
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
Claude Code on behalf of gnodet
2 participants
Reverts b47bcec, which commented out the
mimir.session.mirrorsline in.github/ci-mimir-session.properties.Why it was commented out
The Google Cloud mirror of Central lags a fresh release.
apache-maven-4.0.0-rc-6was still a 404 there days after Central served it, which broke CI during the rc-6 rollout.Why restore it now
The mirror has caught up — both of these are 200 today:
https://maven-central.storage-download.googleapis.com/maven2/.../apache-maven-4.0.0-rc-6.pomhttps://repo1.maven.org/maven2/.../apache-maven-4.0.0-rc-6.pomThe mirror is cheaper for CI than hitting Central directly on every job, and the lag is a release-day problem rather than a steady-state one — so the mirror is the better default between releases.
Caveat worth knowing
This does re-expose CI to the same 404 window when the next RC drops. If that happens, commenting the line out again is a one-line change.
A matching PR for
maven-4.0.xfollows, since that branch carries the same commit (c05b25d).