Skip to content

[prometheus-blackbox-exporter] optional set path per target - #7201

Merged
monotek merged 1 commit into
prometheus-community:mainfrom
itsmethemojo:path-config
Aug 31, 2026
Merged

[prometheus-blackbox-exporter] optional set path per target#7201
monotek merged 1 commit into
prometheus-community:mainfrom
itsmethemojo:path-config

Conversation

@itsmethemojo

@itsmethemojo itsmethemojo commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it

This adds the option to set the path per target if needed when creating service monitors for the blackbox exporter. When testing multiple domains and pathes with one blackbox exporter chart installation this will help.

Which issue this PR fixes

none

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Signed-off-by: Marian Poeschmann <github@mail.itsmethemojo.eu>
@itsmethemojo

itsmethemojo commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

i actually do not understand if the unittest task is mandatory here because only a couple of charts do have unittests

should i add tests only for this feature for the chart prometheus-blackbox-exporter?

$ ls -1 -R | grep unit
unittests
./charts/alertmanager/unittests:
unittests
unittests
./charts/kube-prometheus-stack/charts/crds/unittests:
./charts/kube-prometheus-stack/unittests:
./charts/kube-prometheus-stack/unittests/alertmanager:
./charts/kube-prometheus-stack/unittests/exporters:
./charts/kube-prometheus-stack/unittests/exporters/kube-scheduler:
./charts/kube-prometheus-stack/unittests/grafana:
./charts/kube-prometheus-stack/unittests/prometheus:
./charts/kube-prometheus-stack/unittests/prometheus-operator:
./charts/kube-prometheus-stack/unittests/thanos-ruler:
unittests
./charts/kube-state-metrics/unittests:
unittests
./charts/prom-label-proxy/unittests:
unittests
./charts/prometheus-kafka-exporter/unittests:

UPDATE i looked up how previous PRs in this chart handlet it and removed the unit test checkbox

@monotek
monotek merged commit d46b896 into prometheus-community:main Aug 31, 2026
16 checks passed
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.

2 participants