Route Gradle distribution through MASS pull-through cache#11461
Conversation
Fix GRADLE_VERSION to match gradle-wrapper.properties (8.14.5). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master resultsStartup Time
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
AlexeyKuznetsov-DD
left a comment
There was a problem hiding this comment.
LGTM, I can see on CI in logs:
Downloading https://mass-read.us1.ddbuild.io/internal/artifact/services.gradle.org/distributions/gradle-8.14.5-bin.zip
.............10%.............20%.............30%.............40%.............50%.............60%..............70%.............80%.............90%.............100%
But let's get review from @bric3 too.
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 558c98298f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
bric3
left a comment
There was a problem hiding this comment.
Pre-approving, but Gitlab is probably "hydrating" gradle from a cache. I believe this might prevent gradle wrapper from fetching from MASS. I didn't look at the commit history so bear with me if I missed something, otherwise I'd say that in order to test this properly there could be a commit adding a line that deletes .gradle before the main job script executes. Then remove that change before adding the PR to the merge queue.
| DEPENDENCY_CACHE_POLICY: pull | ||
| BUILD_CACHE_POLICY: pull | ||
| GRADLE_VERSION: "8.14.4" # must match gradle-wrapper.properties | ||
| GRADLE_VERSION: "8.14.5" # must match gradle-wrapper.properties |
@bric3 AFAIK nightly |
|
@AlexeyKuznetsov-DD Yeah I meant only for this PR. |
|
/merge --commit-message "Route Gradle distribution through MASS pull-through cache" |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
|
@AlexeyKuznetsov-DD Thanks for fixing and merging |
Summary
mass-read.us1.ddbuild.io) instead of directly hittingservices.gradle.orgMASS_READ_URLCI variable pointing to the prod MASS read endpointGRADLE_VERSIONto match the version ingradle/wrapper/gradle-wrapper.properties(8.14.5)How it works
A
sedinbefore_scriptrewritesgradle-wrapper.propertiesat CI time:MASS pull-through is already enabled in prod (
mass-pullthrough-enabledfeature flag). Theservices.gradle.orgdomain is already in the MASS egress allowlist.Test plan
distributionUrlPart of DPCYMGMT-3608.
🤖 Generated with Claude Code