ci: migrate 3 of 7 jobs to smithy self-hosted runners#29
Merged
Conversation
rust-cpu clippy
light format, validation
Stays on ubuntu-latest:
- pre-commit setup-python + pip install pre-commit not yet validated
against smithy's preinstalled Python 3.12
- test cross-OS matrix (ubuntu/macos/windows)
- build cross-OS matrix (ubuntu/macos-13/macos-14/windows)
- security cargo-audit < 0.22.1 rejects CVSS 4.0 advisories
The npm-publish.yml release workflow is left untouched: every job is
either a cross-OS build matrix or a Node-only publish step that runs
once per tag, so the smithy compute win is negligible.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Moves the three Linux-only single-host jobs of
ci.yml(clippy,format,validation) to the pulseengine self-hosted fleet(smithy). The four jobs that stay on
ubuntu-latesteither span across-OS matrix (
test,build), depend on tooling that is not yetclean on smithy (
pre-commit's pip-installed Python tooling), or hita known smithy-side issue (
security's cargo-audit advisory parser).The release pipeline (
npm-publish.yml) is left untouched: every jobis either a cross-OS build matrix or a Node-only publish step that
runs once per tag, so the smithy compute win is negligible.
Coverage
clippyrust-cpuformatlightvalidationlightStays on ubuntu-latest
pre-commitactions/setup-python+ systempip install pre-commit detect-secretsinteracts with smithy's preinstalled Python 3.12 in ways we haven't validated yet (no--user, no venv). Keep here until the recipe is clean.testubuntu-latest,macos-latest,windows-latest); the matrix exists to validate per-PR coverage on all three.buildubuntu-latest,macos-13,macos-14,windows-latest).securitycargo-auditadvisory parser shipped on smithy (v0.21.2) rejects CVSS 4.0 advisories (RUSTSEC-2026-0037). Move back once smithy bumps cargo-audit to 0.22.1+.Workarounds applied
None — the three migrated jobs are clean
runs-on:swaps. Noinstaller scripts, no sudo, no
cargo-deny-action/cargo-semver-checks-action/free-disk-spacepatterns.Test plan
smithy runner class within seconds (no GitHub queue).
clippysucceeds end-to-end onrust-cpu.format,validationsucceed onlight.pre-commit,test,build,securitystill run onubuntu-latestexactly as before.Rollback
Revert this commit. Every migrated job returns to
ubuntu-latestandthe next run uses GitHub-hosted compute.