-
Notifications
You must be signed in to change notification settings - Fork 323
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
.NET Framework test fails after updating to .NET 8 preview 5 SDK #4549
Comments
Trying. |
Ok so this seems to be running net472 tests on Linux. An they pass when 17.6.0-preview-20230223-05 is used, but not when 17.7.0-preview-23281-03+4ce2ff1f8 is used. |
Is this scenario - .NET Framework tests running on Linux - a supported scenario, or was it an accident that it worked? Do you know if this bug be fixed before .NET 8 is released? Right now, it is breaking this PR: grpc/grpc-dotnet#2021. I could disable the net472 tests on the build server to unblock. The disable could be temporary or permanent based on whether this will be fixed. |
Still trying to figure this out. |
We are constructing the .CLI package that we insert into dotnet sdk differently so we end up with a slightly newer version of System.Diagnostics.TraceSource.dll in the package than what we build with. This causes no issues on windows, but on Mono it matters. |
Seeing what looks to be the same issue but with .NET 7.0.400 SDK installed, and running tests that target .NET Framework 4.6.1 and 4.7.2, on macOS with VS Mac, or from the command line with dotnet test. Seems OK with .NET 7.0.307 SDK installed. Would this also affect .NET 7.0.400? |
Yes, 7.0.400 ships with 17.7 which has that bug, is that a big problem for you? We did not want to backport the change last minute because on Windows it works and there is a single report (now 2) of the issue. |
backported in #4632 Inserted into sdk in dotnet/sdk#34383 Not sure when it will land into dotnet/installer, the last insertion I see there is from 2 weeks ago, probably because of branch lockdown dotnet/installer#16952 hopefully we see release/7.0.4xx update from dotnet/sdk in the PRs soon. |
Temporary workaround for microsoft/vstest#4549.
Temporary workaround for microsoft/vstest#4549.
Updated workaround for microsoft/vstest#4549
Workaround for the following issues with the Microsoft.TestPlatform.ObjectModel package: - microsoft/vstest#4549 - microsoft/vstest#4671 - microsoft/vstest#4627 Resolves: No entry
Revert workaround for microsoft/vstest#4549.
Revert workaround for microsoft/vstest#4549
Description
When I updated my repository to use .NET 8 preview 5 I got an error when running tests that target the .NET Framework.
Steps to reproduce
I have a PR that updates the repo to .NET 8: grpc/grpc-dotnet#2021
47ae763
(#2021)f903df4
(#2021)Expected behavior
Tests pass.
Actual behavior
I see this error in logs:
Diagnostic logs
Attached: net8p5-error.txt
Environment
The text was updated successfully, but these errors were encountered: