Skip to content

{Graph} Rerun keyvault tests related with az ad/role#22188

Merged
jiasli merged 1 commit intoAzure:betafrom
evelyn-ys:keyvault_graph_recording
May 7, 2022
Merged

{Graph} Rerun keyvault tests related with az ad/role#22188
jiasli merged 1 commit intoAzure:betafrom
evelyn-ys:keyvault_graph_recording

Conversation

@evelyn-ys
Copy link
Copy Markdown
Member

Description

As #22174 mentioned, keyvault tests have used az ad app commands. Since there's underlying dependency change from AAD Graph to MS Graph, recordings have to be regenerated.

Testing Guide

Switch to each profile to run all profiles' tests:

  • latest: az cloud set -n AzureCloud --profile latest
  • 2018-03-01-hybrid: az cloud set -n AzureCloud --profile 2018-03-01-hybrid
  • ...

And run tests:
azdev test keyvault


This checklist is used to make sure that common guidelines for a pull request are followed.

@ghost ghost added the Auto-Assign Auto assign by bot label Apr 26, 2022
@ghost ghost requested a review from yonzhan April 26, 2022 09:10
@ghost ghost assigned evelyn-ys Apr 26, 2022
@ghost ghost added the KeyVault az keyvault label Apr 26, 2022
@ghost ghost assigned jiasli Apr 26, 2022
@ghost ghost added the RBAC az role label Apr 26, 2022
@ghost ghost requested a review from wangzelin007 April 26, 2022 09:11
@ghost ghost added Graph (doesn't work with label-triggered comments; use Graph.Microsoft instead) az ad AAD labels Apr 26, 2022
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Apr 26, 2022

MSGraph

@evelyn-ys
Copy link
Copy Markdown
Member Author

The recording is also regenerated in #22203, so we can merge #22203 only.

@jiasli jiasli changed the title {MSGraph} Rerun keyvault tests related with az ad/role {Graph} Rerun keyvault tests related with az ad/role May 7, 2022
@jiasli jiasli merged commit 494e47d into Azure:beta May 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Graph (doesn't work with label-triggered comments; use Graph.Microsoft instead) az ad KeyVault az keyvault Microsoft Graph RBAC az role

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants