Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Turn off prometheus.io/scrape annotation for node and windows exporters #1079

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

petewall
Copy link
Collaborator

@petewall petewall commented Jan 8, 2025

Also fix typo in integration test name
Also add new integration test to check for DPM when using the prometheus.io/scrape annotations

@petewall petewall self-assigned this Jan 8, 2025
…s exporter.

Also fix typo in integration test name
Also add new integration test to check for DPM when using the prometheus.io/scrape annotations

Signed-off-by: Pete Wall <pete.wall@grafana.com>
Signed-off-by: Pete Wall <pete.wall@grafana.com>
@petewall petewall force-pushed the fix/no-prom-annotation branch from 2f71b94 to 3f39dea Compare January 8, 2025 23:04
@petewall petewall merged commit b43fcf2 into main Jan 8, 2025
28 checks passed
@petewall petewall deleted the fix/no-prom-annotation branch January 9, 2025 15:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure that systems we deploy are not annotated with prometheus.io/scrape
2 participants