Skip to content

Let the route job run off GitHub-hosted minutes - #13

Merged
jonbeckman merged 1 commit into
masterfrom
claude/project-thread-4bs466-router
Oct 30, 2026
Merged

jonbeckman merged 1 commit into
masterfrom
claude/project-thread-4bs466-router

Conversation

@jonbeckman

Copy link
Copy Markdown
Contributor

Requested by Jonathan · project thread

Summary

The first Routact pilot (grokcast #24) failed before routing: GitHub did not start the route job on ubuntu-slim because the org's GitHub-hosted minutes for private repos are used up ("recent account payments have failed or your spending limit needs to be increased"). The router itself needs GitHub minutes, so the "wait on the primary" fallback can never happen in that state.

  • route.yml takes a new runs-on input (JSON: a label or a label array) for the route job itself.
  • Without the input, it reads the org or repo variable ROUTACT_ROUTER_RUNS_ON, and then falls back to ubuntu-slim, the current default. Callers do not change.
  • The README has a new section, "Where the route job runs", with the gh variable set command and a warning: do not point the router at a pool it checks.

For SolZeroAI: gh variable set ROUTACT_ROUTER_RUNS_ON --org SolZeroAI --visibility private --body '"blacksmith-2vcpu-ubuntu-2404"'.

Testing

  • The workflow YAML parses. The change is in the reusable workflow only, so npm run all is unaffected.
  • The units-test workflow does not call route.yml. After merge, grokcast #24 exercises the new runner.

Generated by Claude Code

When a private repository's GitHub-hosted minutes run out, GitHub does
not start the route job on ubuntu-slim, so no job can be routed. The
reusable workflow now takes a runs-on input, or the
ROUTACT_ROUTER_RUNS_ON variable, so an organization can route from a
runner such as Blacksmith. ubuntu-slim stays the default.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vdh4Z431pzGdWVWoAFNQqc
@jonbeckman jonbeckman self-assigned this Oct 30, 2026
@jonbeckman
jonbeckman merged commit 04d66ff into master Oct 30, 2026
7 checks passed
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.

2 participants