-
Notifications
You must be signed in to change notification settings - Fork 1
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
Revise the Getting Started Monitoring
guide to support Scalar Manager
#248
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!
|
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.
Sorry, my editor automatically removes the spaces.
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.
I've added some comments and suggestions. PTAL!
@josh-wong @kota2and3kan Could you take a look at the suggestions as well?
Thank you 🙇 |
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.
Thank you for the update!
Overall looks good to me, but I have one very minor question.
@josh-wong
Could you please take a look at my question when you have time?
Co-authored-by: Josh Wong <[email protected]>
Co-authored-by: Josh Wong <[email protected]>
Co-authored-by: Josh Wong <[email protected]>
@josh-wong @kota2and3kan |
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! Thank you!🙇♂️
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! Thank you!
Description
This PR revised the "Getting Started with Helm Charts (Monitoring using Prometheus Operator)" document to add the description about supporting Scalar Manager to collect CPU and memory resources.
Scalar Manager relies on Prometheus Stack to do so and some configurations need to be set correctly.
Related issues and/or PRs
N/A
Changes made
Checklist
Release notes
Added description to "Getting Started with Helm Charts (Monitoring using Prometheus Operator)" to support Scalar Manager to collect the CPU and memory resources