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

System.Net.Security.Tests.NegotiateAuthenticationKerberosTest asserts on Android #74708

Closed
karelz opened this issue Aug 27, 2022 · 7 comments · Fixed by #74709
Closed

System.Net.Security.Tests.NegotiateAuthenticationKerberosTest asserts on Android #74708

karelz opened this issue Aug 27, 2022 · 7 comments · Fixed by #74709

Comments

@karelz
Copy link
Member

karelz commented Aug 27, 2022

Tests affected:

  • Loopback_Success - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Loopback_Success
  • Invalid_Token - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Invalid_Token

Failure frequency - results with "Test Results" link in last 30 days - as of 8/27:

Day Run Details
8/27 Rolling run 1971504 net7.0-Linux-Release-arm64-Mono_Release_LinuxBionic-Windows.10.Amd64.Android.Open
8/26 Rolling run 1970381 net7.0-Linux-Release-arm64-Mono_Release_LinuxBionic-Windows.10.Amd64.Android.Open
  • Loopback_Success - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Loopback_Success
Microsoft.DotNet.RemoteExecutor.RemoteExecutionException : Remote process failed with an unhandled exception.

Child exception:
  Xunit.Sdk.TrueException: Client authentication failed with Unsupported
Expected: True
Actual:   False
   at System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.<>c.<Loopback_Success>b__2_0() in /_/src/libraries/System.Net.Security/tests/FunctionalTests/NegotiateAuthenticationKerberosTest.cs:line 48
   at System.Reflection.MethodInvoker.InterpretedInvoke(Object obj, Span`1 args, BindingFlags invokeAttr) in /_/src/mono/System.Private.CoreLib/src/System/Reflection/MethodInvoker.Mono.cs:line 33

Child process:
  System.Net.Security.Tests, Version=7.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 System.Net.Security.Tests.NegotiateAuthenticationKerberosTest+<>c Void <Loopback_Success>b__2_0()
  • Invalid_Token - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Invalid_Token
Microsoft.DotNet.RemoteExecutor.RemoteExecutionException : Remote process failed with an unhandled exception.

Child exception:
  Xunit.Sdk.EqualException: Assert.Equal() Failure
Expected: ContinueNeeded
Actual:   GenericFailure
   at System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.<>c.<Invalid_Token>b__3_0() in /_/src/libraries/System.Net.Security/tests/FunctionalTests/NegotiateAuthenticationKerberosTest.cs:line 75
   at System.Reflection.MethodInvoker.InterpretedInvoke(Object obj, Span`1 args, BindingFlags invokeAttr) in /_/src/mono/System.Private.CoreLib/src/System/Reflection/MethodInvoker.Mono.cs:line 33

Child process:
  System.Net.Security.Tests, Version=7.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 System.Net.Security.Tests.NegotiateAuthenticationKerberosTest+<>c Void <Invalid_Token>b__3_0()

cc @filipnavara

Likely related to #73343 and #74224

Report

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 0 0
@karelz karelz added area-System.Net.Security blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' os-android labels Aug 27, 2022
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Aug 27, 2022
@ghost
Copy link

ghost commented Aug 27, 2022

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

Issue Details

Tests affected:

  • Loopback_Success - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Loopback_Success
  • Invalid_Token - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Invalid_Token

Failure frequency - results with "Test Results" link in last 30 days - as of 8/27:

Day Run Details
8/27 Rolling run 1971504 net7.0-Linux-Release-arm64-Mono_Release_LinuxBionic-Windows.10.Amd64.Android.Open
8/26 Rolling run 1970381 net7.0-Linux-Release-arm64-Mono_Release_LinuxBionic-Windows.10.Amd64.Android.Open
  • Loopback_Success - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Loopback_Success
Microsoft.DotNet.RemoteExecutor.RemoteExecutionException : Remote process failed with an unhandled exception.

Child exception:
  Xunit.Sdk.TrueException: Client authentication failed with Unsupported
Expected: True
Actual:   False
   at System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.<>c.<Loopback_Success>b__2_0() in /_/src/libraries/System.Net.Security/tests/FunctionalTests/NegotiateAuthenticationKerberosTest.cs:line 48
   at System.Reflection.MethodInvoker.InterpretedInvoke(Object obj, Span`1 args, BindingFlags invokeAttr) in /_/src/mono/System.Private.CoreLib/src/System/Reflection/MethodInvoker.Mono.cs:line 33

Child process:
  System.Net.Security.Tests, Version=7.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 System.Net.Security.Tests.NegotiateAuthenticationKerberosTest+<>c Void <Loopback_Success>b__2_0()
  • Invalid_Token - System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.Invalid_Token
Microsoft.DotNet.RemoteExecutor.RemoteExecutionException : Remote process failed with an unhandled exception.

Child exception:
  Xunit.Sdk.EqualException: Assert.Equal() Failure
Expected: ContinueNeeded
Actual:   GenericFailure
   at System.Net.Security.Tests.NegotiateAuthenticationKerberosTest.<>c.<Invalid_Token>b__3_0() in /_/src/libraries/System.Net.Security/tests/FunctionalTests/NegotiateAuthenticationKerberosTest.cs:line 75
   at System.Reflection.MethodInvoker.InterpretedInvoke(Object obj, Span`1 args, BindingFlags invokeAttr) in /_/src/mono/System.Private.CoreLib/src/System/Reflection/MethodInvoker.Mono.cs:line 33

Child process:
  System.Net.Security.Tests, Version=7.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 System.Net.Security.Tests.NegotiateAuthenticationKerberosTest+<>c Void <Invalid_Token>b__3_0()

cc @filipnavara

Likely related to #73343 and #74224

Author: karelz
Assignees: -
Labels:

area-System.Net.Security, blocking-clean-ci, os-android

Milestone: -

@filipnavara
Copy link
Member

This is same as #74224. As stated in #74225 (comment) the code is guarded not to execute on Android. We need to figure out what the h*ck is this pipeline and why that check doesn't correctly trigger.

@filipnavara
Copy link
Member

filipnavara commented Aug 27, 2022

@akoeplinger Got any idea what run is that and why would OperatingSystem.IsLinux() return true for Android? (or does it get incorrectly substituted by linker?)

@filipnavara
Copy link
Member

Hmm, apparently the Linux Bionic pipeline is something weird that I didn't know...and it's not Android. I can add extra check much like https://github.com/dotnet/runtime/pull/72256/files did.

@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Aug 27, 2022
@karelz
Copy link
Member Author

karelz commented Aug 28, 2022

Thanks for figuring it out @filipnavara!

@karelz karelz added this to the 8.0.0 milestone Aug 28, 2022
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Aug 28, 2022
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Aug 29, 2022
@karelz
Copy link
Member Author

karelz commented Sep 3, 2022

Reopening for backport to 7.0 -- it was hit there as well in #75008

@karelz karelz reopened this Sep 3, 2022
@ghost ghost added in-pr There is an active PR which will close this issue when it is merged and removed in-pr There is an active PR which will close this issue when it is merged labels Sep 3, 2022
@steveisok steveisok removed the blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' label Sep 7, 2022
@wfurt
Copy link
Member

wfurt commented Nov 9, 2022

7.0 is fixed by #75047

@wfurt wfurt closed this as completed Nov 9, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants