Skip to content

Conversation

@yrodiere
Copy link
Member

Similar to #9176 , but more conservative as we didn't publish build scan from GitHub Actions in this branch. Almost identical to #9177.

This is a backport of #9163 (limited to two commits) #9165 #9166.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


beikov and others added 4 commits October 28, 2024 17:31
So that we're extra sure we're not relying on cache left by a previous
Gradle execution.

(cherry picked from commit 543e095)
Ripped off from Quarkus.

Here's how it will work:

1. We create a cache entry on push only. Pull requests only restore it.
2. We create a new cache entry every day, prefixed with something like
   2024-10-25.
3. When restoring the cache, we try the entry for the day first
   (2024-10-25) and default to the one for the month (2024-10-*).
   Critically, this means we will build each day's cache based on the
   previous day's cache.
4. Atlas infra uses its own, separate cache entries.

Co-Authored-By: Guillaume Smet <[email protected]>
(cherry picked from commit 5eaaff2)
We don't need the whole cache, since most (all?) of it is stored on
Develocity already.

Including ./gradle/caches/modules-2 is enough, because that's where
Maven dependencies are stored, and *those* are not stored on Develocity.

(cherry picked from commit 013bb46)
@hibernate-github-bot
Copy link

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [ce59fe8]

› This message was automatically generated.

@beikov beikov merged commit c1e6627 into hibernate:6.2 Oct 29, 2024
18 of 24 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.

2 participants