-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Disable package aliasing tests #39144
Conversation
Attempting to fix CI failures for these tests
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.
LGTM - should we merge this to unblock all the other PRs?
/backport to release/8.0.3xx |
Started backporting to release/8.0.3xx: https://github.com/dotnet/sdk/actions/runs/8145805473 |
/backport to release/9.0.1xx-preview2 |
Started backporting to release/9.0.1xx-preview2: https://github.com/dotnet/sdk/actions/runs/8149693709 |
The tests in
GivenThatWeHaveAPackageReferenceWithAliases
have started failing somewhat consistently in CI. I haven't been able to reproduce the issue locally. The failures look something like this:This seems like it's probably a test issue, perhaps related to the fact that these tests create and then consume NuGet packages within the test. Because of this, I am going to disable these tests for now. To further investigate, we probably need to figure out how to get a binlog of the failures in Helix.