Skip to content
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: Clarify metric collection methods for NTH modes #1084

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

younsl
Copy link

@younsl younsl commented Nov 7, 2024

Issue #, if available:

Description of changes:

Updated the README to clarify the Prometheus metric collection methods for NTH in IMDS and Queue modes, to ensure users understand how to configure Prometheus based on the NTH mode they are using.

  • Explained how Prometheus metric collection differs between IMDS and Queue modes.

  • Added a warning about serviceMonitor and podMonitor being Prometheus Operator resources.

  • Updated Queue mode to include two methods for metrics collection: using serviceMonitor with the Prometheus Operator or adding the aws-node-termination-handler service in scrape_configs.

  • Provided an example scrape_configs for Prometheus in the Helm chart.

  • Updated IMDS mode to mention podMonitor for metrics collection.

image

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

* Explained how Prometheus metric collection differs between IMDS and Queue modes.

* Added a warning about serviceMonitor and podMonitor being Prometheus Operator resources.

* Updated Queue mode to include two methods for metrics collection: using serviceMonitor with the Prometheus Operator or adding the aws-node-termination-handler service in `scrape_configs`.

* Provided an example `scrape_configs` for Prometheus in the Helm chart.

* Updated IMDS mode to mention podMonitor for metrics collection.
@younsl younsl requested a review from a team as a code owner November 7, 2024 14:29
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.

1 participant