Skip to content

Split up the monster ci.yml workflow#7472

Merged
AdamGS merged 2 commits intodevelopfrom
adamg/split-up-ci
Apr 16, 2026
Merged

Split up the monster ci.yml workflow#7472
AdamGS merged 2 commits intodevelopfrom
adamg/split-up-ci

Conversation

@AdamGS
Copy link
Copy Markdown
Contributor

@AdamGS AdamGS commented Apr 16, 2026

Summary

ci.yml is reaching 1000 lines of YAML, which no one wants. This PR splits into 5 files:

  1. ci.yml - now only contains linters and test runs
  2. codspeed.yml - just codspeed benchmarks
  3. cuda.yaml - all the cuda incantations
  4. publish-dry-runs.yaml - all the dark magic that we have to try and make sure we can actually publish
  5. rust-instrumented.yaml - rust-specific jobs that require nightly and some extra instrumentations (coverage, miri, etc)

The real magic here is that there are no changes to the job names, so all rules still apply!

This also makes github retries much easier, as github only allows retries after the whole workflow finishes.

AdamGS added 2 commits April 16, 2026 15:05
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 16, 2026

Merging this PR will improve performance by 20.22%

⚡ 3 improved benchmarks
✅ 1160 untouched benchmarks
⏩ 1457 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation take_map[(0.1, 0.5)] 1,154.5 µs 985.6 µs +17.14%
Simulation take_map[(0.1, 1.0)] 2 ms 1.6 ms +20.22%
Simulation new_alp_prim_test_between[f32, 16384] 121.1 µs 106.9 µs +13.29%

Comparing adamg/split-up-ci (755dcd6) with develop (3cd74b9)

Open in CodSpeed

Footnotes

  1. 1457 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@AdamGS AdamGS requested review from 0ax1 and robert3005 April 16, 2026 14:21
@AdamGS AdamGS marked this pull request as ready for review April 16, 2026 14:21
@AdamGS AdamGS added the changelog/chore A trivial change label Apr 16, 2026
@AdamGS AdamGS merged commit 8b09d0c into develop Apr 16, 2026
65 of 67 checks passed
@AdamGS AdamGS deleted the adamg/split-up-ci branch April 16, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants