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

Add configuration for keeping original labels in mappings #521

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

rabenhorst
Copy link
Contributor

When a label is assigned in mappings that already exists, it will be replaced. This PR introduces a honor_labels parameter. When this parameter is set to true, it will preserve the original value of the label.

Signed-off-by: Sebastian Rabenhorst <sebastian.rabenhorst@shopify.com>
@matthiasr matthiasr merged commit 1e89c26 into prometheus:master Dec 6, 2023
4 checks passed
@matthiasr
Copy link
Contributor

Awesome, thank you!

matthiasr added a commit that referenced this pull request Dec 6, 2023
with documentation for #521 and changelog.

Signed-off-by: Matthias Rampke <matthias@prometheus.io>
matthiasr added a commit to prometheus/graphite_exporter that referenced this pull request Dec 6, 2023
and some others that `go get -u` pulled in. This brings in the `honor_labels`
support from prometheus/statsd_exporter#521.
matthiasr added a commit to prometheus/graphite_exporter that referenced this pull request Dec 6, 2023
and some others that `go get -u` pulled in. This brings in the `honor_labels`
support from prometheus/statsd_exporter#521.

Signed-off-by: Matthias Rampke <matthias@prometheus.io>
matthiasr added a commit to prometheus/graphite_exporter that referenced this pull request Dec 6, 2023
and some others that `go get -u` pulled in. This brings in the `honor_labels`
support from prometheus/statsd_exporter#521.

Signed-off-by: Matthias Rampke <matthias@prometheus.io>
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