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

Need to set DOTNET_CLI_DO_NOT_USE_MSBUILD_SERVER=true on linux builds #11369

Closed
2 tasks
MattGal opened this issue Oct 21, 2022 · 5 comments
Closed
2 tasks

Need to set DOTNET_CLI_DO_NOT_USE_MSBUILD_SERVER=true on linux builds #11369

MattGal opened this issue Oct 21, 2022 · 5 comments

Comments

@MattGal
Copy link
Member

MattGal commented Oct 21, 2022

Build

https://dev.azure.com/dnceng-public/public/_build/results?buildId=59411&view=results

Build leg reported

Android_Simulators Build_Release

Pull Request

#11364

Action required for the engineering services team

To triage this issue (First Responder / @dotnet/dnceng):

  • Open the failing build above and investigate
  • Add a comment explaining your findings

If this is an issue that is causing build breaks across multiple builds and would get benefit from being listed on the build analysis check, follow the next steps:

  1. Add the label "Known Build Error"
  2. Edit this issue and add an error string in the Json below that can help us match this issue with future build breaks. You should use the known issues documentation
{
   "ErrorMessage" : "Unhandled exception: System.IO.IOException: Connection timed out : 'Global\\msbuild-server-launch-",
   "BuildRetry": false
}

Additional information about the issue reported

This is a known issue with .NET Core MSBuild in RC1 and RC2 and impacts any repo using this for build in linux

Report

Build Definition Step Name Console log Pull Request
63280 dotnet/roslyn Build with analyzers Log dotnet/roslyn#64983
63149 dotnet/roslyn Build with analyzers Log dotnet/roslyn#64977
62842 dotnet/roslyn Build with analyzers Log
61673 dotnet/roslyn Restore Log
61490 dotnet/roslyn Restore Log
61357 dotnet/roslyn Build with analyzers Log dotnet/roslyn#64936
61126 dotnet/arcade XHarness Android Helix Testing (Linux) Log
60694 dotnet/roslyn Restore Log
59926 dotnet/roslyn Restore Log dotnet/roslyn#64893
59850 dotnet/arcade Run Helix Tests Log #11374
59697 dotnet/roslyn Restore Log dotnet/roslyn#64890
59629 dotnet/arcade XHarness Apple Simulator Helix Testing Log
59526 dotnet/roslyn Restore Log dotnet/roslyn#64741
59170 dotnet/arcade XHarness Apple Simulator Helix Testing Log #11359
59122 dotnet/arcade XHarness Apple Simulator Helix Testing Log #11359
58747 dotnet/roslyn Restore Log dotnet/roslyn#64879
58707 dotnet/roslyn Restore Log dotnet/roslyn#64741

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
3 17 17
@MattGal
Copy link
Member Author

MattGal commented Oct 21, 2022

#11371 should prevent this for arcade and anyone else using build.sh. Roslyn does its own thing so not much we can do about that directly.

@MattGal
Copy link
Member Author

MattGal commented Oct 24, 2022

@ilyas1974 I am tossing this into "tracking/blocked" since it should be fixed for arcade and users of tools.sh but Roslyn (top hitter) doesn't use this. It should just go away at some point when newer .NET Core SDKs improve the functionality of the MSbuild server

@MattGal
Copy link
Member Author

MattGal commented Oct 26, 2022

Discussed with @ilyas1974 ; since we have added the workaround in arcade and the only remaining hitter is using a custom build entry point, I am closing this issue as complete. I reached out to some contacts on the Roslyn side to encourage workaround.

@MattGal MattGal closed this as completed Oct 26, 2022
@dougbu
Copy link
Member

dougbu commented Oct 27, 2022

@agocke mentioned in CI Council that server was disabled by default in 7.0.1xx SDK versions. Not sure of the exact version where that changed nor whether it's also in an available 8.0.1xx SDK. Is updating the SDK versions we use in release/7.0 and main a viable workaround now @rainersigwald @dsplaisted

If release/7.0 is still likely to hit this, suggest backporting the DOTNET_CLI_DO_NOT_USE_MSBUILD_SERVER to that Arcade branch (if you haven't already) @MattGal

Separately, note this can impact integration and e2e testing on Helix agents. In aspnetcore, we added the environment variable in our test driver. But maybe it should be included in the Python code that does "stuff" for Helix work items❔

/cc @Chrisboh @ilyas1974 @steveisok @carlossanlop

@rainersigwald
Copy link
Member

MSBuild Server is off by default in the GA build of 7.0.100 and 8.0 dailies, though there might have been a few 8.0 builds before that became true. If you're within the last week you should be fine. Updating to 7.0.100 GA should be viable after it releases but I don't think it's usable today because of packages.

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

No branches or pull requests

4 participants