Skip to content

ci: pin .NET 8 runtime for mikeio1d#663

Closed
ecomodeller wants to merge 2 commits into
mainfrom
ci/pin-dotnet-8
Closed

ci: pin .NET 8 runtime for mikeio1d#663
ecomodeller wants to merge 2 commits into
mainfrom
ci/pin-dotnet-8

Conversation

@ecomodeller
Copy link
Copy Markdown
Member

Summary

The GitHub Actions ubuntu-24.04 runner image rolled forward from 20260513.135 to 20260518.149 on 2026-05-18 and now ships .NET 10. mikeio1d only loads under .NET 8 — see its README (sudo apt install dotnet-runtime-8.0) and its own CI config which pins dotnet-version: '8.0.x'.

Result: every modelskill job that pulls the networks group started failing with DllNotFoundException: Unable to load shared library 'ufs'.

This PR adds actions/setup-dotnet@v4 with 8.0.x to the three workflows that install the networks group, mirroring upstream.

Notes

  • mikeio1d's own scheduled CI didn't catch this because it pins .NET 8 explicitly — so it never tests against the runner's default. Worth flagging upstream so future .NET bumps in the runner are caught at the library, not at downstream consumers.

The GitHub Actions ubuntu-24.04 runner image bumped from
20260513.135 to 20260518.149 on 2026-05-18 and now ships .NET 10.
mikeio1d's native shim only loads under .NET 8 (documented in its
README and matched by its own CI), so the new image breaks the
networks-group tests with a DllNotFoundException for libufs.so.

Install .NET 8 explicitly via actions/setup-dotnet in the workflows
that pull the networks group: full_test, notebooks_test, docs.
@ecomodeller ecomodeller requested a review from ryan-kipawa May 28, 2026 10:58
setup-dotnet@v4 installs .NET 8 alongside the runner's pre-existing
.NET 10 in /usr/share/dotnet, and the .NET host still rolls forward
to 10.x at load time. Delete the newer runtimes after install so
mikeio1d resolves to .NET 8 as documented.
@ecomodeller
Copy link
Copy Markdown
Member Author

I surrender.

@ecomodeller ecomodeller deleted the ci/pin-dotnet-8 branch May 28, 2026 11:05
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