Skip to content

ci: migrate CI runners to Blacksmith#37

Merged
ben-miru merged 2 commits into
mainfrom
ci/migrate-to-blacksmith
Jun 26, 2026
Merged

ci: migrate CI runners to Blacksmith#37
ben-miru merged 2 commits into
mainfrom
ci/migrate-to-blacksmith

Conversation

@ben-miru

@ben-miru ben-miru commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Migrate GitHub-hosted runners to like-for-like Blacksmith sizes.

Workflow Job Before After
ci.yml lint ubuntu-latest-m blacksmith-4vcpu-ubuntu-2404
ci.yml test ubuntu-latest-m blacksmith-4vcpu-ubuntu-2404
ci.yml surface-lint (reusable workflow runs-on input) ubuntu-latest-m blacksmith-4vcpu-ubuntu-2404
codeql-analysis.yml codeql ubuntu-latest blacksmith-2vcpu-ubuntu-2404

Note: jobs will only run on Blacksmith once the Blacksmith GitHub App has been granted access to this repo (org-level setting).

🤖 Generated with Claude Code


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

ben-miru and others added 2 commits June 25, 2026 16:04
Move GitHub-hosted runners to like-for-like Blacksmith sizes
(2c->2vcpu, 4c->4vcpu, 8c->8vcpu). Jobs run on Blacksmith once the
Blacksmith GitHub App has access to this repo (org-level setting).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
surface-lint runs actionlint, which flags the new blacksmith-* runner
labels as unknown. Declare them under self-hosted-runner.labels (keeping
the existing custom labels).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ben-miru
ben-miru merged commit ab77a39 into main Jun 26, 2026
3 checks passed
@ben-miru
ben-miru deleted the ci/migrate-to-blacksmith branch June 26, 2026 01:35
ben-miru added a commit that referenced this pull request Jul 3, 2026
## What

Reverts the covgate **build-cache pre-warm** feature in its entirety.
covgate no longer pre-warms the Go build cache before its parallel
coverage runs.

Reverts, in order (newest first):
- #36 `perf(covgate): instrument the build-cache pre-warm` (`794e725`)
- #35 `feat(covgate): report build-cache pre-warm duration` (`6a0cc47`)
- #34 `perf(covgate): pre-warm build cache before parallel coverage
runs` (`0c3c0af`)

## Why

We no longer want covgate to pre-warm the build cache. The pre-warm
compiled an extra artifact set that inflated the CI `gobuild-test` cache
and pushed the backend repo back over the 10 GB Actions cache quota (the
very regression #36 tried to walk back). Rather than keep iterating on
the pre-warm, we're removing it.

## How

Three `git revert` commits, preserving history/attribution. The reverted
`internal/services/covgate/` and `internal/services/gocover/` files are
**byte-identical** to their pre-feature state, and the feature's design
doc (`plans/completed/20260624-covgate-prewarm-build-cache.md`) is
removed.

## Validation

- No lingering `prewarm` / `PrewarmBuild` references anywhere in
`internal/`, `cmd/`, or `scripts/`.
- Full `scripts/preflight.sh` passes clean: covgate coverage 94.3%
across all 30 packages, gocover 92.6% (≥ 92.1% required), custom lint +
gofumpt + golangci-lint + deadcode + surface-lint all green.
- Clean revert with no conflicts against current `main`; unrelated later
commits (#37 CI, #38 dep bumps, #39 max-param-count) required no
adjustment.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- codesmith:footer -->
---
<a
href="https://app.blacksmith.sh/mirurobotics/codesmith/gotools/pr/40"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v2.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-light-v2.svg"><img
alt="View with Codesmith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v2.svg"></picture></a>
<a
href="https://backend.blacksmith.sh/track/enable-autofix?expires=1785713638&installation_id=142588636&pr_number=40&repository=mirurobotics%2Fgotools&return_to=https%3A%2F%2Fgithub.com%2Fmirurobotics%2Fgotools%2Fpull%2F40&signature=00cc206a36c548207ac3606f293163ba579f08070387cbe5cc6ca77ed2f8cb48"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-light.svg"><img
alt="Autofix with Codesmith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"></picture></a>
<sup>Need help on this PR? Tag <code>/codesmith</code> with what you
need. Autofix is disabled.</sup>

<!-- codesmith:autofix:disabled -->
<!-- /codesmith:footer -->
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.

1 participant