-
Notifications
You must be signed in to change notification settings - Fork 228
Add Azure Key Vaults Doc changes with Dashboards and alerts #5188
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
Add Azure Key Vaults Doc changes with Dashboards and alerts #5188
Conversation
jpipkin1
left a comment
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.
Approved, @Apoorvkudesia-sumologic. I made some minor changes. See commits "Updates from review" and "Remove duplicate step".
amee-sumo
left a comment
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.
Made a few suggestions. Rest LGTM
amee-sumo
left a comment
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
amee-sumo
left a comment
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.
Reviewed the new changes. LGTM
amee-sumo
left a comment
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
Purpose of this pull request
Docs changes to cater the change of Azure Key Vaults app from doc only to normal app.
Addition of new Dashboards and monitors.
Updates in content in app setup section
This pull request...
Select the type of change
Ticket (if applicable)