-
Notifications
You must be signed in to change notification settings - Fork 183
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
Clarify unit annotation format for metric semantic conventions #1680
Comments
@arminru do yo plan to work on this? If not we need a "help wanted" label. |
I'd like to take a look at this. @jsuereth do you see this issue depending on any output from the Semantic Conventions Working Group, or can it be worked on in parallel? |
Was going through my backlog and saw this issue, I'm no longer able to work on this. @jsuereth can this be closed, or is it still needed? |
transferring to https://github.com/open-telemetry/semantic-conventions |
I believe this can be closed now. I checked and there are no units with spaces in them. And I believe this section now addresses the requested guidance: https://github.com/open-telemetry/semantic-conventions/blob/main/docs/general/metrics.md#instrument-units |
The semantic conventions for metrics currently use both spaces and underscores in unit annotations consisting of multiple words. Additionally, we have examples where the spaces between words are just omitted.
Before calling any of the metrics semantic conventions stable, we should add guidance on what's preferred and align the existing conventions to be consistent.
Also, as per the UCUM standard, both
1
and{count}
are considered equivalent, which might lead to inconsistent conventions. A preference for either of them could also be laid out by the OTel spec.Examples
https://github.com/open-telemetry/opentelemetry-specification/blob/534d4562b9e4f9ba030146783a1785f94b266862/specification/metrics/semantic_conventions/instrumentation/kafka.md?plain=1#L40-L42
https://github.com/open-telemetry/opentelemetry-specification/blob/534d4562b9e4f9ba030146783a1785f94b266862/specification/metrics/semantic_conventions/instrumentation/kafka.md?plain=1#L78-L83
https://github.com/open-telemetry/opentelemetry-specification/blob/534d4562b9e4f9ba030146783a1785f94b266862/specification/metrics/semantic_conventions/instrumentation/kafka.md?plain=1#L65
https://github.com/open-telemetry/opentelemetry-specification/blob/534d4562b9e4f9ba030146783a1785f94b266862/specification/metrics/semantic_conventions/faas-metrics.md?plain=1#L43
The text was updated successfully, but these errors were encountered: