Skip to content

.github/workflows: Migrate workflows to Blacksmith runners#166

Open
blacksmith-sh[bot] wants to merge 5 commits intodevfrom
blacksmith-migration-8445043
Open

.github/workflows: Migrate workflows to Blacksmith runners#166
blacksmith-sh[bot] wants to merge 5 commits intodevfrom
blacksmith-migration-8445043

Conversation

@blacksmith-sh
Copy link
Copy Markdown

@blacksmith-sh blacksmith-sh Bot commented May 3, 2026

This PR has been automatically generated by a team member in your GitHub organization using Blacksmith's Migration Wizard. This PR changes the following:

  1. 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.
  2. 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.
  3. Your GitHub Actions will now actually be observable. Learn more about Blacksmith's logging and other observability features.
  4. 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.

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 3, 2026

Greptile Summary

This PR migrates all GitHub Actions workflows from GitHub-hosted runners (ubuntu-latest, ubuntu-22.04, macos-latest, windows-latest) to Blacksmith-hosted equivalents, and swaps docker/setup-buildx-action@v3 for useblacksmith/setup-docker-builder@v1 in the Docker publish workflow. The migration is consistent across all 28 files with one exception: the self-host-daemon-e2e matrix in tests.yml updated its Ubuntu entry but left macos-latest unchanged while every other macOS runner in the PR was migrated to blacksmith-6vcpu-macos-15.

Confidence Score: 4/5

Safe to merge; one macOS runner in a matrix was missed but still runs correctly on GitHub-hosted hardware.

All changes are runner label swaps with no logic modifications. The only finding is a P2 inconsistency where macos-latest was not migrated in one matrix entry, which doesn't block functionality.

tests.ymlself-host-daemon-e2e matrix still references macos-latest.

Important Files Changed

Filename Overview
.github/workflows/tests.yml Migrated all runner labels to Blacksmith equivalents, except macos-latest in the self-host-daemon-e2e matrix which was missed.
.github/workflows/publish-docker.yml Replaced docker/setup-buildx-action@v3 with useblacksmith/setup-docker-builder@v1 and migrated runner label to Blacksmith; QEMU step retained correctly for multi-platform builds.
.github/workflows/build-tauri.yml Migrated linux runner labels to Blacksmith; Windows and macOS Tauri build targets left untouched (correct).
.github/workflows/release.yml All ubuntu-latest runners migrated to blacksmith-4vcpu-ubuntu-2404; no logic changes.
.github/workflows/extended-db-tests.yml Both ubuntu-22.04 and ubuntu-latest jobs correctly migrated to matching Blacksmith equivalents.
.github/workflows/build-ui-mobile-local.yml All runner labels migrated; macos-26 correctly mapped to blacksmith-6vcpu-macos-26.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[GitHub Actions Trigger] --> B{Runner Type}
    B --> |ubuntu-latest| C[blacksmith-4vcpu-ubuntu-2404]
    B --> |ubuntu-22.04| D[blacksmith-4vcpu-ubuntu-2204]
    B --> |macos-latest| E[blacksmith-6vcpu-macos-15]
    B --> |macos-26| F[blacksmith-6vcpu-macos-26]
    B --> |windows-latest| G[blacksmith-4vcpu-windows-2025]
    B --> |macos-latest in matrix| H[⚠️ macos-latest — NOT migrated
self-host-daemon-e2e]
    C & D & E & F & G --> I[Blacksmith Runner
Faster cache + observability]
    H --> J[GitHub-hosted Runner]
Loading

Reviews (1): Last reviewed commit: "Migrate workflows to Blacksmith" | Re-trigger Greptile

Comment thread .github/workflows/tests.yml Outdated
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
os: [blacksmith-4vcpu-ubuntu-2404, macos-latest]
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.

P2 Incomplete macOS migration in matrix

The macos-latest entry in this matrix was not migrated to blacksmith-6vcpu-macos-15, unlike every other macos-latest occurrence in this PR (e.g. installers-smoke-macos, self-host-launchd-e2e, mobile-e2e-ios). This self-host-daemon-e2e job will still run on GitHub-hosted macOS runners rather than Blacksmith.

Suggested change
os: [blacksmith-4vcpu-ubuntu-2404, macos-latest]
os: [blacksmith-4vcpu-ubuntu-2404, blacksmith-6vcpu-macos-15]

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@blacksmith-sh

This comment has been minimized.

leeroybrun and others added 3 commits May 3, 2026 14:44
…sertion

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
…ght stubs

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
…e sources

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
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