Skip to content

Require RemoteExecutorSupported on color env test#126191

Merged
svick merged 2 commits intodotnet:mainfrom
am11:patch-51
Mar 27, 2026
Merged

Require RemoteExecutorSupported on color env test#126191
svick merged 2 commits intodotnet:mainfrom
am11:patch-51

Conversation

@am11
Copy link
Copy Markdown
Member

@am11 am11 commented Mar 27, 2026

It was failing on NativeAOT legs:

[SKIP] Microsoft.Extensions.Logging.Console.Test.ConsoleLoggerTest.AddConsole_IsOutputRedirected_ColorDisabled
[FAIL] Microsoft.Extensions.Logging.Console.Test.ConsoleLoggerTest.DoesConsoleSupportAnsi_RespectsColorEnvVars(envVarName1: "FORCE_COLOR", envVarValue1: "1", envVarName2: null, envVarValue2: null, expectAnsiEscapes: True)
System.PlatformNotSupportedException : RemoteExecutor is not supported on this platform.
   at Microsoft.DotNet.RemoteExecutor.RemoteExecutor.Invoke(MethodInfo method, String[] args, RemoteInvokeOptions options, Boolean pasteArguments) in /_/src/arcade/src/Microsoft.DotNet.RemoteExecutor/src/RemoteExecutor.cs:line 415
   at Microsoft.Extensions.Logging.Console.Test.ConsoleLoggerTest.DoesConsoleSupportAnsi_RespectsColorEnvVars(String envVarName1, String envVarValue1, String envVarName2, String envVarValue2, Boolean expectAnsiEscapes) in /_/src/libraries/Microsoft.Extensions.Logging.Console/tests/Microsoft.Extensions.Logging.Console.Tests/ConsoleLoggerTest.cs:line 468
   at System.Reflection.DynamicInvokeInfo.InvokeWithManyArguments(IntPtr, Byte&, Byte&, Object[], BinderBundle, Boolean) in /_/src/coreclr/nativeaot/System.Private.CoreLib/src/System/Reflection/DynamicInvokeInfo.cs:line 426
[FAIL] Microsoft.Extensions.Logging.Console.Test.ConsoleLoggerTest.DoesConsoleSupportAnsi_RespectsColorEnvVars(envVarName1: "DOTNET_SYSTEM_CONSOLE_ALLOW_ANSI_COLOR_REDIRECTION", envVarValue1: "1", envVarName2: null, envVarValue2: null, expectAnsiEscapes: True)
System.PlatformNotSupportedException : RemoteExecutor is not supported on this platform.

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Mar 27, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/area-extensions-logging
See info in area-owners.md if you want to be subscribed.

@svick svick self-assigned this Mar 27, 2026
@svick svick enabled auto-merge (squash) March 27, 2026 11:57
Copy link
Copy Markdown
Member

@mrek-msft mrek-msft left a comment

Choose a reason for hiding this comment

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

LGTM. Seems like regression came from recently merged 3b8cdd5

@jkotas
Copy link
Copy Markdown
Member

jkotas commented Mar 27, 2026

/ba-g timeouts

@svick svick merged commit 63aff8d into dotnet:main Mar 27, 2026
82 of 90 checks passed
@am11 am11 deleted the patch-51 branch March 27, 2026 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-Extensions-Logging community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants