Skip to content

IGNITE-29134 Unify the Maven repository cache of GitHub Actions workflows - #13682

Open
NSAmelchev wants to merge 1 commit into
apache:masterfrom
NSAmelchev:ignite-29134
Open

NSAmelchev wants to merge 1 commit into
apache:masterfrom
NSAmelchev:ignite-29134

Conversation

@NSAmelchev

@NSAmelchev NSAmelchev commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

All workflows cache ~/.m2/repository under the same -m2- key, with Ignite *-SNAPSHOT artifacts excluded: every job installs what it needs itself, and the Sonar scan takes classes from the build artifact. Each pom state is now stored once as a 497 MB entry instead of 500 MB + 792 MB.

Behavior change: actions/cache does not save the entry when the job fails, unlike setup-java; master builds are unaffected, PR runs fall back to the master entry via restore-keys.

Also removed a duplicated Setup JDK17 step in sonar-branch.yml and duplicated modules/core entries in the javadoc check command.

Checked in the fork:

NSAmelchev/ignite#50 (checkstyle failed, cache not saved)
NSAmelchev/ignite#51 (build passed, 497 MB cache entry saved)

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.

1 participant