chore(ci): migrate runners to ubuntu-latest, drop macOS build leg#4850
Merged
Conversation
timwu20
force-pushed
the
chore/migrate-ci-runners-to-warp
branch
from
5495f16 to
1fb39c0
Compare
BuildJet hosted runners are no longer available, causing all PR workflows that target `buildjet-*-ubuntu-2204` labels to queue indefinitely and ultimately fail with a 24h timeout. Warp runners configured for the org also failed to pick up queued jobs. Switch every `runs-on:` to GitHub-hosted `ubuntu-latest`, which is free for public repositories and has no provisioning dependency. Note that some jobs previously targeted 16vcpu runners (unit-tests, docker-stress, docker-rpc) and may now run slower on the 2 vCPU / 7 GB standard runners — revisit with a paid larger runner if needed. Also drop `macos-latest` from the `build` job matrix. No one ships gossamer to macOS users and the leg consumed CI minutes without catching anything in recent memory. Collapse the single-platform matrix to a plain `runs-on: ubuntu-latest`. The `CI=buildjet` environment variable passed to `go test` in unit-tests and integration-tests workflows is intentionally preserved — it's a signal value read by test code, not a runner reference.
timwu20
force-pushed
the
chore/migrate-ci-runners-to-warp
branch
from
1fb39c0 to
e86c729
Compare
timwu20
changed the title
The unit-tests job's 15-minute job-level and go test internal timeouts were sized for the previous 16vcpu runner. On the standard 2 vCPU ubuntu-latest runner the suite now takes longer to complete and is hitting the 15m limit. Bump both timeouts to 30m.
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
1 participant
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
runs-on:to GitHub-hostedubuntu-latest(free for public repos, no provisioning dependency).macos-latestfrom thebuildmatrix and collapse the single-platform matrix to a plainruns-on.Trade-offs
unit-tests,docker-stress,docker-rpc) now run on the standard 2 vCPU / 7 GB runner. They may be slower; revisit with a paid GitHub larger runner if a job consistently bumps into itstimeout-minutes.The
CI=buildjetenv var passed togo testinunit-tests.ymlandintegration-tests.ymlis intentionally left alone — it's a signal value read by test code, not a runner reference. Rename in a follow-up if desired.