Skip to content

chore(github-runner): remove unwired ARC overlay - #59

Merged
surrealwolf merged 1 commit into
mainfrom
chore/remove-github-runner
Oct 6, 2026
Merged

surrealwolf merged 1 commit into
mainfrom
chore/remove-github-runner

Conversation

@surrealwolf

@surrealwolf surrealwolf commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

github-runner/overlays/nprd-apps (an ARC AutoscalingRunnerSet) was dead code. It isn't in any Fleet GitRepo path (live paths: gitlab-runner, harbor, grafana, monitoring on nprd; monitoring on poc; monitoring and authentik on prd), and fleet-gitrepo.yaml only had it commented out. The ARC controller it needed was uninstalled from all clusters on 2026-10-05, after confirming zero runner scale sets anywhere (DataKnifeAI/rancher-deploy#46).

Changes

  • Delete github-runner/, docs/github-runner/, docs/GITHUB_RUNNER.md, secrets/github-runner/.
  • scripts/runner-setup.sh: GitLab only now (all kept as an alias; it no longer creates actions-runner-system).
  • docs/DEPLOYMENT.md, docs/FLEET_STRUCTURE.md, docs/README.md, README.md, fleet-gitrepo.yaml: drop GitHub runner references.

No live impact: Fleet never deployed this path.


Note

Low Risk
Documentation and dead-manifest removal only; Fleet was not deploying the removed overlay.

Overview
Removes the unused GitHub Actions Runner (official ARC AutoscalingRunnerSet) stack from the repo after the controller was uninstalled and Fleet never synced github-runner/overlays/nprd-apps.

Deletes github-runner/ (base + nprd-apps overlay), docs/github-runner/, docs/GITHUB_RUNNER.md, and secrets/github-runner/. fleet-gitrepo.yaml drops the commented github-runner path so monitored paths match what actually deploys (GitLab runner, Harbor, Grafana, monitoring).

scripts/runner-setup.sh is GitLab-only: no actions-runner-system namespace or GitHub PAT/App secret flow; default action is gitlab (all still aliases to GitLab). Deployment and Fleet docs (DEPLOYMENT.md, FLEET_STRUCTURE.md, docs/README.md, root README structure tree) are trimmed to Harbor + GitLab runner workflows.

No expected cluster impact if Fleet was already excluding this path.

Reviewed by Cursor Bugbot for commit 4210843. Bugbot is set up for automated code reviews on this repo. Configure here.

github-runner/overlays/nprd-apps was never in any Fleet GitRepo path and
the ARC controller it depended on was uninstalled from all clusters on
2026-10-05 (rancher-deploy#46). Drop the overlay, its docs and secret
example, and the GitHub half of runner-setup.sh.

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved. Cursor Bugbot completed successfully and reported no findings that require human review. This automation is approving based on that clean signal.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@surrealwolf
surrealwolf merged commit 9c461f3 into main Oct 6, 2026
3 checks passed
@surrealwolf
surrealwolf deleted the chore/remove-github-runner branch October 6, 2026 03:57
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