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 metric semantic conventions for timed operations #129

Conversation

justinfoote
Copy link
Member

This OTEP attempts to address open-telemetry/opentelemetry-specification#654.

I'd like us to come to agreement that we ought to have semantic conventions describing timing metrics. These semantic conventions will describe what instrument(s) are created, how they are named, and what labels are applied to them. They will not describe how or where the instruments are created and incremented.

@justinfoote justinfoote requested review from a team as code owners August 6, 2020 00:14
@justinfoote justinfoote force-pushed the timing_metrics_semantic_conventions branch from 1405c44 to 663ac61 Compare August 6, 2020 00:16
Copy link

@gfuller1 gfuller1 left a comment

Choose a reason for hiding this comment

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

As long as we avoid assuming a span exists when these metrics are getting generated (because one might not exist) I'm fine with this.

@justinfoote
Copy link
Member Author

@open-telemetry/specs-metrics-approvers would y'all have a look at this? I'd like us to come to an agreement sooner rather than later that we want some semantic conventions about naming and labeling of metrics, and that we're not intending to specify API or SDK changes with them.

### Naming

The semantic conventions should specify the naming of this metric instrument, following
the [Metric Instrument Naming Guidelines](./108-naming-guidelines.md#guidelines).
Copy link
Member

Choose a reason for hiding this comment

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

broken link?

Copy link
Member Author

Choose a reason for hiding this comment

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

😬 Fixed.
Thanks!

@fatsheep9146
Copy link

Any update on this?
@jmacd @MrAlias @justinfoote

@tedsuo tedsuo added the triaged label Feb 6, 2023
@tedsuo tedsuo added the stale This issue or PR is stale and will be closed soon unless it is resurrected by the author. label Jul 24, 2023
@tedsuo
Copy link
Contributor

tedsuo commented Jul 24, 2023

@justinfoote we are marking OTEPs which are no longer being updated as stale. We will close this PR in one week. If you wish to resurrect this PR, that is fine.

@tedsuo tedsuo closed this Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
metrics Relates to the Metrics API/SDK stale This issue or PR is stale and will be closed soon unless it is resurrected by the author. triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants