Skip to content

refactor: rename GitHub runner ScaledJob - #20081

Open
martinothamar wants to merge 4 commits into
mainfrom
refactor/rename-github-runner-scaledjob
Open

refactor: rename GitHub runner ScaledJob#20081
martinothamar wants to merge 4 commits into
mainfrom
refactor/rename-github-runner-scaledjob

Conversation

@martinothamar

Copy link
Copy Markdown
Contributor

Summary

Rename the remaining live ScaledJob identity from github-runners-sandbox to github-runners, now that the sandbox-backed runner is the only GitHub runner variant. The self-hosted-ubuntu runner label and runtime configuration are unchanged.

This PR is stacked on #20079. Its diff against main will contain the parent cleanup until #20079 merges.

Merge conditions

Deleting the old ScaledJob identity while it owns active Jobs may cascade-delete those Jobs.

Verification

  • production Kustomize overlay renders exactly one ScaledJob named github-runners
  • rendered runner label remains self-hosted-ubuntu
  • all coordinator, runner-image, and prepared-image replacements resolve
  • git diff --check passes

@martinothamar
martinothamar requested a review from a team as a code owner August 20, 2026 06:31
@github-actions github-actions Bot added the skip-releasenotes Issues that do not make sense to list in our release notes label Aug 20, 2026
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@martinothamar, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 14 minutes

Limit details: You’ve used all 2 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 46ce3120-0ffc-4935-bbf6-85f30cfc911c

📥 Commits

Reviewing files that changed from the base of the PR and between 6888e85 and e41d4ae.

📒 Files selected for processing (12)
  • .github/actionlint.yaml
  • .github/workflows/deploy-github-runners.yaml
  • src/github-runner/Dockerfile
  • src/github-runner/coordinator/src/coordinator.rs
  • src/github-runner/coordinator/src/github.rs
  • src/github-runner/infra/kustomize/base/kustomization.yaml
  • src/github-runner/infra/kustomize/base/scaledjob-sandbox.yaml
  • src/github-runner/infra/kustomize/base/scaledjob-single.yaml
  • src/github-runner/infra/kustomize/base/scaledjob.yaml
  • src/github-runner/run-sandbox.sh
  • src/github-runner/run-with-dockerd.sh
  • src/github-runner/run.sh

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-releasenotes Issues that do not make sense to list in our release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant