From d1a6eec04d8025c9a850e28e741c739ed58f85ad Mon Sep 17 00:00:00 2001 From: David Ashpole Date: Tue, 22 Aug 2023 18:12:44 +0000 Subject: [PATCH] remove non-ucum units from prometheus readme --- pkg/translator/prometheus/README.md | 7 ------- 1 file changed, 7 deletions(-) diff --git a/pkg/translator/prometheus/README.md b/pkg/translator/prometheus/README.md index 7c29facac63c..3f50583c5247 100644 --- a/pkg/translator/prometheus/README.md +++ b/pkg/translator/prometheus/README.md @@ -29,7 +29,6 @@ | Percentages (unit is `1`) | Append `_ratio` (for gauges only) | `system.memory.utilization` with unit `1` → `system_memory_utilization_ratio` | | Percentages (unit is `%`) | Replace `%` with `percent` `_percent` | `storage.filesystem.utilization` with unit `%` → `storage_filesystem_utilization_percent` | | Rates (unit contains `/`) | Replace `/` with `per` | `astro.light.speed` with unit `m/s` → `astro_light_speed_meters_per_second` | -| Dollars (unit is `$`) | Replace `$` with `dollars` | `crypto.dogecoin.value` with unit `$` → `crypto_dogecoin_value_dollars` | | Counter | Append `_total` | `system.processes.created` → `system_processes_created_total` | List of standard OpenTelemetry units that will be translated to [Prometheus standard base units](https://prometheus.io/docs/practices/naming/#base-units): @@ -54,11 +53,6 @@ List of standard OpenTelemetry units that will be translated to [Prometheus stan | `MBy` | `megabytes` | | `GBy` | `gigabytes` | | `TBy` | `terabytes` | -| `B` | `bytes` | -| `KB` | `kilobytes` | -| `MB` | `megabytes` | -| `GB` | `gigabytes` | -| `TB` | `terabytes` | | **SI Units** | | | `m` | `meters` | | `V` | `volts` | @@ -70,7 +64,6 @@ List of standard OpenTelemetry units that will be translated to [Prometheus stan | `Cel` | `celsius` | | `Hz` | `hertz` | | `%` | `percent` | -| `$` | `dollars` | > **Note** > Prometheus also recommends using base units (no kilobytes, or milliseconds, for example) but these functions will not attempt to convert non-base units to base units.