-
Notifications
You must be signed in to change notification settings - Fork 15
Update CI to run .NET 9 and 10 on alpine #374
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
On Fedora/RHEL, the @ayakael is this intentional? If this is intentional, we can add these packages to the README as dependencies for testing on alpine. |
|
Hey, @ayakael any thoughts? |
Hi! Thanks for pinging again, this is not intentional! I'll add these dependencies to AOT subpackage and backport to 3.22. |
|
MR:
dotnet10 is also set for inclusion with v3.23. (MR: https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/92484) |
Per redhat-developer/dotnet-regular-tests#374, matches Fedora behavior
Per redhat-developer/dotnet-regular-tests#374, matches Fedora behavior
|
The MR adding those dependencies just got merged. dotnet10 package will also be released with v3.23 in a few weeks. |
|
Thanks! |
.NET 9 and 10 are now available on alpine and edge.
|
Hey @ayakael do you have the cycles to investigate the test failures and propose fixes for this PR? These failing legs in our CI are becoming concerning and we are wondering if dropping alpine from our matrix would be easier for everyone. |
|
Thanks for pinging me! I maintain the dotnet package as a volunteer so can't be as proactive as I'd like to be. I'll take a look at these failures and come up with proposals. |
|
For all version, lttng can be disabled: diff --git a/lttng/test.json b/lttng/test.json
index 5bc879a..6bbf31c 100644
--- a/lttng/test.json
+++ b/lttng/test.json
@@ -10,6 +10,7 @@
"runtime=mono", // mono does not use lttng
],
"ignoredRIDs":[
+ "alpine", // lttng <=2,13,0 is broken
"centos",
"fedora", // see https://github.com/redhat-developer/dotnet-regular-tests/issues/202
"rhel"On dotnet10, we use bundled libunwind since it now seems broken on all arches instead of just on arm: diff --git a/system-libunwind/test.json b/system-libunwind/test.json
index aaa8e53..1ca2e40 100644
--- a/system-libunwind/test.json
+++ b/system-libunwind/test.json
@@ -7,6 +7,7 @@
"type": "bash",
"cleanup": true,
"skipWhen": [
+ "os=alpine,version=10",// system libunwind broken on dotnet10
"os=fedora", // using bundled libunwind
"vmr-ci", // using system libunwind is configured in the rpm build
"runtime=mono", // mono does not use libunwindBash completions on dotnet10 is a packaging bug, MR is open to address it: https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/94577
I'll look into |
|
The alpine:edge currently in use is old (from january 2025, before diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 095c822..43cd368 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -69,6 +69,7 @@ jobs:
if grep edge /etc/os-release; then
echo "https://dl-cdn.alpinelinux.org/alpine/edge/testing" >> /etc/apk/repositories
fi
+ apk upgrade -a
apk add dotnet-sdk-${{ matrix.dotnet_version }} dotnet-doc
apk add \
dotnet-sdk-dbg-${{ matrix.dotnet_version }} |
|
MR on Alpine is merged for latest and edge, implemented fixes for testsuite here: #403. Don't hesitate to flag other issues for me to fix. Every passes over there. |
|
Thanks! Closing this PR in favour of #403 |
.NET 9 and 10 are now available on alpine and edge.