Skip to content

ci: pin every GitHub-hosted Linux job to ubuntu-24.04 - #869

Merged
igerber merged 1 commit into
mainfrom
fix/ci-pin-ubuntu-24-04
Oct 25, 2026
Merged

igerber merged 1 commit into
mainfrom
fix/ci-pin-ubuntu-24-04

Conversation

@igerber

@igerber igerber commented Oct 25, 2026

Copy link
Copy Markdown
Owner

Summary

Methodology references (required if estimator / math changes)

  • Method name(s): N/A - no estimator or math changes.
  • Paper / source link(s): N/A
  • Any intentional deviations from the source (and why): None

Validation

  • Tests added/updated: No test changes. All ten workflow files parse as YAML; tests/test_doc_deps_integrity.py, tests/test_notebook_md_extract.py, tests/test_tracking_files.py, tests/test_openai_review.py (incl. TestLintWorkflowPinSync) and tests/test_changelog_fragments.py pass (677). The pull_request run of this PR itself exercises the pinned review job.
  • Backtest / simulation / notebook evidence (if applicable): N/A

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Changelog

  • changelog.d/ fragment added (or N/A - no user-visible change): N/A - CI configuration only, no diff_diff/ change.

Follow-ups

GitHub rolled `ubuntu-latest` to Ubuntu 26.04 on 2026-10-24. Two jobs
break on the new image: openai/codex-action's `drop-sudo` safety
strategy fails before codex starts (upstream codex-action#188), so the
AI PR review posted nothing on #868, and actions/python-versions has no
Python 3.9 build for 26.04, so the 3.9 floor leg of the test matrix and
the 3.9 wheel-test leg cannot set up.

Replace `ubuntu-latest` with `ubuntu-24.04` in all ten active workflows
(the same image every run used until yesterday) and record the pin and
its un-pin conditions in docs/dev-status.md. No test or source change.
@github-actions

Copy link
Copy Markdown

✅ Looks good

No P0–P3 findings identified in the supplied diff.

Executive summary

  • All active Linux runner references consistently use ubuntu-24.04 or ubuntu-24.04-arm.
  • The Python 3.9 matrix entry is updated correctly; ARM entries remain unchanged.
  • Changes affect CI configuration and developer documentation, with no estimator or inference changes.

Methodology

No findings. The Methodology Registry was consulted; no affected methods, equations, assumptions, weighting, or variance calculations require paper-level verification.

Code Quality

No findings. Direct runner settings and matrix entries are updated consistently. No conditions still depend on ubuntu-latest.

Performance

No findings. The manylinux build containers remain unchanged.

Maintainability

No findings. docs/dev-status.md:L159-L175 records the compatibility issues and concrete conditions for removing the pin.

Tech Debt

No findings. The temporary compatibility constraint and its resolution criteria are documented.

Security

No findings. The diff introduces no changes to permissions, action revisions, secrets handling, or sandbox settings.

Documentation/Tests

No findings. CI-only changes do not require a user-facing changelog fragment.

Validation confirmed that all 11 active workflows parse as YAML, all 23 Linux runner references use the intended labels, and both lint-pin consistency checks pass when executed directly. The full pytest suites could not run because pytest and scientific dependencies are absent from this environment.

@igerber igerber added the ready-for-ci Triggers CI test workflows label Oct 25, 2026
@igerber
igerber merged commit 2ede7bd into main Oct 25, 2026
53 of 54 checks passed
@igerber
igerber deleted the fix/ci-pin-ubuntu-24-04 branch October 25, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-ci Triggers CI test workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant