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

[Flaky Test] BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs #21005

Closed
pakrym opened this issue May 11, 2021 · 10 comments · Fixed by #21732
Closed

[Flaky Test] BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs #21005

pakrym opened this issue May 11, 2021 · 10 comments · Fixed by #21732
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault test-reliability Issue that causes tests to be unreliable

Comments

@pakrym
Copy link
Contributor

pakrym commented May 11, 2021

BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs

Result Details
Failed 2h ago on 36a07c10c000006
Duration0:00:01.070
Ownernot available
Date started5/11/2021, 11:16:16 AM
Date completed5/11/2021, 11:16:17 AM
Failing since2h ago
Failing since build20210511.13

Debug

Work items

Attachments

History

Attachments*

Error message
Expected: True
But was:  False


Stack trace
   at Azure.Core.Tests.BearerTokenAuthenticationPolicyTests.BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs() in /mnt/vss/_work/1/s/sdk/core/Azure.Core/tests/BearerTokenAuthenticationPolicyTests.cs:line 716
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   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.Execution.SimpleWorkItem.<>c__DisplayClass4_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

@pakrym pakrym added KeyVault Client This issue points to a problem in the data-plane of the library. test-reliability Issue that causes tests to be unreliable labels May 11, 2021
@pakrym
Copy link
Contributor Author

pakrym commented May 11, 2021

cc @christothes

@christothes
Copy link
Member

This should be address by PR #20336

cc @AlexanderSher

@pakrym
Copy link
Contributor Author

pakrym commented May 25, 2021

BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs

Result Details
Failed 2h ago on e2c2d09cc000002
Duration0:00:01.917
Ownernot available
Date started5/25/2021, 10:43:24 AM
Date completed5/25/2021, 10:43:26 AM
Failing since2h ago
Failing since build20210525.4

Debug

Work items

Attachments

History

Attachments*

Error message
Expected: True
But was:  False


Stack trace
   at Azure.Core.Tests.BearerTokenAuthenticationPolicyTests.BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs() in D:\a\_work\1\s\sdk\core\Azure.Core\tests\BearerTokenAuthenticationPolicyTests.cs:line 716
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   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.Execution.SimpleWorkItem.<>c__DisplayClass4_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

@pakrym
Copy link
Contributor Author

pakrym commented Jun 8, 2021

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=934951&view=ms.vss-test-web.build-test-results-tab&runId=20051536&resultId=101680&paneView=debug

Failed 5h ago on 9acdbe4cc000000
Duration0:00:02.757
Ownernot available
Date started6/8/2021, 3:54:25 AM
Date completed6/8/2021, 3:54:28 AM
Failing since5h ago
Failing since build20210608.1

Debug

Work items

Attachments

History

Attachments*

Error message
Expected: True
But was:  False


Stack trace
   at Azure.Core.Tests.BearerTokenAuthenticationPolicyTests.BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs() in D:\a\_work\1\s\sdk\core\Azure.Core\tests\BearerTokenAuthenticationPolicyTests.cs:line 716
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   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.Execution.SimpleWorkItem.<>c__DisplayClass4_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

@pakrym
Copy link
Contributor Author

pakrym commented Jun 9, 2021

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=938660&view=ms.vss-test-web.build-test-results-tab&runId=20120642&resultId=100599&paneView=debug

BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs

Result Details
Failed 4h ago on c1f943f4c000000
Duration0:00:01.120
Ownernot available
Date started6/9/2021, 3:43:40 AM
Date completed6/9/2021, 3:43:41 AM
Failing sinceYesterday
Failing since build20210608.1

Debug

Work items

Attachments

History

Attachments*

Error message
Expected: True
But was:  False


Stack trace
   at Azure.Core.Tests.BearerTokenAuthenticationPolicyTests.<BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs>d__20.MoveNext() in D:\a\_work\1\s\sdk\core\Azure.Core\tests\BearerTokenAuthenticationPolicyTests.cs:line 716
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   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.Execution.SimpleWorkItem.<>c__DisplayClass4_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

@pakrym
Copy link
Contributor Author

pakrym commented Jun 9, 2021

@christothes @schaabs can one of you disable the test please considering Alexander is out for a while?

@christothes
Copy link
Member

I'll take over his PR and merge the fix.

@pakrym
Copy link
Contributor Author

pakrym commented Jul 2, 2021

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=981633&view=logs&j=6e2e8c59-a131-5a64-1743-fd2912068bf6

BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs

Result Details
Failed 5h ago on d1b24d0cc000002
Duration0:00:01.473
Ownernot available
Date started7/2/2021, 3:43:50 AM
Date completed7/2/2021, 3:43:52 AM
Failing since5h ago
Failing since build20210702.1

Debug

Work items

Attachments

History

Attachments*

Error message
Expected: True
But was:  False


Stack trace
   at Azure.Core.Tests.BearerTokenAuthenticationPolicyTests.<BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs>d__20.MoveNext() in D:\a\_work\1\s\sdk\core\Azure.Core\tests\BearerTokenAuthenticationPolicyTests.cs:line 728
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   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.Execution.SimpleWorkItem.<>c__DisplayClass4_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

@pakrym
Copy link
Contributor Author

pakrym commented Jul 29, 2021

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1022747&view=ms.vss-test-web.build-test-results-tab&runId=22005434&resultId=102584&paneView=debug

Error message
Expected: True
But was:  False


Stack trace
   at Azure.Core.Tests.BearerTokenAuthenticationPolicyTests.<BearerTokenAuthenticationPolicy_BackgroundRefreshCancelledAndLogs>d__20.MoveNext() in D:\a\_work\1\s\sdk\core\Azure.Core\tests\BearerTokenAuthenticationPolicyTests.cs:line 728
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()
   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaiter)
   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.Execution.SimpleWorkItem.<>c__DisplayClass4_0.<PerformWork>b__0()
   at NUnit.Framework.Internal.ContextUtils.<>c__DisplayClass1_0`1.<DoIsolated>b__0(Object _)

@JoshLove-msft
Copy link
Member

Looks like this was fixed.

azure-sdk pushed a commit to azure-sdk/azure-sdk-for-net that referenced this issue Oct 5, 2022
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-net that referenced this issue Oct 6, 2022
[Hub Generated] Review request for Microsoft.MachineLearningServices to add version preview/2022-10-01-preview (Azure#20818)

* Adds base for updating Microsoft.MachineLearningServices from version preview/2022-06-01-preview to version 2022-10-01-preview

* Updates readme

* Updates API version in new specs and examples

* Registries public preview 2022-10-01-preview (Azure#20200)

* Registries public preview 2022-10-01-preview

* Fix validation errors

* Fix LintDiff and Avocado errors

* Fixing prettier errors

* Fixing more LintDiff errors

* Another LintDiff error

* Another Lint error

Co-authored-by: Fahd Kamal <fahdk@microsoft.com>

* Added schedule api (Azure#20273)

* added schedule api

* add updateSchedules to examples parent folder

* fix typo

* change position of allOf

* remove subnet from example compute create

* after prettier run

* Description change for recurrence freq

* point to mfe for common objects

* add hash operator

* removed triggertype as well

* change year

* get rid of breaking changes since we didn't change the api

Co-authored-by: Naman Agarwal <naagarw@microsoft.com>

* Added Compute Instance OS Patching Properties (Azure#20284)

* Added CI Os Patching related properties

* Updated descriptions as per Dennis's suggestions

Co-authored-by: Srivatsa Sinha <srsinha@microsoft.com>

* Maulik/sync preview (Azure#20554)

* Added changes to sync with main

* Fixed Prettier Check

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Remove update schedules REST call for compute instance (Azure#20699)

* change post call to put

* change put to patch

* remove updateschedules rest call

Co-authored-by: Naman Agarwal <naagarw@microsoft.com>

* Update connection type and auth type (Azure#20493)

* Update connection type and auth type

* update connection category

* update x-ms-secret tag

* Add autologger_settings for CI (Azure#20302)

* Add autologger_settings for CI

Add settings to opt-in/out of default mlflow autologger on CIs.

* update machinelearningservices.json

* Add for command job

* fix comma

* Add autologger for command job

* fix CI failures

* Remove mfe.json from PR

* Remove whitespace

* Update custom-words.txt

* Delete incorrect custom-words.txt

* Fixing auto-rest issues in registries (Azure#20772)

* Fixing auto-rest issues in registries

* Removing location from the required list

Co-authored-by: Fahd Kamal <fahdk@microsoft.com>

* Maulik/mfe preview changes (Azure#20372)

* Added mfe.json preview changes

* Added resources to Registries and Workspaces folder

* removed unwanted resources

* removed unwanted resources

* Ran prettier check

* removed unwanted resources

* removed unwanted resources

* Added ManagedOnlineDeployment resource

* Added files to OnlineDeplyment

* Added Model Validation changes

* Added mfe.json changes to handle duplicate operation id changes

* Updated mfe to fix underscore error

* Update title of mfe.json

* Renamed name path parameter

* Updated examples

* Fixed Semantic validations and examples

* Fixed Semantic validations and examples

* Fixed Semantic validations and examples

* Removed name changes from workspaces

* Changes related to Model Validation

* Changes related to Model validation

* Added consumes amd produces in registries.json

* Removed changes from registries

* Updated parameter mlFlowAutologger to mlflowAutologger in mfe.json

* Changed property name to mlflowAutologger

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Updated custom words to resolve merge conflict (Azure#20833)

* Updated custom words to resolve merge conflict

* Updated custom words to resolve merge conflict

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Resolved merge conflict (Azure#20845)

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Fixed LintDiff checks (Azure#20847)

* Fixed LintDiff checks

* Fixed LintDiff checks

* Fixed LintDiff checks

* Fixed LintDiff checks

* Updated examples

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Resolved merge conflict of cutsom-words.txt (Azure#20956)

* Resolved merge conflict

* Resolved merge conflict

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* [Hub Generated] Review request for Microsoft.MachineLearningServices to add version stable/2022-10-01 (Azure#20800) (Azure#20982)

* Adds base for updating Microsoft.MachineLearningServices from version stable/2022-05-01 to version 2022-10-01

* Updates readme

* Updates API version in new specs and examples

* Added readOnly property (Azure#20506)

* Added readOnly property

* Added changes to sync with main

* Fixed Prettier check

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Add schedule related schemas for Compute Instance (Azure#20697)

* support for update schedules on Compute Instance

* change put to patch

* remove update schedules rest call

Co-authored-by: Naman Agarwal <naagarw@microsoft.com>

* Added mfe.json changes (Azure#20514)

* Added mfe.json changes

* Added checks related changes

* Updated mfe.json to handle online endpoint and deployment changes

* Updated mfe.json with AutoRebuild changes

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Removed duplicate definitions and updated with mfe references (Azure#20840)

* Removed duplicate definitions and updated with mfe references

* Replaced Cron to CronTrigger, Recurrence to RecurrenceTrigger from mfe

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>
Co-authored-by: Naman Agarwal <namanag16@gmail.com>
Co-authored-by: Naman Agarwal <naagarw@microsoft.com>

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>
Co-authored-by: Naman Agarwal <namanag16@gmail.com>
Co-authored-by: Naman Agarwal <naagarw@microsoft.com>

* Revert "[Hub Generated] Review request for Microsoft.MachineLearningS… (Azure#20983)

* Revert "[Hub Generated] Review request for Microsoft.MachineLearningServices to add version stable/2022-10-01 (Azure#20800) (Azure#20982)"

This reverts commit 649ef8334aca13558e534babdcf53cef6831f22a.

* Updated readme.md to resolve merge conflict

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* Maulik/fix preview (Azure#20994)

* Merged readme from main

* Added preview tag changes

Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>

* shuffle readme tags (Azure#21005)

* conflict resolution (Azure#21007)

Co-authored-by: fahdkmsft <51756016+fahdkmsft@users.noreply.github.com>
Co-authored-by: Fahd Kamal <fahdk@microsoft.com>
Co-authored-by: Naman Agarwal <namanag16@gmail.com>
Co-authored-by: Naman Agarwal <naagarw@microsoft.com>
Co-authored-by: srivatsasinha <102133347+srivatsasinha@users.noreply.github.com>
Co-authored-by: Srivatsa Sinha <srsinha@microsoft.com>
Co-authored-by: maulikjoshi <maulikjoshi@microsoft.com>
Co-authored-by: ZhidaLiu <zhili@microsoft.com>
Co-authored-by: prakharg-msft <75808410+prakharg-msft@users.noreply.github.com>
Co-authored-by: Teddy Todorov <thtodoro@microsoft.com>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault test-reliability Issue that causes tests to be unreliable
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants