Skip to content

ci: unify arm64 cache keys to reduce repeated compilation#103

Merged
ferhimedamine merged 1 commit into
mainfrom
fix/ci-cache-optimization
Jun 18, 2026
Merged

ci: unify arm64 cache keys to reduce repeated compilation#103
ferhimedamine merged 1 commit into
mainfrom
fix/ci-cache-optimization

Conversation

@ferhimedamine

Copy link
Copy Markdown
Contributor

Summary

  • Unify shared-key for check, clippy, and test jobs from separate keys (check, clippy, test) to a single arm64-ci key
  • Each arm64 job now reads from the others' cached target/ artifacts on subsequent runs, eliminating duplicate compilation of shared deps
  • Add cache-all-crates: "true" to integration-test to cache crate source extraction for the release build

Baseline vs Expected

Run type Before Expected after
Warm cache (PR CI) 143s ~90s
Cold cache (new dep bump) 247s ~140s

The warm-cache improvement comes from clippy and test restoring check's compiled artifacts instead of recompiling from scratch.

Test plan

  • CI passes on this PR
  • Observe check, clippy, test all restore from the same arm64-ci cache key on the next PR run

🤖 Generated with Claude Code

check/clippy/test all use shared-key: arm64-ci so each job reads
from the others' cached target/ artifacts instead of compiling
from scratch. Clippy and test can reuse check's compiled deps,
cutting repeat compilation on warm cache runs by ~40-60%.

Also adds cache-all-crates: true to integration-test to cache crate
source extraction for the release binary build.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ferhimedamine ferhimedamine added the auto-merge Auto-merge when CI passes label Jun 18, 2026
@ferhimedamine

Copy link
Copy Markdown
Contributor Author

🤖 [Agent: CTO] Code Review — APPROVED

Type: type:ci-infra — CI cache optimization, no bench needed.

Review:

  • Unifies check/clippy/test cache keys → single arm64-ci shared key. First warm job caches all dependencies; subsequent jobs get instant cache hits instead of re-compiling from scratch.
  • Integration test adds cache-all-crates: "true" to include all dependencies in its cache, not just workspace crates.
  • Expected 40-60% CI time reduction on warm runs.
  • No code changes. No behavioral impact. No risk.

Merge Risk Assessment: Zero blast radius. CI-only change. All 6 checks green.

Merging.

@ferhimedamine ferhimedamine merged commit 58183bc into main Jun 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Auto-merge when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant