-
Notifications
You must be signed in to change notification settings - Fork 556
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
docs: updated Azure kms commands. #1972
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your eagle eyes!
@JBrejnholt, it looks like you didn't sign your commits. Can you please sign them? |
Added the signature. Thanks. |
It seems like it didn't quite work :( |
dc82336
to
f574065
Compare
Use "--kms" instead of "-kms", and used "--key" instead of "-key", as the single dash is deprecated. Signed-off-by: Jinhong Brejnholt <jinhong.brejnholt@gmail.com> Author: Jinhong Brejnholt <jinhong.brejnholt@gmail.com> Date: Tue Jun 7 21:30:42 2022 +0200
Can you please check again @dlorenc |
All set! |
@developer-guy are there anything else missing? :) |
All set now! Thanks! |
Use "--kms" instead of "-kms", and used "--key" instead of "-key", as the single dash is deprecated.
Summary
Updated the example script to not use deprecated parameters.
Release Note