Skip to content

ci: pin native checks to Ubuntu 24.04 - #111

Merged
AojdevStudio merged 1 commit into
mainfrom
fix/ci-ubuntu24
Nov 11, 2026
Merged

AojdevStudio merged 1 commit into
mainfrom
fix/ci-ubuntu24

Conversation

@AojdevStudio

@AojdevStudio AojdevStudio commented Nov 11, 2026 •

Copy link
Copy Markdown
Owner

Problem

The hosted Linux check fails three Hush installer tests on Ubuntu's multicall coreutils binaries.
The binaries have extra hard links that the installer custody check correctly rejects.
This blocks the clipboard regression test in #109.

Change

Pin only the native-check job to the supported Ubuntu 24.04 runner.

   check:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04

The security scan and PR contract jobs keep their existing runners.
All test commands and executable-custody rules stay the same.

Proof

Check Before After
Hosted installer tests 254 passed, 3 failed because wc and printf have extra hard links Pass in both hosted Ubuntu 24.04 runs
actionlint Not run Pass
Full local check on macOS Not run Pass
Package dry run Not run Pass
Offline Pi extension smoke Not run Pass

Before run: https://github.com/AojdevStudio/agentic-utilities/actions/runs/43074331325

Passing run: https://github.com/AojdevStudio/agentic-utilities/actions/runs/43077730922

Review

Finding Disposition Reason
Other jobs still use ubuntu-latest Decline Only the check job runs the installer tests, at .github/workflows/ci.yml:24. The pin stays limited to that job.

Claude Sonnet 5.5 reviewed the exact head. PASS, zero required fixes.

Merge danger

Door: two-way. Revert the runner label to undo this change.
Blast radius: low. Only the Linux check environment changes. The full unchanged suite must pass before merge.

Closes #110
Changes made by GPT-6.1 Sol via Codex

@coldtea-pr-lens

Copy link
Copy Markdown

Note

The title starts with ci:, so PR Lens left this pull request undrawn. Comment @pr-lens draw to draw it

github.comment.notice: false in .github/pr-lens.yml turns this note off

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Pre-merge checks: Inconclusive 2

Failed Failed checks: 2 inconclusive

Check name Status Explanation Resolution
Title check Pending This assessment has not completed. The check will be retried on the next pre-merge check run.
Description check Pending This assessment has not completed. The check will be retried on the next pre-merge check run.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes

Reviewed the native CI runner change and its surrounding workflow configuration.

  • Pin native checks — The check job now runs on ubuntu-24.04; the separate secret-scan job remains unchanged.

Pullfrog  | View workflow run | Using gpt-6-luna | 𝕏

@AojdevStudio

Copy link
Copy Markdown
Owner Author

Spec fidelity checked against #110 on 9b2ba02. Only the native check runner label changes. All test commands and executable-custody checks remain intact. Both hosted Ubuntu 24.04 runs passed the full check and package dry run. No deviations.

@AojdevStudio
AojdevStudio merged commit ac8aca7 into main Nov 11, 2026
10 checks passed
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.

CI rejects hosted Ubuntu multicall coreutils in installer tests

1 participant