Skip to content

Stabilize Prometheus Counter->OTLP Sum#4862

Merged
jmacd merged 1 commit intoopen-telemetry:mainfrom
ArthurSens:prom-stabilize-counter
Feb 4, 2026
Merged

Stabilize Prometheus Counter->OTLP Sum#4862
jmacd merged 1 commit intoopen-telemetry:mainfrom
ArthurSens:prom-stabilize-counter

Conversation

@ArthurSens
Copy link
Copy Markdown
Member

@ArthurSens ArthurSens commented Feb 1, 2026

Depends on #4860 being merged first.

Changes

In this PR, I'm declaring the transformation of Prometheus Counters into OTLP Monotonic Sums stable.

I'm still not declaring Gauges stable, as requested in #4744, because I suspect we want to discuss when a Prometheus Gauge becomes an OTLP Gauge or an OTLP UpDownCounter.

@ArthurSens ArthurSens requested review from a team as code owners February 1, 2026 06:40
@ArthurSens ArthurSens changed the title Prom stabilize counter Stabilize Prometheus Counter->OTLP Sum Feb 1, 2026
@reyang
Copy link
Copy Markdown
Member

reyang commented Feb 2, 2026

@ArthurSens could you update the changelog? Thanks!

Copy link
Copy Markdown
Contributor

@jmacd jmacd left a comment

Choose a reason for hiding this comment

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

Thank you @ArthurSens!

@ArthurSens
Copy link
Copy Markdown
Member Author

@ArthurSens could you update the changelog? Thanks!

Happy to add it! I'm a bit confused about when a changelog entry is needed, though. Is this written down somewhere? #4860 didn't need, for example, and the PRs are quite similar

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@dashpole
Copy link
Copy Markdown
Contributor

dashpole commented Feb 3, 2026

#4860 didn't change the actual spec or its status. It just reorganized things. This is making a material change a consumer of the spec might care about, so we should have a changelog entry.

@jmacd jmacd added this pull request to the merge queue Feb 4, 2026
Merged via the queue into open-telemetry:main with commit 8a39837 Feb 4, 2026
7 of 8 checks passed
@ArthurSens
Copy link
Copy Markdown
Member Author

Oh, sorry for the delay here... I was attending FOSDEM and OTel Unplugged.

Do you still want a changelog entry? Happy to add in a follow-up PR

@ArthurSens ArthurSens deleted the prom-stabilize-counter branch February 4, 2026 19:18
github-merge-queue bot pushed a commit that referenced this pull request Feb 4, 2026
Just adds a changelog entry for #4862

Signed-off-by: Arthur Silva Sens <arthursens2005@gmail.com>
@carlosalberto carlosalberto mentioned this pull request Feb 11, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 13, 2026
### Context

- Deprecate Jaeger propagator and make propagator implementation
optional.

([#4827](#4827))
- Deprecate OT Trace propagator and make propagator implementation
optional.

([#4851](#4851))

### Metrics

- Add normative language to the Metrics API/SDK spec concurrency
requirements.

([#4868](#4868))

### Logs

- Add optional `Exception` parameter to Emit LogRecord.

([#4824](#4824))
- Add normative language to the Logging API/SDK spec concurrency
requirements.

([#4885](#4885))

### Resource

- Refine the handling of OTEL_RESOURCE_ATTRIBUTES.

([#4856](#4856))

### Common

- Add string representation guidance for complex attribute value types
(byte arrays,
  empty values, arrays, and maps) for non-OTLP protocols.

([#4848](#4848))

### Compatibility

- Stabilize Prometheus Counter to OTLP Sum transformation.

([#4862](#4862))
- Stabilize Prometheus Gauge to OTLP Gauge transformation.

([#4871](#4871))

### SDK Configuration

- Swap Tracer/Meter/LoggerConfig `disabled` for `enabled` to avoid
double negatives

([#4823](#4823))
- Declarative configuration: rename `ComponentProvider` to
`PluginComponentProvider`, `CreatePlugin` to `CreateComponent` in effort
to
  use consistent vocabulary

([#4806](#4806))
- Declarative configuration: Update instrumentation config behavior to
return
  empty object when not set

([#4817](#4817))
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.

4 participants