Skip to content

Investigate test failures with "An operation is not legal in the current state." #16817

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

Closed
wtgodbe opened this issue Nov 4, 2019 · 5 comments
Assignees
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework test-failure
Milestone

Comments

@wtgodbe
Copy link
Member

wtgodbe commented Nov 4, 2019

Starting with #14425, we've been seeing lots of test failures of the following type:

System.IO.FileLoadException : Could not load file or assembly 'Microsoft.Net.Http.Headers, Version=3.0.0.0, Culture=neutral, PublicKeyToken=adb9793829ddae60'. An operation is not legal in the current state. (0x80131509)

We think this is related to the large number of Explicit references that we've added to the tests. We should investigate these failures as they may represent an issue with the runtime.

CC @aspnet/build @mmitche

@wtgodbe wtgodbe added test-failure area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework labels Nov 4, 2019
@wtgodbe wtgodbe added this to the 3.0.1 milestone Nov 4, 2019
@dougbu
Copy link
Contributor

dougbu commented Nov 4, 2019

Would like to first determine whether this is about flakiness in our eng\targets\ResolveReferences.targets versus in the .NET SDK. To that end, the initial task here might be to create a project with lots and lots of explicit <ProjectReference>s and <PackageReference>s but none of the AspNetCore <Reference> infrastructure, then build it over and over.

@mkArtakMSFT
Copy link
Contributor

@wtgodbe this should move to a proper milestone. I think this milestone is already locked down, so we should close it.

@wtgodbe wtgodbe modified the milestones: 3.0.1, 3.0.2 Dec 6, 2019
@wtgodbe
Copy link
Member Author

wtgodbe commented Dec 6, 2019

We think this will be fixed by #17311 - if so I'll close when that's merged

@dougbu
Copy link
Contributor

dougbu commented Dec 13, 2019

Close once merge into 'release/3.1' is done

@wtgodbe
Copy link
Member Author

wtgodbe commented Dec 16, 2019

We stopped seeing these after #17311, closing

@wtgodbe wtgodbe closed this as completed Dec 16, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Jan 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework test-failure
Projects
None yet
Development

No branches or pull requests

3 participants