Skip to content

Comments

Switch to Blacksmith runners for CI#967

Open
skyfallwastaken wants to merge 1 commit intomainfrom
blacksmith
Open

Switch to Blacksmith runners for CI#967
skyfallwastaken wants to merge 1 commit intomainfrom
blacksmith

Conversation

@skyfallwastaken
Copy link
Member

In theory at least, these should be way faster!

Copilot AI review requested due to automatic review settings February 17, 2026 11:33
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 17, 2026

Greptile Summary

This PR migrates all CI jobs from GitHub's standard ubuntu-latest runners to Blacksmith's blacksmith-4vcpu-ubuntu-2404 runners. The change affects 6 active jobs (scan_ruby, scan_js, lint, frontend, zeitwerk, test) plus one commented-out test job.

Key changes:

  • All runs-on: ubuntu-latest declarations replaced with runs-on: blacksmith-4vcpu-ubuntu-2404
  • No changes to job logic, dependencies, or workflow structure
  • Consistent application across all jobs in the file

Impact:

  • Expected performance improvement through 4-vCPU dedicated runners
  • Uses Ubuntu 24.04 as base image (vs Ubuntu 22.04 in ubuntu-latest)
  • No compatibility concerns - Blacksmith runners are GitHub Actions-compatible

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk - it's a straightforward infrastructure change
  • The change is purely infrastructure-related, switching runner types without modifying any job logic, dependencies, or workflow structure. Blacksmith runners are GitHub Actions-compatible and the change is applied consistently across all jobs. No code, tests, or functionality are affected.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/ci.yml All CI job runners switched from ubuntu-latest to blacksmith-4vcpu-ubuntu-2404 for faster execution

Last reviewed commit: 8482240

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CI workflow to use Blacksmith-provided Ubuntu 24.04 runners instead of ubuntu-latest, aiming to speed up job execution across the pipeline.

Changes:

  • Switch runs-on for all active CI jobs from ubuntu-latest to blacksmith-4vcpu-ubuntu-2404.
  • Update the commented-out test job runner label to match the new runner.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

jobs:
scan_ruby:
runs-on: ubuntu-latest
runs-on: blacksmith-4vcpu-ubuntu-2404
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

The runner label is repeated across multiple jobs. Consider defining it once (e.g., via a YAML anchor/alias or a top-level env + expression in runs-on) to avoid missed updates if the runner name changes again.

Copilot uses AI. Check for mistakes.
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