Skip to content

ci(pr): pin Test job runner to ubuntu-24.04 (RIG-9206) - #12

Closed
rigel-mintaka wants to merge 2 commits into
dependencies/rig-9272-publish-solid-virtual-tarball-to-githubfrom
dependencies/rig-9206-pin-test-job-runner-to-ubuntu-24-04
Closed

rigel-mintaka wants to merge 2 commits into
dependencies/rig-9272-publish-solid-virtual-tarball-to-githubfrom
dependencies/rig-9206-pin-test-job-runner-to-ubuntu-24-04

Conversation

@rigel-mintaka

@rigel-mintaka rigel-mintaka commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator

This PR is part of a stack containing 2 PRs:

  1. rigel-release
  2. chore: add hk pre-push config and pin the Test runner (RIG-9272) #10
  3. "ci(pr): pin Test job runner to ubuntu-24.04 (RIG-9206)" (this PR)

🎯 Changes

Pins the Test job in .github/workflows/pr.yml to ubuntu-24.04. Every other job keeps ubuntu-latest.

ubuntu-latest now resolves to an image Playwright does not support for chromium. The Test job fails at pnpm exec playwright install chromium on every PR:

Error: ERROR: Playwright does not support chromium on ubuntu26.04-x64

A runner pin is one line. A Playwright bump would change the lockfile and the browser set. Matt's ruling for this lane is the smaller fix. A later bump can replace the pin.

Stacked on #10. This branch sits on the #10 tip (29e6943a), which reverts the GitHub Release upload (RIG-9386) and keeps hk.pkl. The repo pre-push gate needs hk.pkl, which the #10 chain carries. Merge #10 first, or rebase this PR onto rigel-release once #10 lands.

Verification: yq -e '.jobs.test."runs-on"' .github/workflows/pr.yml returns ubuntu-24.04. The diff against #10 is one line. CI on this PR is the check: the Test job should get past browser install.

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr. Not run locally: CI is the verification for this lane.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

Refs RIG-9206

Co-authored-by: Matt Wilkinson matt@rigel.build

rigel-mintaka and others added 2 commits November 11, 2026 02:35
Co-authored-by: Matt Wilkinson <matt@rigel.build>
The ubuntu-latest image moved to 26.04, which Playwright does not support for chromium. The Test job fails at browser install on every PR. Pinning the runner keeps the Test job on a supported image until Playwright catches up.

Stacked on virtual#10 (RIG-9272 chain) so the hk pre-push gate sees hk.pkl.

Refs RIG-9206

Co-authored-by: Matt Wilkinson <matt@rigel.build>
@linear-code

linear-code Bot commented Nov 11, 2026

Copy link
Copy Markdown

RIG-9206

@github-actions

Copy link
Copy Markdown

🚀 Changeset Version Preview

No changeset entries found. Merging this PR will not cause a version bump for any packages.

@rigel-mintaka

Copy link
Copy Markdown
Collaborator Author

Folded into #10. The Test runner pin now lives on the #10 stack, so #10 carries both the hk.pkl config and the ubuntu-24.04 pin. Closing this PR; the branch is retired.

@rigel-mintaka
rigel-mintaka deleted the dependencies/rig-9206-pin-test-job-runner-to-ubuntu-24-04 branch November 11, 2026 07:42
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