Skip to content

Commit

Permalink
remove recommendation to convert to base units
Browse files Browse the repository at this point in the history
  • Loading branch information
dashpole committed Jan 13, 2023
1 parent 1cb4bc4 commit 36723e0
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion specification/compatibility/prometheus_and_openmetrics.md
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,6 @@ consecutive `_` characters MUST be replaced with a single `_` character.
The Unit of an OTLP metric point SHOULD be converted to the equivalent unit in Prometheus when possible. This includes:

* Converting from abbreviations to full words (e.g. "ms" to "milliseconds").
* Converting to [base units](https://github.com/OpenObservability/OpenMetrics/blob/main/specification/OpenMetrics.md#units-and-base-units) (e.g. "milliseconds" to "seconds"), which also requires adjusting the value.
* Dropping the portions of the Unit within brackets (e.g. {packets}). Brackets MUST NOT be included in the resulting unit. A "count of foo" is considered unitless in Prometheus.
* Special case: Converting "1" to "ratio".
* Converting "foo/bar" to "foo_per_bar".
Expand Down

0 comments on commit 36723e0

Please sign in to comment.