Skip to content

[release/8.0-staging] Use workstation GC in outerloop NativeAOT libs runs#105794

Merged
ilonatommy merged 1 commit intodotnet:release/8.0-stagingfrom
ilonatommy:backport/pr-95896/8.0-staging
Aug 5, 2024
Merged

[release/8.0-staging] Use workstation GC in outerloop NativeAOT libs runs#105794
ilonatommy merged 1 commit intodotnet:release/8.0-stagingfrom
ilonatommy:backport/pr-95896/8.0-staging

Conversation

@ilonatommy
Copy link
Member

Customer Impact

  • Customer reported
  • Found internally

Backport of #95896. Contributes to lowering the hit rate on #99423 - OOM failures in NativeAOT lanes. This is purely test infra change, so I am adding approved label directly.

Regression

  • Yes
  • No

Testing

CI.

Risk

Low, it's not a production-code change.

Linux has a habit of overpromising memory and then underdelivering when one wants to actually use it.

Native AOT test legs often get OOM killed. For Pri-0 runs we had success switching to workstation GC that is slower, but uses less memory. Try the same for libs testing.
@ilonatommy ilonatommy added Servicing-approved Approved for servicing release area-NativeAOT-coreclr labels Aug 1, 2024
@ilonatommy ilonatommy self-assigned this Aug 1, 2024
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Copy link
Member

@MichalStrehovsky MichalStrehovsky left a comment

Choose a reason for hiding this comment

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

Thanks!

@ilonatommy
Copy link
Member Author

/ba-g changes are in nativeAOT and failures on mono and coreclr lanes, failure is #82077

@ilonatommy ilonatommy merged commit 5257f42 into dotnet:release/8.0-staging Aug 5, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-NativeAOT-coreclr Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants