Skip to content

fix(ci): build the ARM64 app image on a 16 vCPU runner (#8292) - #8293

Merged
waleedlatif1 merged 1 commit into
mainfrom
staging
Sep 25, 2026
Merged

waleedlatif1 merged 1 commit into
mainfrom
staging

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

@vercel

vercel Bot commented Sep 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 25, 2026 8:32am UTC

Request Review

@waleedlatif1
waleedlatif1 merged commit 349763c into main Sep 25, 2026
44 checks passed
@greptile-apps

greptile-apps Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[High risk] Changes the build runner for ARM64 container images.

The PR appears safe to merge, though validating the new ARM runner label would reduce the risk of a queued GHCR release.

Findings

  1. P2 Unverified ARM runner label ▶

Summary

The PR moves the ARM64 app image build from an 8-vCPU to a 16-vCPU Blacksmith runner.

  • The larger runner matches the app build’s documented need for more memory.
  • The new main-only runner label should be validated before it becomes part of the release path.

Reviews (1) · Last reviewed commit: "fix(ci): build the ARM64 app image on a ..."

Comment thread .github/workflows/ci.yml
image: ghcr.io/simstudioai/simstudio
gh_runner: linux-arm64-8-core
bs_runner: blacksmith-8vcpu-ubuntu-2404-arm
bs_runner: blacksmith-16vcpu-ubuntu-2404-arm

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 Unverified ARM runner label

This 16-vCPU ARM label is new to the repository, and the build runs only on pushes to main, so PR checks cannot confirm that the runner can take the job. If the label is not provisioned, the app image build will remain queued and block GHCR manifest publication. Please validate the label with a non-release build before relying on it.

Knowledge Base Used: Operations, deployment, and privacy services

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

This branch was successfully deployed

1 active deployment
Preview — a4722872 Deployed Sep 25, 2026 by vercel[bot]
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