Skip to content

Fix #316 current client lacks permissions to read Key Rotation Policy issue#317

Merged
jiaweitao001 merged 1 commit intoAzure:mainfrom
lonegunmanb:f-316
Mar 6, 2023
Merged

Fix #316 current client lacks permissions to read Key Rotation Policy issue#317
jiaweitao001 merged 1 commit intoAzure:mainfrom
lonegunmanb:f-316

Conversation

@lonegunmanb
Copy link
Copy Markdown
Member

Describe your changes

This pull request amended the missing Key Vault keys permissions that examples require now.

Issue number

#316

Checklist before requesting a review

  • The pr title can be used to describe what this pr did in CHANGELOG.md file
  • I have executed pre-commit on my machine
  • I have passed pr-check on my machine

Thanks for your cooperation!

@lonegunmanb
Copy link
Copy Markdown
Member Author

Version upgrade test failed because this issue also exists in previously released version. We can just ignore the version upgrade test.

Copy link
Copy Markdown
Collaborator

@jiaweitao001 jiaweitao001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@jiaweitao001 jiaweitao001 merged commit f54560c into Azure:main Mar 6, 2023
@lonegunmanb lonegunmanb deleted the f-316 branch November 21, 2023 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants