ci: run ui + chart jobs on watcher-runners#52
Merged
Conversation
Move the lightweight ui (setup-node + npm) and chart (helm lint) jobs to the self-hosted watcher-runners — they bring their own tooling and run fine on ARM. server stays on GitHub-hosted: its postgres service container needs a local Docker daemon, which the ARC runners don't have (image builds offload to remote BuildKit). image was already on watcher-runners. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The get-helm-3 script sudo-installs to /usr/local/bin, which fails on the watcher-runners (no_new_privileges blocks sudo). setup-helm uses the tool cache. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
v5.0.0 is the current release; the only change from v4 is the node20->node24 runtime bump, already proven on these runners (checkout@v6 is node24 too). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
Moves the ui and chart jobs to the self-hosted
watcher-runners; server stays onubuntu-latest.npm ci/build) and chart (curl helm +helm lint) bring their own tooling and run fine on the ARM runners.ubuntu-latestbecause itspostgres:16service container needs a local Docker daemon, which the ARC runners don't have (image builds offload to remote BuildKit).imagewas already onwatcher-runners.First run will confirm ui/chart behave on the ARC runners (ARM); if anything trips, it's easy to revert the one job.
🤖 Generated with Claude Code