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

Update prometheus.exporter.blackbox.md #2322

Merged
merged 3 commits into from
Jan 28, 2025

Conversation

maxlemieux
Copy link
Contributor

Added an example of how to configure the instance label to work better with some prebuilt dashboards that expect the target url here.

PR Description

Which issue(s) this PR fixes

Followup on a support ticket

Notes to the Reviewer

This example is by request of an Alloy user and shows how to make the Blackbox exporter work better with dashboards like this one that expect instance label to contain target urls: https://grafana.com/grafana/dashboards/7587-prometheus-blackbox-exporter/

PR Checklist

  • Documentation added

Added an example of how to configure the `instance` label to work better with some prebuilt dashboards that expect the target url here.
@clayton-cornell clayton-cornell added the type/docs Docs Squad label across all Grafana Labs repos label Jan 2, 2025
@clayton-cornell
Copy link
Contributor

@maxlemieux When you have a moment, can you take a look at the review suggestion. If it's OK, we can get this PR finished up and published.

@clayton-cornell
Copy link
Contributor

@maxlemieux The build is failing because of an old Go version. Can you rebase your branch against origin/main and push back to the PR? This will fix that build fail and get this moving again.

@clayton-cornell clayton-cornell merged commit 7cb72f6 into grafana:main Jan 28, 2025
26 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 28, 2025
* Update prometheus.exporter.blackbox.md

Added an example of how to configure the `instance` label to work better with some prebuilt dashboards that expect the target url here.

* Update docs/sources/reference/components/prometheus/prometheus.exporter.blackbox.md

---------

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
(cherry picked from commit 7cb72f6)
github-actions bot pushed a commit that referenced this pull request Jan 28, 2025
* Update prometheus.exporter.blackbox.md

Added an example of how to configure the `instance` label to work better with some prebuilt dashboards that expect the target url here.

* Update docs/sources/reference/components/prometheus/prometheus.exporter.blackbox.md

---------

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
(cherry picked from commit 7cb72f6)
github-actions bot pushed a commit that referenced this pull request Jan 28, 2025
* Update prometheus.exporter.blackbox.md

Added an example of how to configure the `instance` label to work better with some prebuilt dashboards that expect the target url here.

* Update docs/sources/reference/components/prometheus/prometheus.exporter.blackbox.md

---------

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
(cherry picked from commit 7cb72f6)
github-actions bot pushed a commit that referenced this pull request Jan 28, 2025
* Update prometheus.exporter.blackbox.md

Added an example of how to configure the `instance` label to work better with some prebuilt dashboards that expect the target url here.

* Update docs/sources/reference/components/prometheus/prometheus.exporter.blackbox.md

---------

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
(cherry picked from commit 7cb72f6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants