-
Notifications
You must be signed in to change notification settings - Fork 50
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: added prometheus integration guide #88
Conversation
β Deploy Preview for meek-marigold-103c48 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
This change includes activation instructions and explanations for the various analyzers of the Prometheus integration. Signed-off-by: Daniel Clark <danielclark@google.com>
Got time to take a look @AnaisUrlichs ? |
Hey, thanks, looking now :) |
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.
Left a few comments otherwise, it looks good, thank you
- Added a header about Prerequisites - Made previous Prometheus installation note more clear Signed-off-by: Daniel Clark <danielclark@google.com>
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.
Thanks for the feedback. Addressed some comments
This change includes activation instructions and explanations for the various analyzers of the Prometheus integration.
Closes k8sgpt-ai/k8sgpt#858
π Description
β Checks
βΉ Additional Information