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

fixed typo on integration/prometheus page #142

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

Conversation

andrewhu-hcl
Copy link

@andrewhu-hcl andrewhu-hcl commented Oct 27, 2021

What this PR does / why we need it:
Fixed typo for the podMonitor example under the "Prometheus community version (helm) - kube-prometheus-stack with pod monitor" section. Original podMonitor configuration will scrap the metric from all the pods which port is TCP and the pod with target label. Update the podMetricsEndpoints to only scrape the metric from the target pod in the PodMonitor yaml file.

Checklist:

update the podMetricsEndpoints to only scrapt the metric from target pod in the PodMonitor yaml file

Signed-off-by: Andrew Hu <andrew.hu@hcl.com>
@netlify
Copy link

netlify bot commented Oct 27, 2021

❌ Deploy Preview for litmus-docs-411023 failed.

🔨 Explore the source changes: ee66694

🔍 Inspect the deploy log: https://app.netlify.com/sites/litmus-docs-411023/deploys/6179b316bca0700008acfced

Copy link
Contributor

@divya-mohan0209 divya-mohan0209 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@neelanjan00 neelanjan00 changed the title fixed typo on integraion/prometheus page fixed typo on integration/prometheus page Mar 13, 2024
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.

Typo on integraion/prometheus page
5 participants