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.

Sign up for GitHub

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

Jump to bottom

[Key Vault] Add at least one performance test for administration #17169

Closed
mccoyp opened this issue Mar 8, 2021 · 2 comments
Closed

[Key Vault] Add at least one performance test for administration #17169

mccoyp opened this issue Mar 8, 2021 · 2 comments
Assignees
@YalinLi0312
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault MQ This issue is part of a "milestone of quality" initiative.
Milestone
[2022] March

Comments

@mccoyp
Copy link
Member

mccoyp commented Mar 8, 2021

Testing instructions can be found here, and performance tests for already-GA'ed libraries can be found here.
@mccoyp mccoyp added KeyVault Client This issue points to a problem in the data-plane of the library. test enhancement labels Mar 8, 2021
@mccoyp mccoyp added this to the [2021] April milestone Mar 8, 2021
@mccoyp mccoyp self-assigned this Mar 8, 2021
@mccoyp mccoyp modified the milestones: [2021] April, [2021] May Apr 2, 2021
@mccoyp mccoyp modified the milestones: [2021] May, [2021] June May 6, 2021
@mccoyp mccoyp closed this as completed May 27, 2021
@mccoyp mccoyp reopened this May 28, 2021
@mccoyp mccoyp modified the milestones: [2021] June, [2021] July Jun 18, 2021
@mccoyp mccoyp modified the milestones: [2021] July, [2021] August Jul 7, 2021
@mccoyp mccoyp modified the milestones: [2021] August, [2021] September Aug 3, 2021
@YalinLi0312 YalinLi0312 assigned YalinLi0312 and unassigned mccoyp Oct 4, 2021
@mccoyp
Copy link
Member Author

mccoyp commented Oct 5, 2021

Since testing the performance of long-running operations isn't very useful, we can probably focus on testing just the KeyVaultAccessControlClient. We can coordinate with the rest of the team to focus on the most relevant operations

@lmazuel lmazuel added the MQ This issue is part of a "milestone of quality" initiative. label Oct 7, 2021
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this issue Jan 12, 2022
CodeGen from PR 17169 in Azure/azure-rest-api-specs
1d8642b 
Add swagger definitions for deletedAutomationAccounts resource type (Azure#17169)

* add swagger definitions for deletedAutomationAccounts resource type

* fix semantic and model validation error

* fix model validation error

* fix date format and schema type

* fix lint diff error

* fix paths

* fix lint diff

* remove errorresponse schema

* add errorresponse schema

* add reference in readme

* supress error schema

* fix URI path to return list of deleted accounts

* remove automation account name parameter

Co-authored-by: Ravi Yadav <[email protected]>
@mccoyp mccoyp modified the milestones: [2022] February, [2022] March Feb 15, 2022
@YalinLi0312 YalinLi0312 mentioned this issue Mar 3, 2022
Merged
@mccoyp
Copy link
Member Author

mccoyp commented Mar 10, 2022

Resolved by #23303

@mccoyp mccoyp closed this as completed Mar 10, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@YalinLi0312 YalinLi0312

Labels
Client This issue points to a problem in the data-plane of the library. KeyVault MQ This issue is part of a "milestone of quality" initiative.
Projects
None yet
Milestone
[2022] March
Development

No branches or pull requests
3 participants
@lmazuel @YalinLi0312 @mccoyp