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

Disable "MSBuild Server v1" by default #18526

Merged

Conversation

marcpopMSFT
Copy link
Member

@marcpopMSFT marcpopMSFT commented Jun 24, 2021

Description

Disabling the new MS Build server feature as it's not ready yet. Leaving the feature flag in place so we can continue to test it.

Prior change to enable this two weeks ago
#18263

Customer Impact

We've found multiple impacts from early dogfooding so far. The primary one below is that MSBuild can get into a state where it fails to expand wildcards, failing builds. Because of how confusing that was to track down and how unsure we are on other impact, we want to revert this to be safe.

dotnet/msbuild#6602

Another impact was we were blocked from updating stage 0 SDK in the installer repo so this could have a lot of other silent internal impact we haven't tracked down yet:
dotnet/installer#10940

Testing

Passed SDK tests.

Risk

Low, this flips a feature flag back off that was enabled only two weeks ago.

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

marcpopMSFT and others added 2 commits June 24, 2021 14:26
…Logging.cs

Co-authored-by: Rainer Sigwald <raines@microsoft.com>
…Logging.cs

Co-authored-by: Rainer Sigwald <raines@microsoft.com>
@@ -16,7 +16,7 @@ namespace Microsoft.DotNet.Cli.Utils
internal class MSBuildForwardingAppWithoutLogging
{
private static readonly bool AlwaysExecuteMSBuildOutOfProc = Env.GetEnvironmentVariableAsBool("DOTNET_CLI_RUN_MSBUILD_OUTOFPROC");
private static readonly bool DoNotUseMSBUILDNOINPROCNODE = Env.GetEnvironmentVariableAsBool("DOTNET_CLI_DO_NOT_USE_MSBUILDNOINPROCNODE");
private static readonly bool UseMSBUILDNOINPROCNODE = Env.GetEnvironmentVariableAsBool("DOTNET_CLI_USE_MSBUILDNOINPROCNODE");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this because later we can just change the default value to true.

@rainersigwald rainersigwald changed the title Update MSBuildForwardingAppWithoutLogging.cs Disable "MSBuild Server v1" by default Jun 25, 2021
@marcpopMSFT marcpopMSFT merged commit cb63644 into release/6.0.1xx-preview6 Jun 25, 2021
@rainersigwald rainersigwald deleted the marcpopMSFT-revertmsbuildnoinprocnode branch July 16, 2021 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants