[Loader][tests] Add more logging to pinpoint BinderTracingTests hang #104982
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
BinderTracingTests has experienced flakey test hangs #104670 #97735 and #94390
The exact cause of the hang has been elusive due to insufficient diagnostics.
Given the console log from the test
coupled with the native stack trace from the dump
The most recent flakey test hang occurred while waiting for the test running in an isolated subprocess, and it's not clear whether the subprocess hit a race condition, if the subprocess exited and its signal never made it back to
p.WaitForExit()
, or something else.To note, there was only dump collected despite the hang occurring during a test that spins up a subprocess. We would expect the subprocess to have a dump collected as well given that
runtime/src/tests/Common/Coreclr.TestWrapper/CoreclrTestWrapperLib.cs
Line 792 in d710379
This PR aims to add slightly more logging to help pinpoint where the test hang occurs, with the expectation that the next time the test suite flakes (if at all), we can have more information to work with.