Skip to content

Configure Server GC for NativeAOT regression test - #131727

Merged
jkotas merged 1 commit into
dotnet:mainfrom
kkokosa:fix/131669-requires-servergc-in-gettotalallocatedbytes-test
Aug 3, 2026
Merged

Configure Server GC for NativeAOT regression test#131727
jkotas merged 1 commit into
dotnet:mainfrom
kkokosa:fix/131669-requires-servergc-in-gettotalallocatedbytes-test

Conversation

@kkokosa

@kkokosa kkokosa commented Aug 3, 2026

Copy link
Copy Markdown
Member

Fixes #131669.

PR #131069 added a Server GC regression test, but NativeAOT selects the GC implementation at link time. Set ServerGarbageCollection so the test runs correctly in NativeAOT outerloop.

Note

This pull request was created with GitHub Copilot assistance.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 3, 2026 08:18
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).
13 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @anicka-net, @dotnet/gc
See info in area-owners.md if you want to be subscribed.

Copilot AI 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 updates the GC regression test project configuration so the test can reliably run with Server GC when executed under NativeAOT, where the GC flavor is selected at link time rather than via runtime environment variables.

Changes:

  • Set ServerGarbageCollection to true in the test’s .csproj so NativeAOT links the Server GC runtime.
  • Add an explanatory comment clarifying why the property is needed for NativeAOT.

@jkotas

jkotas commented Aug 3, 2026

Copy link
Copy Markdown
Member

/azp run runtime-nativeaot-outerloop

@azure-pipelines

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

@jkotas jkotas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM once the test is confirmed to pass

@jkotas
jkotas merged commit 403a194 into dotnet:main Aug 3, 2026
102 of 110 checks passed
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 11.0-rc1 milestone Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

System.Exception: ERROR: Requires server GC in GC\API\GC\GetTotalAllocatedBytesServerGC\GetTotalAllocatedBytesServerGC.cmd

3 participants