Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add OpenMetrics guidance #1154

Merged

Conversation

justinfoote
Copy link
Member

Fixes #600

This PR adds some guidelines for interoperating with OpenMetrics systems.
I have not fully specified the behavior of accepting OpenMetrics metrics -- I've delegated that to the existing design document in the collector's prometheus receiver. I am making the assumption that this is the only place we expect to accept OpenMetrics data into OpenTelemetry, and that their design document is sufficient for now.

I have included guidelines about metric and label renaming when converting OpenTelemetry metrics to OpenMetrics.
I have not included details about the values, or about how many OpenMetrics metrics should be created for each OpenTelemetry aggregation type. I'd love to chat more about the details here before it gets written up.

Changes

  • Add new openmetrics-guidelines.md to metrics semantic conventions.

@github-actions
Copy link

github-actions bot commented Nov 6, 2020

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Nov 6, 2020
@yurishkuro
Copy link
Member

@jmacd ptal

@github-actions github-actions bot removed the Stale label Nov 7, 2020
Copy link
Contributor

@MrAlias MrAlias left a comment

Choose a reason for hiding this comment

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

This looks like a great start to build from in subsequent PRs. 👍

@SergeyKanzhelev SergeyKanzhelev merged commit 1ebb9e0 into open-telemetry:master Nov 17, 2020
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
* Add OpenMetrics interop guidelines

* Add Prometheus Receiver guidelines; reference collector design

* Add OpenMetrics guidelines to changelog

* Add work in progress note to OpenMetrics guidance

Co-authored-by: Sergey Kanzhelev <S.Kanzhelev@live.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.

More prescriptive guidance on metric naming
5 participants