-
Notifications
You must be signed in to change notification settings - Fork 4.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test failure System.IO.Tests.FileInfo_SymbolicLinks.CreateSymbolicLink_PathToTarget_RelativeToLinkPath #74320
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch Issue DetailsRun: runtime-coreclr libraries-jitstress 20220821.1 Failed test:
Error message:
|
Likely nothing to do with codegen. |
@jeffschwMSFT, who do we need to assign this to? @agocke is out. |
Tagging subscribers to this area: @dotnet/area-system-io Issue DetailsRun: runtime-coreclr libraries-jitstress 20220821.1 Failed test:
Error message:
|
IO owners. Would need to look at dump. |
All the faulting tests are using RemoteExecutor, and the method where the dump occurs is System.Environmet.get_OSVersion() (I think?).
I'm unable to find the symbols for RemoteExecutor so that might give me more info about the error. |
Did you get the helix payload? It has remote executor binary but not symbols? Can you get a native callstack? Sometimes it is necessary to get several dumps and look for a pattern. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Since the payload is what is executed , that is hard to explain |
This comment was marked as off-topic.
This comment was marked as off-topic.
Actually, ignoring the dumps, this is just the RemoteExecutor timing out after 60s, Can we use a larger timeout on JitStress runs? |
I think you can't assume that the test isn't hung. @BruceForstall what environment variables should @jozkee set locally to get the test to run as slow as the slowest jitstress run that would run this test? @jozkee if you can verify that way that it's merely slow, I suppose we could make a change here to increase the timeout (eg x3) when DOTNET_JitStress=1 or whatever. |
In the console log, you can see this failure was run with:
The 2nd one means more JIT compilation. Setting |
I ran it locally with the jitstress variables but they didn't made the test execution slower at all: Baseline:
With set COMPlus_TieredCompilation=0 set COMPlus_ReadyToRun=0 & set COMPlus_ZapDisable=1
With above set + COMPlus_JITMinOpts=1
|
FWIW, I queried for "Half-way through waiting for remote process." issues and found the following. Failures with Jitstress and remotexecutor Failures with Jitstress and remotexecutor that were closed: Similar but not jitstress: Whatever the root cause is, its probably the same for this list of issues. |
All it means is "a test hung" or "a test was running very slowly". You can't conclude anything else without a dump or repro. |
In my experience the CI machines are often heavily loaded, and this leads them to unusual interleaving and delay in test execution that can be hard to repro locally. You might try loading down your machine when running the tests, either via some explicit stress program ( |
No repro since May #90041 (comment), will close for now. |
Run: runtime-coreclr libraries-jitstress 20220821.1
Failed test:
Error message:
The text was updated successfully, but these errors were encountered: