Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.47k stars 4.8k forks source link

[Synapse] CanCheckPrincipalAccessViaGrowUpHelper failing in nightly runs #30737

Closed v-jiaodi closed 2 years ago

v-jiaodi commented 2 years ago

Synapse nightly test runs are failing with:

Error message: Expected string length 7 but was 10. Strings differ at index 0. Expected: "Allowed" But was: "NotAllowed" -----------^ Stack trace: at Azure.Analytics.Synapse.AccessControl.Tests.AccessControlClientLiveTests.CanCheckPrincipalAccessViaGrowUpHelper() in D:\a_work\1\s\sdk\synapse\Azure.Analytics.Synapse.AccessControl\tests\AccessControlClientLiveTests.cs:line 295 at Azure.Analytics.Synapse.AccessControl.Tests.AccessControlClientLiveTests.CanCheckPrincipalAccessViaGrowUpHelper() in D:\a_work\1\s\sdk\synapse\Azure.Analytics.Synapse.AccessControl\tests\AccessControlClientLiveTests.cs:line 300 at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter1.GetResult() at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func1 invoke) at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context) at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context) at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.<>c__DisplayClass1_0.b__0() at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.RunTestMethodInThreadAbortSafeZone(TestExecutionContext context, Action action)

For more details check here:

@jsquire for notification.

azure-sdk commented 2 years ago

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Synapse:0.5096098,Azure.Identity:0.20703286,Network - Front Door:0.052900776'

ghost commented 2 years ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @wonner, @zesluo.

Issue Details
Synapse nightly test runs are failing with: > Error message: >Expected string length 7 but was 10. Strings differ at index 0. Expected: "Allowed" But was: "NotAllowed" -----------^ > Stack trace: > at Azure.Analytics.Synapse.AccessControl.Tests.AccessControlClientLiveTests.CanCheckPrincipalAccessViaGrowUpHelper() in D:\a\_work\1\s\sdk\synapse\Azure.Analytics.Synapse.AccessControl\tests\AccessControlClientLiveTests.cs:line 295 at Azure.Analytics.Synapse.AccessControl.Tests.AccessControlClientLiveTests.CanCheckPrincipalAccessViaGrowUpHelper() in D:\a\_work\1\s\sdk\synapse\Azure.Analytics.Synapse.AccessControl\tests\AccessControlClientLiveTests.cs:line 300 at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.GetResult() at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func`1 invoke) at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context) at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context) at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.<>c__DisplayClass1_0.b__0() at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.RunTestMethodInThreadAbortSafeZone(TestExecutionContext context, Action action) For more details check here: - https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1807437&view=results @jsquire for notification.
Author: v-jiaodi
Assignees: -
Labels: `Service Attention`, `Client`, `Synapse`, `test-reliability`
Milestone: -
yanjungao718 commented 2 years ago

This is not reproduced and all tests passed when investigating, it is likely to be an unstable problem, so now close it and please feel free to open a new one if repro again.