Skip to content

ci: shorten the PR critical path - #786

Merged
Karib0u merged 2 commits into
mainfrom
ci/pr-critical-path
Oct 15, 2026
Merged

Karib0u merged 2 commits into
mainfrom
ci/pr-critical-path

Conversation

@Karib0u

@Karib0u Karib0u commented Oct 15, 2026

Copy link
Copy Markdown
Owner

Stacked on #785 (same file). Merge #785 first; this PR then retargets to main.

Changes

  • Linux atomics get their own job (Atomic (linux), same required-check name). The Windows shards need no eBPF object, so they stop waiting about 37s on Linux eBPF Build. The Atomic (windows) roll-up now covers only the Windows shards, so a Linux failure no longer turns it red too.
  • Three Windows shards instead of two. Each shard ran 150-163s of tests on run 38743554001; three should cut that to about 105s. The repo is public, so the extra runner costs no minutes. The native capture contract stays on shard 1.
  • Per-kernel cache for ebpf-load. Both legs shared one key. The 5.10 leg finishes first and saves a cache without the debug build, so the runner leg's save is skipped and Run live eBPF tests on runner kernel recompiles every dependency: 83s of compile for 49s of tests on run 38743554001.

Expected

On top of #785: the Windows atomic legs start right away and run about 55s shorter, and Linux eBPF Load (runner) drops by about 80s once main has saved its per-kernel cache. These are estimates from one run's logs and need a warm main run to confirm.

zizmor --offline is clean.

Base automatically changed from ci/rust-cache-key-fix to main October 15, 2026 14:51
- Move the Linux atomics into their own job. The Windows shards need no
  eBPF object, so they no longer wait ~37s on ebpf-build, and the
  Atomic (windows) roll-up now covers only the Windows shards.
- Split the Windows atomics across three runners instead of two.
- Give each ebpf-load leg its own cache. With one shared key the quicker
  5.10 leg saved first, so the runner leg's debug build for the live eBPF
  tests was never cached and spent ~83s compiling before 49s of tests.
@Karib0u
Karib0u force-pushed the ci/pr-critical-path branch from 2b86e7c to 2e2a4b7 Compare October 15, 2026 17:27
@Karib0u
Karib0u enabled auto-merge (squash) October 15, 2026 17:28
@Karib0u
Karib0u merged commit 57602d3 into main Oct 15, 2026
20 checks passed
@Karib0u
Karib0u deleted the ci/pr-critical-path branch October 15, 2026 17:38
@Karib0u Karib0u added the ci CI/CD changes label Oct 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant