Skip to content

Fix monitoring metrics format in provider guide#198

Open
jdevera wants to merge 2 commits into
criteo:mainfrom
jdevera:fix-metrics-doc
Open

Fix monitoring metrics format in provider guide#198
jdevera wants to merge 2 commits into
criteo:mainfrom
jdevera:fix-metrics-doc

Conversation

@jdevera
Copy link
Copy Markdown
Contributor

@jdevera jdevera commented May 1, 2026

The documented graphite path and metric set did not match the actual emission. Update the provider guide to describe:

  • the six-segment path including the user partition between command name and metric leaf
  • the four metric leaves (count, duration, ok, ko) as independent siblings rather than ok/ko compounds of count and duration
  • the nanosecond unit on duration
  • the three lines emitted per invocation

jdevera added 2 commits May 1, 2026 20:05
The documented graphite path and metric set did not match the actual
emission. Update the provider guide to describe:

- the six-segment path including the user partition between command name
  and metric leaf
- the four metric leaves (count, duration, ok, ko) as independent siblings
  rather than ok/ko compounds of count and duration
- the nanosecond unit on duration
- the three lines emitted per invocation
Fixes markdownlint MD040 (fenced-code-language) reported by the
gh-pages CI lint job. The new code block in the Monitoring section
of the provider guide was missing a language tag; using `text` to
match the existing convention elsewhere in the file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant