fix: enforce self-hosted runners in workflows - #71
Closed
rtBot wants to merge 2 commits into
Closed
Conversation
Replace disallowed runs-on labels with [self-hosted] to comply with org runner policy. Auto-generated by rtcamp/sys runner enforcement workflow.
There was a problem hiding this comment.
Pull request overview
This PR updates GitHub Actions workflows to comply with an org policy requiring self-hosted runners, replacing runs-on: ubuntu-latest with runs-on: [self-hosted] across the repository’s workflows.
Changes:
- Switched all workflow jobs from GitHub-hosted runners (
ubuntu-latest) to self-hosted runners ([self-hosted]). - Applied the runner update consistently across CI/test and auto-merge workflows.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| .github/workflows/test-measure.yml | Updates all CI jobs (lint, PHPStan, WP/PHP matrix tests, coverage) to run on self-hosted runners. |
| .github/workflows/auto-merge.yml | Updates the Dependabot auto-merge workflow job to run on a self-hosted runner. |
dipankardas011
previously approved these changes
Jul 24, 2026
Signed-off-by: Dipankar Das <65275144+dipankardas011@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (2)
.github/workflows/test-measure.yml:112
- The PR description says it only replaces disallowed
runs-onlabels, but this workflow also adds a Docker Compose setup step (and related env changes elsewhere). Either update the PR description to reflect these functional changes, or split them into a separate PR so the runner-enforcement change remains isolated.
- name: Set up Docker Compose
uses: docker/setup-compose-action@4eb059ff7f16592f9c84d5ca339c53cb7c5064e2 # v2.3.0
with:
version: v2.36.2
.github/workflows/auto-merge.yml:20
- Switching from
ubuntu-latestto only[self-hosted]can change the OS/arch these jobs land on if your org has mixed self-hosted runner pools, potentially breaking assumptions (Linux tooling, paths, shell behavior). To preserve the prior environment constraints, consider adding the appropriate runner labels (e.g.,linux/x64) that match your self-hosted fleet.
runs-on: [self-hosted]
Comment on lines
102
to
104
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6 |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (1)
.github/workflows/test-measure.yml:112
- The PR description says it only replaces
runs-onlabels, but this workflow also adds new behavior (Docker Compose setup steps andWP_ENV_HOMEenv changes). Please update the PR description to reflect these additional changes (or split them into a separate PR) so reviewers/operators understand the full impact.
- name: Set up Docker Compose
uses: docker/setup-compose-action@4eb059ff7f16592f9c84d5ca339c53cb7c5064e2 # v2.3.0
with:
version: v2.36.2
Comment on lines
+102
to
+104
| # Must live on the shared work emptyDir mounted by ARC's dind containerMode | ||
| # so the dind sidecar can resolve the bind-mount source paths wp-env generates. | ||
| WP_ENV_HOME: /home/runner/_work/_temp/wp-env-home |
Comment on lines
+135
to
+138
| WP_ENV_PHP_VERSION: "8.3" | ||
| # Must live on the shared work emptyDir mounted by ARC's dind containerMode | ||
| # so the dind sidecar can resolve the bind-mount source paths wp-env generates. | ||
| WP_ENV_HOME: /home/runner/_work/_temp/wp-env-home |
|
as this repo became Private -> public |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR replaces disallowed
runs-onlabels with[self-hosted]in GitHub Actions workflow files.Why: Organization policy requires all private repos to use self-hosted runners.
Changes: All
runs-onvalues in workflow files have been updated to[self-hosted].