Skip to content

Enable Develocity remote build cache#1033

Open
ribafish wants to merge 1 commit intoapache:trunkfrom
ribafish:enable-remote-build-cache
Open

Enable Develocity remote build cache#1033
ribafish wants to merge 1 commit intoapache:trunkfrom
ribafish:enable-remote-build-cache

Conversation

@ribafish
Copy link
Copy Markdown
Contributor

Summary

Enable the Develocity remote build cache for all builds. Push is limited to CI (GITHUB_ACTIONS), so CI populates the cache and developers benefit from it on local builds.

The remote cache was explicitly disabled (enabled = false). This change sets enabled = true with push = isCI, following the standard Develocity configuration pattern.

Enable the Develocity remote build cache for all builds, with push
limited to CI. This allows developers to reuse cached outputs from CI
builds, reducing local build times.
@ribafish
Copy link
Copy Markdown
Contributor Author

ribafish commented Apr 1, 2026

Hi @JacquesLeRoux , thank you for the update. No rush, I completely understand it's not a pressing issue, just a quality of life improvement 👍

@jacopoc
Copy link
Copy Markdown
Contributor

jacopoc commented Apr 5, 2026

@ribafish Now that I’ve merged your other pull request #1028, which made the test task cache-friendly, we could also merge this pull request. Currently, our CI workflow runs:

./gradlew check javadoc

However, our goal is to switch to the following as soon as we resolve some build issues:

./gradlew pullAllPluginsSource check javadoc loadAll testIntegration

My only concern is that merging this PR before we understand and fix the issues preventing us from running the new tasks might make it more difficult to debug why they fail on CI but succeed locally. Moreover, since loadAll is not currently cache-friendly, enabling it may again lead to a significant number of cache misses.

I’ll try to investigate these issues, but if you have any advice, please let me know. Thank you in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants