Skip to content

runners/arc: bake CI-capability into register-repo.sh provisioning - #253

Draft
izzywdev wants to merge 1 commit into
mainfrom
claude/issue-251-20260713-1918
Draft

runners/arc: bake CI-capability into register-repo.sh provisioning#253
izzywdev wants to merge 1 commit into
mainfrom
claude/issue-251-20260713-1918

Conversation

@izzywdev

Copy link
Copy Markdown
Owner

Auto-opened from branch claude/issue-251-20260713-1918.

Closes #251

Make every scale set registered via register-repo.sh come up CI-capable
from one provisioning change (issue #251):

- Default RUNNER_IMAGE to the FuzeInfra CI-capable image
  (ghcr.io/izzywdev/fuzeinfra-arc-runner) which ships docker compose-v2
  + buildx plugins, jq/curl, warm Python/Node toolcache, and now
  Playwright browser OS deps — instead of the stock actions-runner
  image that lacks compose (the "unknown shorthand flag: 'f'" failure).
- Add --runner-image and --container-mode flags so onboarding stays a
  single invocation while remaining overridable (dind stays the default).
- Dockerfile: bake Playwright browser system deps via `playwright
  install-deps` so e2e/visual jobs only run `npx playwright install
  <browser>` (no apt/root at job time).
- ONBOARD-REPO.md: docker compose now works by default; document the
  publish-PUBLIC prerequisite for the runner image.

Prerequisite (human/PAT — Claude App cannot touch .github/workflows):
publish the image PUBLIC via build-runner-image.yml before onboarding.

Co-authored-by: Israel Weinberg <99821070+izzywdev@users.noreply.github.com>
@izzywdev

Copy link
Copy Markdown
Owner Author

@claude can this be progressed to merge? or is it obsolete compared to main?

@claude

claude Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Claude finished @izzywdev's task in 1m 36s —— View job


Verdict: obsolete — recommend closing (do not merge)

  • Compare PR changes against current main
  • Check whether register-repo.sh CI-capability changes already landed
  • Determine: mergeable, needs rebase, or obsolete
  • Report recommendation

This PR cannot be cleanly merged and its substance is already in main.

Why it's obsolete

main's history was rewritten/squashed since this branch was cut (2026-07-13) — the two share no merge base (git merge-base origin/main HEAD → none), so GitHub can't do a normal merge anyway. More importantly, the actual content already landed:

PR file branch vs origin/main
runners/arc/register-repo.sh identical — CI-capable provisioning already in main (see lines 60–63, 76, 189–191: fuzeinfra.io/pool: ci, CI-capable default image, etc.)
runners/arc/ONBOARD-REPO.md identical
runners/arc/Dockerfile branch is behind main

The one remaining diff is a regression risk: main's Dockerfile already installs the GitHub CLI (gh) from the official apt repo. This branch predates that, so merging it would delete the gh install block:

-# --- GitHub CLI (gh) ---------------------------------------------------------
-RUN set -eux; \
-    curl -fsSL https://cli.github.com/packages/githubcli-archive-keyring.gpg ...
-    apt-get install -y --no-install-recommends gh; \
-    gh --version

Recommendation

I can't merge or close PRs myself (Claude can't perform those git/GitHub operations), so this needs a human to close #253 and confirm #251.
· branch claude/issue-251-20260713-1918

@izzywdev

Copy link
Copy Markdown
Owner Author

⚠️ Claude responder did not run to completion

The responder errored before finishing your request, so no changes were made and no branch was created.

This notice is automated and will not retry on its own — the responder stops after one failure and waits. If this looks like an API-side problem (key revoked / out of credits / model or 1M-context beta not entitled / Anthropic outage), an operator should check the ANTHROPIC_API_KEY secret and fund/repair the Anthropic account first, then re-trigger by posting a new comment mentioning the responder.

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.

Bake CI-capability into runners/arc/register-repo.sh (containerMode: dind + CI runner image) — provisioning-level fix

1 participant