Skip to content

ci: revert self-hosted toggle on this public repo (hosted-only) - #94

Merged
WomB0ComB0 merged 10 commits into
mainfrom
ci/revert-self-hosted-public
Sep 29, 2026
Merged

WomB0ComB0 merged 10 commits into
mainfrom
ci/revert-self-hosted-public

Conversation

@WomB0ComB0

@WomB0ComB0 WomB0ComB0 commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Reverts the self-hosted runner toggle on this public repo back to ubuntu-latest.

Public repositories get GitHub-hosted Actions for free and are excluded from the org's self-hosted runner group (which also disallows public repos), so the toggle was inert here and never routed to a self-hosted runner. Self-hosted is reserved for the private repos that were affected by the hosted-runner outage. Removing the unused toggle to keep public CI clearly hosted-only.

Merge on green.

Summary by CodeRabbit

  • Chores
    • Automated repository checks and maintenance tasks now run on standard hosted Ubuntu runners instead of conditionally using self-hosted runners. The required-check gate continues to assess the results of its required jobs.
  • User impact
    • No direct changes to application features or behavior.

@WomB0ComB0
WomB0ComB0 requested a review from a team as a code owner September 29, 2026 18:36
@github-actions github-actions Bot added size/S PR size: small (10-29 lines changed) A-DevOps CI, workflows, actions, and repo infra (.github) C-Chore Chore: deps, tooling, or config with no public API change and removed size/S PR size: small (10-29 lines changed) labels Sep 29, 2026
@coderabbitai

coderabbitai Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 99045101-2e06-4f1b-9d83-9044caf17b9d

📥 Commits

Reviewing files that changed from the base of the PR and between 17907fd and f1da723.

📒 Files selected for processing (10)
  • .github/workflows/conventional-title-labeler.yml
  • .github/workflows/dependency-licences.yml
  • .github/workflows/hooks-sync.yml
  • .github/workflows/hooks-tests.yml
  • .github/workflows/label-sync.yml
  • .github/workflows/labeler.yml
  • .github/workflows/pr-size-labeler.yml
  • .github/workflows/repo-drift.yml
  • .github/workflows/required.yml
  • .github/workflows/worker-live.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Workflow jobs that previously selected self-hosted runners based on USE_SELF_HOSTED now use ubuntu-latest unconditionally. The required-check gate still evaluates the results of its three required jobs.

Changes

Workflow runner selection

Layer / File(s) Summary
Set workflow jobs to ubuntu-latest
.github/workflows/*
The listed workflow jobs now use ubuntu-latest unconditionally. The required-check gate still evaluates the results of its three required jobs.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: resq-sw

Merge Risk: ⚪ Minimal · up to f1da7

The workflows now use GitHub-hosted runners as intended, with the required-check gate preserved. No concrete merge-blocking risk is evident.

Architecture Summary

Architecture risk: 🔵 Low · up to f1da7

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/workflows/conventional-title-labeler.yml: The job now always uses ubuntu-latest; the prior conditional selection of the self-hosted Linux x64 zima runner when vars.USE_SELF_HOSTED was true was removed.
  • observed — Modified behavior in .github/workflows/dependency-licences.yml: licence-check now always uses ubuntu-latest; the previous USE_SELF_HOSTED == 'true' condition that selected the self-hosted Linux x64 zima runner was removed.
  • observed — Modified behavior in .github/workflows/hooks-sync.yml: The lint job now always uses ubuntu-latest; it no longer selects the self-hosted Linux x64 zima runner when USE_SELF_HOSTED is true.
  • observed — Modified behavior in .github/workflows/hooks-sync.yml: The smoke job now always uses ubuntu-latest; it no longer selects the self-hosted Linux x64 zima runner when USE_SELF_HOSTED is true.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: reverting the public repository’s CI workflows to hosted Ubuntu runners and removing the self-hosted toggle.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
get-resq-software f1da723 Sep 29 2026, 06:36 PM

@WomB0ComB0
WomB0ComB0 merged commit 3875863 into main Sep 29, 2026
34 checks passed
@WomB0ComB0
WomB0ComB0 deleted the ci/revert-self-hosted-public branch September 29, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-DevOps CI, workflows, actions, and repo infra (.github) C-Chore Chore: deps, tooling, or config with no public API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant