Skip to content

UNOMI-966: Upgrade remaining setup-java actions to v5 and speed up CI workflows - #829

Merged
sergehuber merged 2 commits into
masterfrom
UNOMI-966-setup-java-v5
Jul 18, 2026
Merged

UNOMI-966: Upgrade remaining setup-java actions to v5 and speed up CI workflows#829
sergehuber merged 2 commits into
masterfrom
UNOMI-966-setup-java-v5

Conversation

@sergehuber

@sergehuber sergehuber commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bump actions/setup-java from @v1 to @v5 in unomi-ci-docs-deploy.yml (both jobs) and codeql-analysis-java.yml
  • Add explicit distribution: temurin (required by modern setup-java) while keeping JDK 17 and existing Nexus credential wiring
  • Speed up and harden all five CI workflows:
    • Add cache: maven to the CodeQL Java build and both docs-deploy jobs (the other workflows already had it)
    • Add a final "Clean Unomi artifacts from Maven cache" step to every cached job, so the post-job cache save keeps only third-party dependencies — Unomi's own snapshots (including the large kar/Karaf distribution) are rebuilt each run and would only bloat the cache and risk staleness
    • Add concurrency groups everywhere: superseded PR runs are cancelled (cancel-in-progress only when github.event_name == 'pull_request'), master pushes and scheduled CodeQL runs are never cancelled, and the docs-deploy workflow serializes runs without cancelling so a snapshot publish is never killed mid-upload
  • Remove dead Nexus server-id/server-username/server-password config from the publish-docker-image job (it only pushes to Docker Hub and never had the NEXUS env vars wired)
  • Quote distribution/java-version values consistently with the workflows upgraded in #795

Closes the remaining Node 24–compatible Actions gap after #795.

JIRA: UNOMI-966

Test plan

  • Confirm no remaining actions/setup-java@v1 under .github/workflows/
  • CodeQL Java workflow runs on the PR (or a follow-up push to master after merge)
  • PR runs show superseded runs being cancelled when a new commit is pushed
  • After merge to master, docs-deploy job still publishes with JDK 17 / Nexus credentials unchanged aside from the action major, and the saved Maven cache contains only third-party dependencies

🤖 Generated with Claude Code

sergehuber and others added 2 commits July 17, 2026 17:45
Bump docs-deploy and CodeQL Java workflows from setup-java@v1 to @v5
with Temurin so all CI uses Node 24–compatible action majors.
- Add cache: maven to CodeQL Java build and both docs-deploy jobs
- Purge Unomi's own snapshots from ~/.m2 before every post-job cache
  save (all cached jobs) so caches hold only third-party dependencies
- Add concurrency groups: cancel superseded PR runs, never cancel
  master/scheduled runs; serialize (not cancel) snapshot deploys
- Remove dead Nexus server config from the Docker image job
- Quote distribution/java-version consistently with other workflows

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@sergehuber sergehuber changed the title UNOMI-966: Upgrade remaining setup-java actions to v5 UNOMI-966: Upgrade remaining setup-java actions to v5 and speed up CI workflows Jul 17, 2026
@sergehuber
sergehuber merged commit 24b3641 into master Jul 18, 2026
9 of 10 checks passed
@sergehuber
sergehuber deleted the UNOMI-966-setup-java-v5 branch July 18, 2026 06:13
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