Skip to content

[prometheus] Stabilize SDK exporter port config#5026

Merged
jack-berg merged 2 commits intoopen-telemetry:mainfrom
johannaojeling:prometheus-stabilize-port-config
Apr 23, 2026
Merged

[prometheus] Stabilize SDK exporter port config#5026
jack-berg merged 2 commits intoopen-telemetry:mainfrom
johannaojeling:prometheus-stabilize-port-config

Conversation

@johannaojeling
Copy link
Copy Markdown
Member

Fixes #4985

Changes

Moves Prometheus Metrics Exporter spec Port configuration status from Development to Stable. Implemented by 3+ SDKs.

@johannaojeling johannaojeling requested review from a team as code owners April 14, 2026 09:04
@johannaojeling
Copy link
Copy Markdown
Member Author

@open-telemetry/prometheus-interoperability

@ArthurSens
Copy link
Copy Markdown
Member

Just to inform folks who might have missed discussions elsewhere:

This is currently blocked because SDKs do not implement it consistently. See #4984 (comment)

Copy link
Copy Markdown
Member

@jack-berg jack-berg left a comment

Choose a reason for hiding this comment

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

Same thought I mentioned here: #5024 (review)

Don't care where it answered, but it should get answered 🙂

Comment thread CHANGELOG.md Outdated
github-merge-queue Bot pushed a commit that referenced this pull request Apr 17, 2026
Fixes a remark made in
#5026 (comment).

## Changes

Groups the Prometheus stabilization changelog entries into top level
bullet points.

* [ ] Related issues #
* [ ] Related [OTEP(s)](https://github.com/open-telemetry/oteps) #
* [ ] Links to the prototypes (when adding or changing features)
* [ ]
[`CHANGELOG.md`](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CHANGELOG.md)
file updated for non-trivial changes
* For trivial changes, include `[chore]` in the PR title to skip the
changelog check
* [ ] [Spec compliance
matrix](https://github.com/open-telemetry/opentelemetry-specification/blob/main/spec-compliance-matrix/template.yaml)
updated if necessary
* [ ] [Declarative config data
model](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/data-model.md#overview)
is updated if SDK config surface is changed
@johannaojeling johannaojeling force-pushed the prometheus-stabilize-port-config branch from a166e52 to cfbaf9c Compare April 20, 2026 08:16
hilmarf pushed a commit to apeirora/opentelemetry-specification that referenced this pull request Apr 22, 2026
)

Fixes a remark made in
open-telemetry#5026 (comment).

## Changes

Groups the Prometheus stabilization changelog entries into top level
bullet points.

* [ ] Related issues #
* [ ] Related [OTEP(s)](https://github.com/open-telemetry/oteps) #
* [ ] Links to the prototypes (when adding or changing features)
* [ ]
[`CHANGELOG.md`](https://github.com/open-telemetry/opentelemetry-specification/blob/main/CHANGELOG.md)
file updated for non-trivial changes
* For trivial changes, include `[chore]` in the PR title to skip the
changelog check
* [ ] [Spec compliance
matrix](https://github.com/open-telemetry/opentelemetry-specification/blob/main/spec-compliance-matrix/template.yaml)
updated if necessary
* [ ] [Declarative config data
model](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/configuration/data-model.md#overview)
is updated if SDK config surface is changed
@jack-berg jack-berg enabled auto-merge April 23, 2026 19:57
@jack-berg jack-berg disabled auto-merge April 23, 2026 19:57
@jack-berg
Copy link
Copy Markdown
Member

@ArthurSens ready to merge?

Copy link
Copy Markdown
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.

Yep, might need a rebase for the changelog though!

@jack-berg
Copy link
Copy Markdown
Member

I already fixed the changelog. In there, I moved the stabilization entries for the prometheus metric exporter spec from ### Compatibility to ### Metrics, since prometheus exporter is a document under metrics.

@jack-berg jack-berg added this pull request to the merge queue Apr 23, 2026
Merged via the queue into open-telemetry:main with commit 80b21d8 Apr 23, 2026
9 checks passed
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.

Stabilize Prometheus Metrics Exporter spec: Port configuration

7 participants