opentelemetry-specification icon indicating copy to clipboard operation
opentelemetry-specification copied to clipboard

Update Prometheus spec based on new updates in upstream Prometheus

Open ywwg opened this issue 7 months ago • 6 comments

Prometheus updated its data model to allow for, among other things, widen the restrictions for allowable metric and label names.

The corresponding specification for Prometheus in this Open Telemetry specification should be updated to match. Specifically, I'd like to make it clear that exporters can optionally pass through Otel names untranslated, with dot separators and no suffixes, to Prometheus. If there are any other differences between the two specifications those should be resolved at the same time.

Tasks:

  • [ ] Translation strategies from Otel to Prometheus (underscores with suffixes, underscores without suffixes, utf-8 with suffixes, utf-8 without suffixes ("no-translation"): https://github.com/open-telemetry/opentelemetry-specification/pull/4533
  • [ ] UTF-8 character support: https://github.com/open-telemetry/opentelemetry-specification/pull/4533
  • [ ] Transformation of Exponential and Explicit Histograms into Native Histograms.: https://github.com/open-telemetry/opentelemetry-specification/pull/4561
  • [ ] Support histogram NaNs
  • [x] Switch from otel_scope_info metric to labels

ywwg avatar May 05 '25 19:05 ywwg

Related: https://github.com/open-telemetry/opentelemetry-specification/issues/4371#issuecomment-2649608000

cijothomas avatar May 06 '25 01:05 cijothomas

cc @open-telemetry/prometheus-interoperability

jpkrohling avatar May 12 '25 09:05 jpkrohling

Updating this bug to make it a tracking bug

ywwg avatar May 22 '25 18:05 ywwg

@ywwg do you have any context on the histogram stale NaNs? Is https://github.com/open-telemetry/opentelemetry-specification/pull/4561 sufficient, or is more detail needed here?

dashpole avatar Jun 23 '25 17:06 dashpole

I do not have much histogram context -- I'll ask Krajo

ywwg avatar Jun 25 '25 15:06 ywwg

Krajo says it's unclear if there will be more updates necessary, delta support may cause some churn. And there are a lot of open NH bugs: https://github.com/prometheus/prometheus/issues?q=is%3Aissue%20state%3Aopen%20milestone%3A%22Native%20Histograms%22

ywwg avatar Jun 25 '25 16:06 ywwg

This work is separate from the spec updates for native histograms, so I've removed those mentions from this task list. Instead, this Issue is specifically about the name translation and utf8 updates

ywwg avatar Jun 26 '25 19:06 ywwg

Can we close this one since #4533 has been merged?

ArthurSens avatar Jul 25 '25 18:07 ArthurSens