Skip to content

Commit

Permalink
Remove misleading pluralization wording related to count metrics (ope…
Browse files Browse the repository at this point in the history
…n-telemetry#488)

Co-authored-by: Alexander Wert <AlexanderWert@users.noreply.github.com>
  • Loading branch information
2 people authored and pyohannes committed Jan 17, 2024
1 parent a7f02da commit e228e05
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,9 @@ release.

### Fixes

- Remove misleading pluralization wording related to count metrics
([#488](https://github.com/open-telemetry/semantic-conventions/pull/488))

## v1.23.0 (2023-11-03)

This release marks the first where the core of HTTP semantic conventions have
Expand Down
6 changes: 2 additions & 4 deletions docs/general/metrics.md
Original file line number Diff line number Diff line change
Expand Up @@ -124,12 +124,10 @@ should be pluralized, even if only a single data point is recorded.

If the value being recorded represents the count of concepts signified
by the namespace then the metric should be named `count` (within its namespace).
The pluralization rule does not apply in this case.

For example if we have a namespace `system.processes` which contains all metrics related
For example if we have a namespace `system.process` which contains all metrics related
to the processes then to represent the count of the processes we can have a metric named
`system.processes.count`. The suffix `count` here indicates that it is the count of
`system.processes`.
`system.process.count`.

#### Do not use `total`

Expand Down

0 comments on commit e228e05

Please sign in to comment.