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

Keyvault Sovereign test failure #17829

Closed
tzhanl opened this issue Jan 7, 2021 · 9 comments
Closed

Keyvault Sovereign test failure #17829

tzhanl opened this issue Jan 7, 2021 · 9 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault
Milestone

Comments

@tzhanl
Copy link
Contributor

tzhanl commented Jan 7, 2021

use this issue to track keyvault sovereign test failures.

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=677066&view=results

@danieljurek @benbp @jongio

@tzhanl tzhanl added KeyVault test-sovereign-cloud Client This issue points to a problem in the data-plane of the library. labels Jan 7, 2021
@jsquire jsquire added the needs-team-triage Workflow: This issue needs the team to triage. label Jan 7, 2021
@heaths
Copy link
Member

heaths commented Jan 8, 2021

This requires infrastructure changes which @benbp is working on.

@heaths heaths assigned benbp and unassigned heaths Jan 8, 2021
@tzhanl
Copy link
Contributor Author

tzhanl commented Feb 22, 2021

test VerifyCancelCertificateOperation always fail in all job in china cloud

@benbp
Copy link
Member

benbp commented Feb 22, 2021

The build has been deleted so I'm not sure what this issue is about. @tzhanl can you be more descriptive in the issue description and title?

@heaths
Copy link
Member

heaths commented Feb 22, 2021

That test frequently fails because of timing issues. I've made several changes already to try to combat it, but it's especially problematic. @tzhanl are you saying it's still failing even recently? Changes I have made see this fail much less in AzureCloud, at least.

@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Feb 22, 2021
@tzhanl
Copy link
Contributor Author

tzhanl commented Feb 23, 2021

@heaths it failed in 2/21: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=745075&view=results

all Jobs in China Cloud failed:

2021-02-22T02:14:08.9916889Z Failed VerifyCancelCertificateOperation [4 s]
2021-02-22T02:14:08.9918040Z Error Message:
2021-02-22T02:14:08.9919053Z Expected exception OperationCanceledException not thrown. Operation status: inProgress, error:
2021-02-22T02:14:08.9920121Z Expected: <System.OperationCanceledException>
2021-02-22T02:14:08.9920903Z But was: null
2021-02-22T02:14:08.9921239Z
2021-02-22T02:14:08.9921841Z Stack Trace:
2021-02-22T02:14:08.9923060Z at Azure.Security.KeyVault.Certificates.Tests.CertificateClientLiveTests.VerifyCancelCertificateOperation() in /mnt/vss/_work/1/s/sdk/keyvault/Azure.Security.KeyVault.Certificates/tests/CertificateClientLiveTests.cs:line 109

@heaths heaths removed Central-EngSys This issue is owned by the Engineering System team. needs-team-triage Workflow: This issue needs the team to triage. labels Feb 23, 2021
@heaths heaths assigned heaths and unassigned benbp Feb 23, 2021
@heaths heaths added this to the [2021] March milestone Feb 23, 2021
@tzhanl
Copy link
Contributor Author

tzhanl commented Mar 4, 2021

@heaths
Copy link
Member

heaths commented Mar 4, 2021

Seems Key Vault is taking longer than expected to cancel the operation; however, this is not a priority right now. I'll resolve this as soon as possible.

@heaths
Copy link
Member

heaths commented Mar 11, 2021

A similar failure on mac, but not necessarily sovereign-specific:

Error message
System.Threading.Tasks.TaskCanceledException : A task was canceled.


Stack trace
   at Azure.Core.OperationHelpers.DefaultWaitForCompletionAsync[TResult](Operation`1 operation, TimeSpan pollingInterval, CancellationToken cancellationToken) in /_/sdk/core/Azure.Core/src/Shared/OperationHelpers.cs:line 53
   at System.Threading.Tasks.ValueTask`1.get_Result()

@jsquire jsquire modified the milestones: [2021] March, Backlog Mar 16, 2021
@heaths heaths modified the milestones: Backlog, [2021] April Mar 17, 2021
@heaths
Copy link
Member

heaths commented Mar 17, 2021

Duplicate of #17718

@heaths heaths marked this as a duplicate of #17718 Mar 17, 2021
@heaths heaths closed this as completed Mar 17, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 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
Projects
None yet
Development

No branches or pull requests

4 participants