-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[main] Update dependencies from dotnet/roslyn-analyzers #106072
Conversation
…uild 20240805.1 Microsoft.CodeAnalysis.Analyzers , Microsoft.CodeAnalysis.NetAnalyzers From Version 3.11.0-beta1.24402.1 -> To Version 3.11.0-beta1.24405.1
|
@mangod9 this looks like reflection and not related to the flow but I didn't want to open a known issue without checking |
Adding @ericstj who owns reflection. I havent seen this particular failure before. Is it consistent? |
This is in the VM: runtime/src/coreclr/vm/comdelegate.cpp Line 992 in 4bd1492
There's a log with dump analysis It claims an access violation here:
Can someone from runtimne have a look? @jkotas @AaronRobinsonMSFT |
Seems like same symptom as #55536. I wish we had the dump here - it seems like it wasn't uploaded @carlossanlop @hoyosjs |
Taking a look. |
I think it is a regression introduced by #105480 . |
This reverts commit 81976ed.
Pushed a revert |
@ericstj - I see a dump dotnet.exe.7864.dmp and for your particular one dotnet.exe.1912.dmp |
Huh, for some reason I couldn't see in the test tab, nor would runfo pull it down. How did you find that? |
@cshung had also hit an issue where the dump was not available where it should have been collected. Perhaps some generic issue here? |
If you did not see the test tab, it is a problem in AzDo. The AzDo is responsible to show the test tab. I can see the test tab fine for the both the Windows and Linux crashes earlier, and runfo command worked fine too.
We have several generic issues with collecting dumps (e.g. dumps not collected for build crashes). There are tracking issues opened for the known generic issues. But I do not see an issue with collecting crash dumps in this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All failures are known issues according to the build analysis
This pull request updates the following dependencies
From https://github.com/dotnet/roslyn-analyzers