Skip to content

Add common labels to blackbox-exporter ClusterRole#2667

Merged
simonpasquier merged 2 commits intoprometheus-operator:mainfrom
haskjold:add_missing_labels_bb_cr
Jun 4, 2025
Merged

Add common labels to blackbox-exporter ClusterRole#2667
simonpasquier merged 2 commits intoprometheus-operator:mainfrom
haskjold:add_missing_labels_bb_cr

Conversation

@haskjold
Copy link
Copy Markdown
Contributor

@haskjold haskjold commented Jun 2, 2025

Description

The blackbox-exporter ClusterRole doesn't have the common labels applied like the rest of the blackbox-exporter resources.

Type of change

What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry

Please put a one-line changelog entry below. Later this will be copied to the changelog file.

Add common labels to blackbox-exporter ClusterRole resource.

The blackbox-exporter ClusterRole doesn't have the common labels applied like the rest of the blackbox-exporter resources.
@haskjold haskjold requested a review from a team as a code owner June 2, 2025 16:29
@simonpasquier
Copy link
Copy Markdown
Contributor

You'd need to regenerate the manifests.

https://github.com/prometheus-operator/kube-prometheus/blob/main/CONTRIBUTING.md#generated-files

@simonpasquier simonpasquier merged commit 3b77a19 into prometheus-operator:main Jun 4, 2025
11 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.

3 participants