Skip to content

Conversation

@ChrsMark
Copy link
Member

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Not sure if this is desirable for development, but if a from version is set, the description for the metric will also include a message to give end users more information in the telemetry being emitted

@ChrsMark
Copy link
Member Author

ChrsMark commented Sep 26, 2025

Not sure if this is desirable for development, but if a from version is set, the description for the metric will also include a message to give end users more information in the telemetry being emitted

That's reasonable. If we end up making mdatagen to require stability for new metrics, we can add an extra check to require the from for levels that are beyond development.

I will add this in the mdatagen to-dos.

edit: ref -> open-telemetry/opentelemetry-collector#13910

@atoulme
Copy link
Contributor

atoulme commented Oct 4, 2025

@antonblock @ishleenk17 please review

Copy link
Member

@ishleenk17 ishleenk17 left a comment

Choose a reason for hiding this comment

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

Looks good!

@ChrsMark ChrsMark added ready to merge Code review completed; ready to merge by maintainers and removed waiting-for-code-owners labels Oct 6, 2025
@songy23 songy23 merged commit 8fb3c7c into open-telemetry:main Oct 6, 2025
184 of 185 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 6, 2025
graphaelli pushed a commit to graphaelli/opentelemetry-collector-contrib that referenced this pull request Oct 6, 2025
…#42973)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

open-telemetry/opentelemetry-collector#13756
added support for exposing metrics' stability level in the generated
documentation. This PR makes use of this functionality.

We start by setting the stability of all metrics to `development`. More
info about levels can be found at
https://github.com/open-telemetry/opentelemetry-specification/blob/v1.49.0/oteps/0232-maturity-of-otel.md#maturity-levels.

Related to:
- open-telemetry/opentelemetry-collector#11878
- open-telemetry/opentelemetry-collector#13297
-
open-telemetry#35325
-
open-telemetry#42809

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes ~

<!--Describe what testing was performed and which tests were added.-->
#### Testing
~

<!--Describe the documentation added.-->
#### Documentation
Updated

<!--Please delete paragraphs that you did not use before submitting.-->

Signed-off-by: ChrsMark <[email protected]>
mashhurs pushed a commit to mashhurs/opentelemetry-collector-contrib that referenced this pull request Oct 9, 2025
…#42973)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

open-telemetry/opentelemetry-collector#13756
added support for exposing metrics' stability level in the generated
documentation. This PR makes use of this functionality.

We start by setting the stability of all metrics to `development`. More
info about levels can be found at
https://github.com/open-telemetry/opentelemetry-specification/blob/v1.49.0/oteps/0232-maturity-of-otel.md#maturity-levels.

Related to:
- open-telemetry/opentelemetry-collector#11878
- open-telemetry/opentelemetry-collector#13297
-
open-telemetry#35325
-
open-telemetry#42809

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes ~

<!--Describe what testing was performed and which tests were added.-->
#### Testing
~

<!--Describe the documentation added.-->
#### Documentation
Updated

<!--Please delete paragraphs that you did not use before submitting.-->

Signed-off-by: ChrsMark <[email protected]>
tommyers-elastic pushed a commit to tommyers-elastic/opentelemetry-collector-contrib that referenced this pull request Oct 10, 2025
…#42973)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

open-telemetry/opentelemetry-collector#13756
added support for exposing metrics' stability level in the generated
documentation. This PR makes use of this functionality.

We start by setting the stability of all metrics to `development`. More
info about levels can be found at
https://github.com/open-telemetry/opentelemetry-specification/blob/v1.49.0/oteps/0232-maturity-of-otel.md#maturity-levels.

Related to:
- open-telemetry/opentelemetry-collector#11878
- open-telemetry/opentelemetry-collector#13297
-
open-telemetry#35325
-
open-telemetry#42809

<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes ~

<!--Describe what testing was performed and which tests were added.-->
#### Testing
~

<!--Describe the documentation added.-->
#### Documentation
Updated

<!--Please delete paragraphs that you did not use before submitting.-->

Signed-off-by: ChrsMark <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/mysql

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants