Skip to content
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

Activate Develocity's Build Cache #22523

Merged
merged 7 commits into from
Mar 4, 2025
Merged

Conversation

adpi2
Copy link
Member

@adpi2 adpi2 commented Feb 5, 2025

  1. Enable read and write to local cache for all users
  2. Enable read from remote cache for all users
  3. Enable write to remote cache on CI only
  4. Disable test caching on all projects: We generally don't rerun tests if nothing changed. When users do run the tests locally, they probably want to see their executions.

ping @hamzaremmal

@tgodzik
Copy link
Contributor

tgodzik commented Feb 5, 2025

Och, awesome! Will we be able to use that locally?

@adpi2
Copy link
Member Author

adpi2 commented Feb 5, 2025

Och, awesome! Will we be able to use that locally?

Yes, once this is merged you will be able to download compilation outputs from the remote server without any further configuration.

@adpi2
Copy link
Member Author

adpi2 commented Feb 21, 2025

Superseded by #22578

@adpi2 adpi2 reopened this Feb 26, 2025
@adpi2 adpi2 marked this pull request as ready for review February 26, 2025 10:17
@hamzaremmal hamzaremmal enabled auto-merge (squash) February 26, 2025 10:17
@adpi2
Copy link
Member Author

adpi2 commented Feb 26, 2025

I rebased the branch because Mima job failed: https://github.com/scala/scala3/actions/runs/13541557343

@adpi2
Copy link
Member Author

adpi2 commented Feb 26, 2025

Mima failed again: https://github.com/scala/scala3/actions/runs/13544643262/job/37853234696

I suspect there is an incorrect cache hit. We read something from the cache instead of recompiling it and then we get a TASTy version mismatch.

Each time the compiler changes it should invalidate all bootstrapped modules
@adpi2 adpi2 force-pushed the develocity-cache branch from 123f6b4 to 3c18411 Compare March 4, 2025 15:27
@hamzaremmal hamzaremmal merged commit bef520f into scala:main Mar 4, 2025
29 checks passed
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.

3 participants