Skip to content

Conversation

ywwg
Copy link
Member

@ywwg ywwg commented Jul 9, 2025

The last permutation of the translation options does underscore translation but does not add suffixes. This translation option already exists in Mimir as otel_metric_suffixes_enabled, indicating external demand for this strategy. There is an accompanying update to prometheus-docs to explain the use of this mode: prometheus/docs#2688

The last permutation of the translation options does underscore translation but does not add suffixes.
This translation option already exists in Mimir as otel_metric_suffixes_enabled, indicating external demand for this strategy.
There is an accompanying update to prometheus-docs to explain the use of this mode: prometheus/docs#2688

Signed-off-by: Owen Williams <owen.williams@grafana.com>
@ywwg ywwg force-pushed the owilliams/withoutsuffixes branch from dc4b1ab to f0c7b84 Compare July 9, 2025 18:27
ywwg added 2 commits July 9, 2025 14:30
Signed-off-by: Owen Williams <owen.williams@grafana.com>
Signed-off-by: Owen Williams <owen.williams@grafana.com>
Copy link
Member

@ArthurSens ArthurSens left a comment

Choose a reason for hiding this comment

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

LGTM :)

@ywwg
Copy link
Member Author

ywwg commented Jul 10, 2025

Confirmed that mimir also uses the same settings struct so once they migrate to latest prom/otlptranslator the functionality will be identical: https://github.com/grafana/mimir/blob/main/pkg/distributor/otel.go#L278

@ywwg ywwg merged commit d2f1f4f into main Jul 10, 2025
45 checks passed
@ywwg ywwg deleted the owilliams/withoutsuffixes branch July 10, 2025 15:27
ywwg added a commit to prometheus/docs that referenced this pull request Jul 11, 2025
…xes translation strategy (#2688)

ref: prometheus/prometheus#16849

Signed-off-by: Owen Williams <owen.williams@grafana.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.

2 participants