From 07c5da6c53adb1c00b653481fcb7192666201ffb Mon Sep 17 00:00:00 2001 From: Jack Berg Date: Wed, 8 Mar 2023 10:47:27 -0600 Subject: [PATCH 1/2] Metric units should use UCUM case sensitive variant --- CHANGELOG.md | 3 +++ specification/metrics/semantic_conventions/README.md | 2 ++ 2 files changed, 5 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index bc4b5390cb1..801019d3914 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -13,6 +13,9 @@ release. ### Metrics +- Clarify that units should use UCUM case sensitive variant. + ([#TBD](https://github.com/open-telemetry/opentelemetry-specification/pull/TBD)) + ### Logs ### Resource diff --git a/specification/metrics/semantic_conventions/README.md b/specification/metrics/semantic_conventions/README.md index 5efdc859f08..b5d943b9326 100644 --- a/specification/metrics/semantic_conventions/README.md +++ b/specification/metrics/semantic_conventions/README.md @@ -184,6 +184,8 @@ total) are dimensionless and SHOULD use the default unit `1` (the unity). [annotations](https://ucum.org/ucum.html#para-curly) with curly braces to give additional meaning *without* the leading default unit (`1`). For example, use `{packets}`, `{errors}`, `{faults}`, etc. +- Instrument units SHOULD be specified using the UCUM case sensitive ("c/s") + variant. For example, "Cel" for the unit with full name "degree Celsius". ### Instrument Types From da6b6d1b43c0ac6b33ddee6e8e88f9ca2d85747e Mon Sep 17 00:00:00 2001 From: Jack Berg Date: Wed, 8 Mar 2023 10:47:57 -0600 Subject: [PATCH 2/2] Changelog --- CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 801019d3914..575de96fe09 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -14,7 +14,7 @@ release. ### Metrics - Clarify that units should use UCUM case sensitive variant. - ([#TBD](https://github.com/open-telemetry/opentelemetry-specification/pull/TBD)) + ([#3306](https://github.com/open-telemetry/opentelemetry-specification/pull/3306)) ### Logs