Skip to content

Switch away from AzureLinux host docker testing temporarily - #115993

Merged
jakobbotsch merged 1 commit into
dotnet:mainfrom
jakobbotsch:disable-azure-linux-testing-temporarily
May 27, 2025
Merged

Switch away from AzureLinux host docker testing temporarily#115993
jakobbotsch merged 1 commit into
dotnet:mainfrom
jakobbotsch:disable-azure-linux-testing-temporarily

Conversation

@jakobbotsch

@jakobbotschjakobbotsch commented May 26, 2025•
edited
Loading

Copy link
Copy Markdown
Member

containerd on the host has a default 1024 NOFILE limit, which is not enough for our testing. Switch to Ubuntu host testing until the changes that increase this limit are rolled out.

Tracked by dotnet/dnceng#5728 (PR is open internally)

Fix#115640

@jakobbotsch
Switch away from AzureLinux host docker testing temporarily
769915c
containerd on the host has a default 1024 NOFILE limit, which is not
enough for our testing. Switch to Ubuntu host testing until the changes
that increase this limit are rolled out.
CopilotAI review requested due to automatic review settings May 26, 2025 11:48
@github-actionsgithub-actionsBot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 26, 2025
CopilotAI reviewed May 26, 2025
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR temporarily switches the host Docker images for musl-based Linux testing queues from AzureLinux to Ubuntu due to NOFILE limit constraints.

  • Replaces the public Alpine x64 helix queue host from AzureLinux to Ubuntu.2204
  • Replaces the internal Alpine x64 helix queue host from AzureLinux to Ubuntu.2204

Comment threadeng/pipelines/coreclr/templates/helix-queues-setup.yml
Comment threadeng/pipelines/coreclr/templates/helix-queues-setup.yml
@jakobbotsch

jakobbotsch commented May 26, 2025

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-coreclr outerloop

@azure-pipelines

azure-pipelinesBot commented May 26, 2025

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@jkotas

jkotas commented May 26, 2025

Copy link
Copy Markdown
Member

cc @richlander

@richlander

richlander commented May 26, 2025•
edited
Loading

Copy link
Copy Markdown
Member

Responded here: dotnet/dnceng#5728 (comment)

Looks like we are running into: systemd/systemd#25478. I assume we do something special for this on Ubuntu.

@jakobbotsch

jakobbotsch commented May 27, 2025

Copy link
Copy Markdown
MemberAuthor

Looks like we are running into: systemd/systemd#25478. I assume we do something special for this on Ubuntu.

Yes, containerd in ubuntu is configured with

LimitNPROC=infinity
LimitCORE=infinity
LimitNOFILE=infinity

in its systemd configuration. Azure Linux 3 does not come with any such overrides.

@jakobbotschjakobbotsch added the area-Infrastructure-coreclr Only use for closed issues label May 27, 2025
@jakobbotschjakobbotsch removed the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 27, 2025
@dotnet-policy-service

dotnet-policy-serviceBot commented May 27, 2025

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

@jakobbotschjakobbotsch mentioned this pull request May 27, 2025
Closed
@jakobbotsch

jakobbotsch commented May 27, 2025

Copy link
Copy Markdown
MemberAuthor

/ba-g Issue is #116014

@jakobbotsch
jakobbotsch merged commit 2486c13 into dotnet:mainMay 27, 2025
@jakobbotsch
jakobbotsch deleted the disable-azure-linux-testing-temporarily branch May 27, 2025 16:50
jakobbotsch added a commit to jakobbotsch/runtime that referenced this pull request Jun 10, 2025
@jakobbotsch
Revert "Switch away from AzureLinux host docker testing temporarily (d…
e20afe9
…otnet#115993)"
This reverts commit 2486c13.
The limits were increased and the changes have been deployed.
jakobbotsch added a commit that referenced this pull request Jun 11, 2025
@jakobbotsch
Revert "Switch away from AzureLinux host docker testing temporarily (#…
8bcc9e2
…115993)" (#116501)
This reverts commit 2486c13.
The limits were increased and the changes have been deployed.
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Jun 27, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@richlanderrichlanderrichlander approved these changes

Assignees

@jakobbotschjakobbotsch

Labels

area-Infrastructure-coreclrOnly use for closed issues

Projects

Runtime Infra
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

JIT/Regression_2 test failures: No file descriptors available

4 participants

@jakobbotsch@jkotas@richlander