Skip to content
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

17.6.x consumes lot of CPU #4553

Closed
nohwnd opened this issue Jun 15, 2023 · 2 comments · Fixed by #4556
Closed

17.6.x consumes lot of CPU #4553

nohwnd opened this issue Jun 15, 2023 · 2 comments · Fixed by #4556

Comments

@nohwnd
Copy link
Member

nohwnd commented Jun 15, 2023

When multiple test projects are run in sequence, vstest.console can consume a lot of CPU because it is constantly polling for data from testhost that is started but not executing yet.

This was mitigated by disabling testhost pre-start in the linked PR and inserted into VS 17.6.4. This regression was introduced in 17.6.0.

@cremor
Copy link

cremor commented Jun 21, 2023

I just found this entry in the Visual Studio 17.6.4 release notes:

When multiple test projects are run in sequence, vstest.console can consume a lot of CPU because it is constantly polling for data from testhost that is started but not executing yet. This was mitigated by disabling testhost pre-start. This regression was introduced in 17.6.0.

I assume that means this issue?
If yes, does that mean the issue was fixed with a Visual Studio update? So we don't have to wait for an eventually upcoming version 17.6.3 of the NuGet package Microsoft.NET.Test.Sdk?

@nohwnd
Copy link
Member Author

nohwnd commented Jun 26, 2023

@cremor Yes. This issue is impacting vstest.console and it's consumers (such as dotnet test), we've inserted into VS and dotnet SDK, but did not yet release 17.6.3 nuget package, which would fix users that are using VS platform Installer task.

This issue has no impact on Microsoft.NET.Test.Sdk, because that package is used to ship testhost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants