You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Blacksmith is the fastest way to run your GitHub Actions.
What does this PR change?
This PR has been automatically generated by a team member in your GitHub organization using Blacksmith's Migration Wizard, or MigWiz for short. This PR changes the following:
Your selected workflows will now run on Blacksmith's 2x faster hardware (e.g., runs-on: blacksmith-4vcpu-ubuntu-2204). Learn more about the different instances available to choose from.
Your jobs running on Blacksmith will now have all official GitHub and popular third-party cache actions automatically interact with our 4x faster, colocated cache. Learn more about Blacksmith's actions cache.
Your GitHub Actions will now actually be observable. Learn more about Blacksmith's logging and other observability features.
Your Docker builds will now automatically share their Docker layer cache, resulting in up to 40x faster builds. Learn more about Blacksmith's Docker layer caching.
FAQ
Is this free? The first 3,000 minutes per month are free.
Who uses Blacksmith? Clerk, Ashby, VEED, and 600+ others.
What's the catch? There is none. Merge this thing already.
This PR migrates the benchmark-codspeed job in .github/workflows/pr-test-suite.yml from the Buildjet runner (buildjet-8vcpu-ubuntu-2204) to the Blacksmith runner (blacksmith-8vcpu-ubuntu-2204) for faster CI execution.
The change is a single-line runs-on substitution in the benchmark-codspeed job.
Issue: The Swatinem/rust-cache@v2 step's cache-provider: buildjet (line 1240) was not updated to match the new runner. This creates a provider mismatch — the cache action will attempt to communicate with Buildjet's cache backend from a Blacksmith-hosted runner, which will result in cache misses at best and job failures at worst.
The rest of the workflow's jobs (which use standard GitHub-hosted runners) are unaffected by this change.
Confidence Score: 3/5
The PR is mostly safe but has a cache provider mismatch that will break Rust caching for the benchmark job.
The runner substitution itself is straightforward, but the incomplete migration leaves cache-provider: buildjet pointing at Buildjet's backend from a Blacksmith runner, which will cause the Rust cache to be non-functional for this job. This is a minor correctness issue that degrades CI performance rather than breaking correctness of the benchmark results themselves.
pr-test-suite.yml — specifically the Swatinem/rust-cache step in the benchmark-codspeed job needs its cache-provider updated.
Important Files Changed
Filename
Overview
.github/workflows/pr-test-suite.yml
Migrates the benchmark-codspeed job runner from buildjet-8vcpu-ubuntu-2204 to blacksmith-8vcpu-ubuntu-2204, but leaves cache-provider: buildjet in the Swatinem/rust-cache step unchanged, creating a mismatch between the runner and its cache backend.
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[PR triggered] --> B[skipcheck job]
B --> C{skip == 'false'?}
C -- No --> D[Skip benchmark]
C -- Yes --> E[benchmark-codspeed job\nruns-on: blacksmith-8vcpu-ubuntu-2204]
E --> F[Checkout + restore-mtime]
F --> G[moonrepo/setup-rust]
G --> H[Swatinem/rust-cache\ncache-provider: buildjet ⚠️\nMismatch with Blacksmith runner]
H --> I[Setup Python / uv]
I --> J[Install dependencies]
J --> K[Build Rust Library via maturin]
K --> L[Restore TPC-H data cache\nactions/cache v5]
L --> M[CodSpeedHQ/action\nRun benchmarks in simulation mode]
M --> N{Success?}
N -- Failure + main branch --> O[Send Slack notification]
N -- Success --> P[Done]
style H fill:#ff9999,stroke:#cc0000
Loading
Comments Outside Diff (1)
.github/workflows/pr-test-suite.yml, line 1240 (link)
Stale cache-provider after runner migration
The runs-on was changed to blacksmith-8vcpu-ubuntu-2204, but the cache-provider: buildjet on this line was not updated. Swatinem/rust-cache@v2 will attempt to use Buildjet's cache backend from a Blacksmith runner, which will result in cache misses at best or outright failures at worst — completely negating the caching benefit.
This should either be removed (to fall back to the standard GitHub Actions cache) or updated to the appropriate Blacksmith-compatible cache provider.
✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.83%. Comparing base (6448008) to head (db12c5e). ⚠️ Report is 41 commits behind head on main.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To whomever may be reviewing this PR,
Blacksmith is the fastest way to run your GitHub Actions.
What does this PR change?
This PR has been automatically generated by a team member in your GitHub organization using Blacksmith's Migration Wizard, or MigWiz for short. This PR changes the following:
runs-on: blacksmith-4vcpu-ubuntu-2204). Learn more about the different instances available to choose from.FAQ