Skip to content

CI - Runner caching with sccache - #5848

Open
bfops wants to merge 108 commits into
masterfrom
bfops/sccache
Open

CI - Runner caching with sccache#5848
bfops wants to merge 108 commits into
masterfrom
bfops/sccache

Conversation

@bfops

@bfops bfops commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

Description of Changes

This only changes Linux runners.

  • use runner-cached sccache dir for most things
  • use a runner-cached target directory for native/build-script artifacts
    • static CARGO_TARGET_DIR has been replaced with a step that sets it (because it is now based on HOME)

Timing results

These are sums of Cargo's reported Finished ... in ... times, matching the table in #5798. The test run was CI run 33413710732 on spacetimedb-new-runner-2-staging.

Job Cargo time in master this PR Rust sccache hit rate
Test Suite 9m30s - 10m02s 6m44.9s 12.96%
Upload build artifacts (Linux) 2m27s - 2m46s 1m07s 93.33%
Lints 1m46s - 2m09s 55.6s 6.52%
Build smoketests (Linux) 48.9s - 57.1s 10.4s 83.33%
Check CLI docs 56s - 1m08s 13.0s 98.28%
Test spacetimedb-update flow (Linux) 29.6s - 30.7s 2.9s 100%
csharp-testsuite 28.2s - 46.6s 14.2s 0%
TypeScript - Tests 22.8s - 28.4s 2.3s 100%
Build and test wasm bindings 25.0s - 33.2s 3.3s 100%
Docs build 20s - 21s 1.2s 100%
Check that packages are publishable 19s - 21s 1.1s not reported
godot-testsuite 7.6s - 11.2s 3.1s 100%

The next high-value change is to remove commit-specific metadata from shared Rust library artifacts (or otherwise confine it to a leaf artifact that does not participate in the test/lint dependency graph), then add the smoketest archive to the empty-target population phase.

API and ABI breaking changes

None.

Expected complexity level and risk

2

Testing

  • the table is the testing

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
bfops added 2 commits August 31, 2026 12:10
Signed-off-by: Zeke Foppa <196249+bfops@users.noreply.github.com>
@bfops
bfops marked this pull request as ready for review August 31, 2026 19:11
@joshua-spacetime
joshua-spacetime self-requested a review August 31, 2026 19:25

@jdetter jdetter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me, I didn't test anything manually here but the numbers in the table make sense and look fine to me. Thanks @bfops 👍

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.

4 participants