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

helm: Add tetragon_pod label to metrics via ServiceMonitor #2828

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

lambdanis
Copy link
Contributor

Prometheus Operator by default adds "pod" label, representing the metrics exporter pod, to all scraped metrics. However, Tetragon exposes events metrics with "pod" label, which represents the process that generated events, not Tetragon pod. This label is not overridden if using the default ServiceMonitor (with "honorLabels: true") - that's good. However, it still can be useful to know which Tetragon pod exported the metric. Let's add "tetragon_pod" label to metrics scraped via default ServiceMonitor.

@lambdanis lambdanis added area/metrics Related to prometheus metrics release-note/minor This PR introduces a minor user-visible change area/helm Related to the Helm chart labels Aug 20, 2024
@lambdanis lambdanis requested a review from a team as a code owner August 20, 2024 21:23
@lambdanis lambdanis requested a review from olsajiri August 20, 2024 21:23
Prometheus Operator by default adds "pod" label, representing the metrics
exporter pod, to all scraped metrics. However, Tetragon exposes events metrics
with "pod" label, which represents the process that generated events, not
Tetragon pod. This label is not overridden if using the default ServiceMonitor
(with "honorLabels: true") - that's good. However, it still can be useful to
know which Tetragon pod exported the metric. Let's add "tetragon_pod" label to
metrics scraped via default ServiceMonitor.

Signed-off-by: Anna Kapuscinska <[email protected]>
@lambdanis lambdanis force-pushed the pr/lambdanis/tetragon-labels-events-metrics branch from e9356c4 to 571bac0 Compare August 20, 2024 21:26
@lambdanis lambdanis changed the title helm: Add tetragon_pod label to metrics in ServiceMonitor helm: Add tetragon_pod label to metrics via ServiceMonitor Aug 20, 2024
@lambdanis lambdanis merged commit 340c6b0 into main Aug 23, 2024
41 checks passed
@lambdanis lambdanis deleted the pr/lambdanis/tetragon-labels-events-metrics branch August 23, 2024 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm Related to the Helm chart area/metrics Related to prometheus metrics release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants