-
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
Test failure readytorun/determinism/crossgen2determinism/crossgen2determinism.sh #57620
Comments
Turning this into a tracking issue. This is currently one of the big hits. |
This slipped to release/6.0: https://runfo.azurewebsites.net/search/tests/?q=started%3A~7%20definition%3Aruntime%20targetbranch%3Arelease%2F6.0%20name%3A%22readytorun%2Fdeterminism%2Fcrossgen2determinism%2Fcrossgen2determinism.sh%22 it always complains that |
My bad; I looked at the failures before merging that PR, but didn't comprehend that a few of them were real... the problem with so many tests failing is sometimes real signal gets hidden by the noise. That said, I don't know what the problem is, or rather, why it's happening. The signature changed from an argument being non-ref to ref, and apparently the code is built expecting ref but then is running against something without the ref. Is crossgen2 somehow built against live refs but run against an older corelib? |
crossgen2 it is built against live refs, but it runs against the tools runtime in some cases. I think the quick workaround is to just comment out the one place in crossgen2 where the string interpolating assert is used (or replace it with regular assert). I expect we will hit the same problem again for .NET 7 version change (the loader version sanity check will fail on 7.0 assemblies running on 6.0 runtime), so we will likely need to implement proper fix soon. |
On it... |
@trylek, yeah we will need to think about a better way for cg2 to run against older sdk. |
Run: runtime 20210817.93
Failed test:
One of error message:
Runfo Tracking Issue: readytorun/determinism/crossgen2determinism/crossgen2determinism.sh
Displaying 100 of 123 results
Build Result Summary
The text was updated successfully, but these errors were encountered: