Skip to content

accelerate CI via cargo binstall#578

Open
plebhash wants to merge 2 commits into
stratum-mining:mainfrom
plebhash:2026-06-16-cargo-bininstall
Open

accelerate CI via cargo binstall#578
plebhash wants to merge 2 commits into
stratum-mining:mainfrom
plebhash:2026-06-16-cargo-bininstall

Conversation

@plebhash

@plebhash plebhash commented Jun 19, 2026

Copy link
Copy Markdown
Member

accelerate CI via cargo-binstall

instead of compiling cargo-nextest and cargo-semver-checks on every run, we install pre-built binaries via cargo-binstall.

additional hardening:

  • disable quick-install to only use upstream release artifacts (instead of third-party quick-install artifacts)
  • pass GITHUB_TOKEN to cargo-binstall to avoid unauthenticated GitHub API rate-limit fallback
  • use --locked for deterministic fallback behavior if source install is ever needed

estimated impact (latest successful PR run vs median of 5 recent comparable successful PR runs on the same workflows/jobs):

  • ci (ubuntu-latest):
    • tool install: ~2m34s -> ~3s (~98% reduction)
    • total job: ~32m18s -> ~28m41s (~3m37s faster, ~11%)
  • ci (macos-latest):
    • tool install: ~2m38s -> ~5s (~97% reduction)
    • total job: ~31m57s -> ~30m04s (~1m53s faster, ~6%)
  • semver-check:
    • tool install: ~5m09s -> ~4s (~99% reduction)
    • total job: ~15m04s -> ~9m44s (~5m20s faster, ~35%)

critical-path wall clock also improves (~32m18s -> ~30m04s, ~2m14s), while matrix jobs still run in parallel.

@plebhash plebhash force-pushed the 2026-06-16-cargo-bininstall branch from 4da8290 to 094c977 Compare June 19, 2026 15:39
@plebhash plebhash marked this pull request as draft June 19, 2026 15:39
@plebhash plebhash force-pushed the 2026-06-16-cargo-bininstall branch from 094c977 to ecd565f Compare June 19, 2026 20:33
@plebhash plebhash marked this pull request as ready for review June 19, 2026 20:34

@Shourya742 Shourya742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK.

@Shourya742

Shourya742 commented Jun 20, 2026

Copy link
Copy Markdown
Member

@plebhash can you rebase this? And can you also squash the commits (kinda similar changes)

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.

2 participants