Exclude ResolvedFromDifferentContext collectible-unload test from GC stress by Copilot · Pull Request #130528 · dotnet/runtime

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Copilot · 2026-07-10T21:17:19Z

Loader/CollectibleAssemblies/ResolvedFromDifferentContext fails intermittently under GC stress. TestFullUnload polls a fixed number of GC iterations to confirm collectible AssemblyLoadContexts have unloaded; GC stress perturbs timing so collection doesn't complete in that window, yielding a spurious Failed to unload alc1 (exit code 103).

This change adds the SkipOnCoreClr attribute to the test for all GC stress modes.

cc @dotnet/appmodel @AaronRobinsonMSFT

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

Copilot

Pull request overview

This PR updates the CoreCLR loader test configuration to opt ResolvedFromDifferentContext out of GC stress runs, aligning it with other collectible-unload tests that rely on timing-sensitive unload completion.

Changes:

Mark ResolvedFromDifferentContext as GC-stress-incompatible via <GCStressIncompatible>true</GCStressIncompatible>.
Add a brief comment explaining the timing sensitivity (fixed-iteration polling) that makes the test unreliable under GC stress.

Show a summary per file

File	Description
`src/tests/Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj`	Excludes the test from GC stress by setting `GCStressIncompatible` and documenting the rationale.

Copilot's findings

Files reviewed: 1/1 changed files
Comments generated: 0

dotnet-policy-service · 2026-07-13T18:12:36Z

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

Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

elinor-fung · 2026-07-15T19:34:29Z

/ba-g unrelated timeouts

Initial plan
1bef576

CopilotAI assigned Copilot and elinor-fung Jul 10, 2026

CopilotAI requested review from Copilot and removed request for Copilot July 10, 2026 21:17

github-project-automationBot added this to AppModel Jul 10, 2026

Copilotstarted work on behalf of elinor-fung July 10, 2026 21:17 View session

CopilotAI linked an issue Jul 10, 2026 that may be closed by this pull request
Test failure: Loader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext/ResolvedFromDifferentContext.cmd #112618
Closed

Mark ResolvedFromDifferentContext test as GCStressIncompatible
99414a7
Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

CopilotAI requested review from Copilot and removed request for Copilot July 10, 2026 21:21

CopilotAI changed the title ~~[WIP] Fix test failure in Loader/CollectibleAssemblies~~Exclude ResolvedFromDifferentContext collectible-unload test from GC stressJul 10, 2026

Copilotfinished work on behalf of elinor-fung July 10, 2026 21:22

CopilotAI requested a review from elinor-fung July 10, 2026 21:22

github-actionsBot added the area-AssemblyLoader-coreclr only use for closed issues label Jul 10, 2026

CopilotAItemporarily deployed to copilot-pat-pool July 11, 2026 00:44 Inactive

build-analysisBot mentioned this pull request Jul 11, 2026
[ci-scan] Test failure: System.Globalization.Tests.IdnMappingGetUnicodeTests DllNotFoundException 'libSystem.Globalization.Native' on browser-wasm CoreCLR #130479
Closed

elinor-fung marked this pull request as ready for review July 13, 2026 17:50

CopilotAI review requested due to automatic review settings July 13, 2026 17:50

Copilotstarted reviewing on behalf of elinor-fung July 13, 2026 17:51 View session

CopilotAI reviewed Jul 13, 2026
View reviewed changes

elinor-fung added area-AssemblyLoader and removed area-AssemblyLoader-coreclr only use for closed issues labels Jul 13, 2026

MichalStrehovsky reviewed Jul 13, 2026
View reviewed changes

Comment thread...oader/CollectibleAssemblies/ResolvedFromDifferentContext/ResolvedFromDifferentContext.csproj Outdated

Copilotstarted work on behalf of elinor-fung July 13, 2026 21:51 View session

Use SkipOnCoreClr attribute instead of GCStressIncompatible property
c637624
Co-authored-by: elinor-fung <47805090+elinor-fung@users.noreply.github.com>

CopilotAI requested review from Copilot and removed request for Copilot July 13, 2026 21:58

Copilotfinished work on behalf of elinor-fung July 13, 2026 21:58

MichalStrehovsky approved these changes Jul 14, 2026
View reviewed changes

CopilotAItemporarily deployed to copilot-pat-pool July 14, 2026 02:22 Inactive

CopilotAItemporarily deployed to copilot-pat-pool July 14, 2026 02:23 Inactive

build-analysisBot mentioned this pull request Jul 14, 2026
The Operation will be canceled. The next steps may not contain expected logs. dotnet/dnceng#3008
Open
3 tasks

elinor-fung merged commit 9e39cbe into mainJul 15, 2026
87 of 90 checks passed

elinor-fung deleted the copilot/fix-loader-collectibleassemblies-test branch July 15, 2026 19:34

github-project-automationBot moved this to Done in AppModel Jul 15, 2026

dotnet-maestroBot mentioned this pull request Jul 16, 2026
[main] Source code updates from dotnet/runtime dotnet/dotnet#7778
Merged

dotnet-milestone-botBot added this to the 11.0-preview7 milestone Jul 16, 2026

github-actionsBot locked and limited conversation to collaborators Aug 16, 2026