Skip to content

Use upload/download-artifact for CI inter-job repo sharing#2498

Merged
jbonofre merged 1 commit intoapache:mainfrom
jbonofre:fix/ci-cache-artifact
Apr 2, 2026
Merged

Use upload/download-artifact for CI inter-job repo sharing#2498
jbonofre merged 1 commit intoapache:mainfrom
jbonofre:fix/ci-cache-artifact

Conversation

@jbonofre
Copy link
Copy Markdown
Member

@jbonofre jbonofre commented Apr 1, 2026

Summary

  • Replace actions/cache/save and actions/cache/restore with actions/upload-artifact and actions/download-artifact for passing the Maven local repository between the build and test CI jobs
  • This fixes test job failures when re-running a workflow after GitHub's 7-day cache eviction (e.g. PR build(deps): bump biz.aQute.bnd:biz.aQute.bndlib from 7.2.1 to 7.2.3 #2482 where the apache-karaf:tar.gz SNAPSHOT artifact couldn't be resolved)
  • Set retention-days: 1 to keep storage usage minimal

Test plan

  • Verify CI build + test jobs pass on this PR
  • Verify re-running the test job after some time still works

…haring

The cache/save and cache/restore actions are subject to GitHub's 7-day
cache eviction policy. When a test job is re-run after the cache expires,
the Maven local repository is empty and SNAPSHOT artifacts like
apache-karaf:tar.gz cannot be resolved, causing build failures.

Switching to upload-artifact/download-artifact ties the Maven local repo
to the workflow run itself, making test job re-runs reliable regardless
of cache expiration.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Test Results

237 files  ±0  237 suites  ±0   23m 16s ⏱️ -9s
923 tests ±0  875 ✅ ±0  48 💤 ±0  0 ❌ ±0 
927 runs  ±0  879 ✅ ±0  48 💤 ±0  0 ❌ ±0 

Results for commit b228f76. ± Comparison against base commit 2da19de.

@jbonofre jbonofre merged commit a6a8258 into apache:main Apr 2, 2026
3 checks passed
@jbonofre jbonofre deleted the fix/ci-cache-artifact branch April 2, 2026 08:20
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