Skip to content

test_groups: route FMI + Initialization to GitHub-hosted runners (fix OOM kills on pre)#4646

Merged
AayushSabharwal merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:fmi-init-highmem-runner
Jun 22, 2026
Merged

test_groups: route FMI + Initialization to GitHub-hosted runners (fix OOM kills on pre)#4646
AayushSabharwal merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:fmi-init-highmem-runner

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor

Fixes the OOM kills @AayushSabharwal hit on the FMI and Initialization groups (esp. pre) — e.g. run 27742584454 jobs 82073267896 / 82073267635, both on self-hosted-4vcpu-8gb-… runners.

Per @ChrisRackauckas's test_groups runner-designation suggestion: set runner = "ubuntu-24.04" on FMI + Initialization (root) and the ModelingToolkitBase Initialization sublib group. GitHub-hosted ubuntu runners have ~16GB (2x the 8GB self-hosted), and ubuntu-24.04 forces GitHub-hosted (the self-hosted pool only carries ubuntu-latest).

Note: runner is per-group, not per-version, so this applies to lts/1/pre for those groups (not pre-only) — harmless, just gives the lighter versions more headroom too. Verified locally via compute_affected_sublibraries.jl --root-matrix: FMI/Initialization emit runner=ubuntu-24.04, all other groups unchanged. The PR's own CI on those groups is the end-to-end confirmation. If 16GB still isn't enough on pre, the next step is a dedicated large runner (org config).

… OOM kills)

The FMI and Initialization groups are memory-heavy and were getting OOM-killed
on the 4vcpu/8GB self-hosted runners that now back the `ubuntu-latest` label,
especially on `pre` (e.g. run 27742584454, jobs 82073267896 / 82073267635).

Designate `runner = "ubuntu-24.04"` for those groups (root + the
ModelingToolkitBase sublibrary's Initialization). GitHub-hosted ubuntu runners
have ~16GB (2x the 8GB self-hosted), and `ubuntu-24.04` deterministically
forces a GitHub-hosted runner because the self-hosted pool only carries the
`ubuntu-latest` label. Verified with compute_affected_sublibraries.jl
--root-matrix: FMI/Initialization now emit runner=ubuntu-24.04, all other
groups stay ubuntu-latest.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@AayushSabharwal AayushSabharwal merged commit 238ab41 into SciML:master Jun 22, 2026
65 of 90 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.

3 participants