Skip to content
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

Bump dotnet SDK to dotnet 8. #3500

Merged
merged 1 commit into from
Oct 16, 2024
Merged

Bump dotnet SDK to dotnet 8. #3500

merged 1 commit into from
Oct 16, 2024

Conversation

TingluoHuang
Copy link
Member

Dotnet 6 is approaching end of life, and this PR updates the dotnet SDK to dotnet 8.

There will be other followup PR to clean up more stuff.

Dotnet8 is no longer supported on centos7, this means the runner will not work on that platform anymore.

@TingluoHuang TingluoHuang requested a review from a team as a code owner October 10, 2024 19:40
@TingluoHuang TingluoHuang merged commit af8dee5 into main Oct 16, 2024
10 checks passed
@TingluoHuang TingluoHuang deleted the users/tihuang/dotnet88 branch October 16, 2024 16:32
YorikSar added a commit to YorikSar/nixpkgs that referenced this pull request Nov 15, 2024
* remove mentions of Node.js 16 which is not supported anymore
  see actions/runner#3503
* switch to .Net SDK 8
  see actions/runner#3500
* fix source path discovery in deterministic build: runtime assumes that
  the root of the project is `/_`, replace it with actual `/build/src`
  so that tests can find their data.
YorikSar added a commit to YorikSar/nixpkgs that referenced this pull request Nov 15, 2024
* remove mentions of Node.js 16 which is not supported anymore
  see actions/runner#3503
* switch to .Net SDK 8
  see actions/runner#3500
* fix source path discovery in deterministic build: runtime assumes that
  the root of the project is `/_`, replace it with actual `/build/src`
  so that tests can find their data.
YorikSar added a commit to YorikSar/nixpkgs that referenced this pull request Nov 17, 2024
* remove mentions of Node.js 16 which is not supported anymore
  see actions/runner#3503
* switch to .Net SDK 8
  see actions/runner#3500
* fix source path discovery in deterministic build: runtime assumes that
  the root of the project is `/_`, replace it with actual `/build/src`
  so that tests can find their data.
github-actions bot pushed a commit to NixOS/nixpkgs that referenced this pull request Nov 19, 2024
* remove mentions of Node.js 16 which is not supported anymore
  see actions/runner#3503
* switch to .Net SDK 8
  see actions/runner#3500
* fix source path discovery in deterministic build: runtime assumes that
  the root of the project is `/_`, replace it with actual `/build/src`
  so that tests can find their data.

(cherry picked from commit db1d5cf)
YorikSar added a commit to YorikSar/nixpkgs that referenced this pull request Nov 19, 2024
* remove mentions of Node.js 16 which is not supported anymore
  see actions/runner#3503
* switch to .Net SDK 8
  see actions/runner#3500
* fix source path discovery in deterministic build: runtime assumes that
  the root of the project is `/_`, replace it with actual `/build/src`
  so that tests can find their data.

(cherry picked from commit db1d5cf)
osnyx pushed a commit to flyingcircusio/nixpkgs that referenced this pull request Nov 25, 2024
* remove mentions of Node.js 16 which is not supported anymore
  see actions/runner#3503
* switch to .Net SDK 8
  see actions/runner#3500
* fix source path discovery in deterministic build: runtime assumes that
  the root of the project is `/_`, replace it with actual `/build/src`
  so that tests can find their data.

(cherry picked from commit db1d5cf)
(cherry picked from commit 2c6e9e4)
theihor added a commit to theihor/bpfci-runner that referenced this pull request Nov 28, 2024
Github Actions Runner.Listener x86_64 binary version 2.321.0 has been
crashing with segfault on startup when emulated on s390x [1].

Ilya pointed out, that the root cause of this crash is most likely a
qemu issue [2].

.NET 8 turns on "write xor execute" feature [3] by default.
And GH actions runner 2.320.0->2.321.0 update includes an upgrade to
.NET 8 [4].

Ilya suggested opting out of this feature in our environment to
mitigate the failure.

I tested this with manually built runner immage on one for s390x BPF
CI hosts: the runner doesn't crash.

[1] actions/runner#3583
[2] https://gitlab.com/qemu-project/qemu/-/issues/2600
[3]
https://devblogs.microsoft.com/dotnet/announcing-net-6-preview-7/#runtime:-w^x-(write-xor-execute)-support-for-all-platforms-and-architectures
[4] actions/runner#3500

Suggested-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Ihor Solodrai <isolodrai@meta.com>
theihor added a commit to kernel-patches/runner that referenced this pull request Nov 28, 2024
Github Actions Runner.Listener x86_64 binary version 2.321.0 has been
crashing with segfault on startup when emulated on s390x [1].

Ilya pointed out, that the root cause of this crash is most likely a
qemu issue [2].

.NET 8 turns on "write xor execute" feature [3] by default.
And GH actions runner 2.320.0->2.321.0 update includes an upgrade to
.NET 8 [4].

Ilya suggested opting out of this feature in our environment to
mitigate the failure.

I tested this with manually built runner immage on one for s390x BPF
CI hosts: the runner doesn't crash.

[1] actions/runner#3583
[2] https://gitlab.com/qemu-project/qemu/-/issues/2600
[3]
https://devblogs.microsoft.com/dotnet/announcing-net-6-preview-7/#runtime:-w^x-(write-xor-execute)-support-for-all-platforms-and-architectures
[4] actions/runner#3500

Suggested-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Ihor Solodrai <isolodrai@meta.com>
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.

2 participants