Skip to content

[prometheus-blackbox-exporter] Add tpl to servicemonitors#6780

Open
poxelyric wants to merge 7 commits into
prometheus-community:mainfrom
poxelyric:patch-tpl-servicemonitors
Open

[prometheus-blackbox-exporter] Add tpl to servicemonitors#6780
poxelyric wants to merge 7 commits into
prometheus-community:mainfrom
poxelyric:patch-tpl-servicemonitors

Conversation

@poxelyric

@poxelyric poxelyric commented Mar 25, 2026

Copy link
Copy Markdown

What this PR does / why we need it

Adds an option to template values of service monitors directly from values.yaml with the tpl function.

Which issue this PR fixes

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: poxelyric <orichen10000@gmail.com>
Signed-off-by: poxelyric <orichen10000@gmail.com>

@monotek monotek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please add tests (non tpl / tpl usage) in ci directory.

Signed-off-by: poxelyric <orichen10000@gmail.com>
@poxelyric

Copy link
Copy Markdown
Author

hi @monotek thanks for the review. I added tpl and non tpl targets in the same value file, is that ok?

@poxelyric poxelyric requested a review from monotek April 4, 2026 13:05
Signed-off-by: poxelyric <orichen10000@gmail.com>
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.

[prometheus-blackbox-exporter] Add tpl the targets url

2 participants