-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add Guidance wrt Labelling to Naming and Rules Best Practices #2691
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 3 commits
e57a291
a40367a
ccf009a
0e87e09
a79aa9b
a0ebfa9
7694523
6b6ceb0
938a3a8
e42cd96
1cb09cd
62bf3d7
675d41f
092cc89
0a7611f
4da0a33
9b7793c
519fedc
2c94ccf
8a013e4
6c95581
3fecc71
c530444
9ac82a7
9b1f734
c13630c
f1b517f
9f4259a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
|
|
@@ -80,7 +80,18 @@ the underlying metric type and unit you work with. | |||
| * **Metric collisions**: With growing adoption and metric changes over time, there are cases where lack | ||||
| of unit and type information in the metric name will cause certain series to collide (e.g. `process_cpu` for seconds and milliseconds). | ||||
|
|
||||
| ## Labels | ||||
| ### Important Labels | ||||
|
|
||||
| * `job` | ||||
|
conallob marked this conversation as resolved.
Outdated
|
||||
| * The `job` label is a primary key to differentiate metrics from each other. | ||||
| * If not specified in PromQL expressions, they will match unrelated metrics with the same name. This is especially true in a multi system or multi tenant installation | ||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure this is really a useful note here, as this applies to all label matching.
Suggested change
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It applies to all labels. But
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, but it's not related to job, but related to "target labels" and discovery. That is a different thing and related to querying, not creating labels. |
||||
|
|
||||
| WARNING: When using `without`, be careful not to strip out the `job` label accidentally. | ||||
|
|
||||
|
conallob marked this conversation as resolved.
Outdated
|
||||
| * `instance` | ||||
| * The `instance` label will include the `ip:port` what was scraped, providing a crucial breadcrumb for debugging scrape time issues | ||||
|
conallob marked this conversation as resolved.
Outdated
|
||||
|
|
||||
| ### General Labelling Advice | ||||
|
|
||||
| Use labels to differentiate the characteristics of the thing that is being measured: | ||||
|
|
||||
|
|
||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,6 +19,8 @@ This page documents proper naming conventions and aggregation for recording rule | |
| Keeping the metric name unchanged makes it easy to know what a metric is and | ||
| easy to find in the codebase. | ||
|
|
||
| IMPORTANT: `job` label acts as a primary key. It is **strongly** recommended that you use it to scope your PromQL expressions to the system you are monitoring. | ||
|
conallob marked this conversation as resolved.
Outdated
|
||
|
|
||
| To keep the operations clean, `_sum` is omitted if there are other operations, | ||
| as `sum()`. Associative operations can be merged (for example `min_min` is the | ||
| same as `min`). | ||
|
|
@@ -27,6 +29,18 @@ If there is no obvious operation to use, use `sum`. When taking a ratio by | |
| doing division, separate the metrics using `_per_` and call the operation | ||
| `ratio`. | ||
|
|
||
| ## Labels | ||
|
|
||
| NOTE: Omitting a label in a PromQL expression is the functional equivalent of specifying `label=*` | ||
|
conallob marked this conversation as resolved.
Outdated
|
||
|
|
||
| * In both recorded rules and alerting expressions, always specify a `job` label to prevent expression mismatches from occuring. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think the need to specify
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you elaborate on why Afaik, |
||
| This is especially important in multi-tenant systems where the same metric names may be exported by different jobs or the | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think multi-tenant has anything to do with job and instance labels.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As above, "Multi-tenent systems" may not be the best term, but I'm referring to a Prometheus, run as a platform for multiple teams (e.g by a DevEx or Platform Engineering team), to prevent every team running their own siloed Prometheus stack. In such a setup, all PromQL expressions should be scoped with a Or framed another way, in such a centralised stack, always write |
||
| same job (e.g `node_exporter) in multiple, distinct deployments | ||
|
|
||
| * Always specify a `without` clause with the labels you are aggregating away. | ||
| This is to preserve all the other labels such as `job`, which will avoid | ||
| conflicts and give you more useful metrics and alerts. | ||
|
|
||
| ## Aggregation | ||
|
|
||
| * When aggregating up ratios, aggregate up the numerator and denominator | ||
|
|
@@ -40,10 +54,6 @@ Instead keep the metric name without the `_count` or `_sum` suffix and replace | |
| the `rate` in the operation with `mean`. This represents the average | ||
| observation size over that time period. | ||
|
|
||
| * Always specify a `without` clause with the labels you are aggregating away. | ||
|
conallob marked this conversation as resolved.
|
||
| This is to preserve all the other labels such as `job`, which will avoid | ||
| conflicts and give you more useful metrics and alerts. | ||
|
|
||
| ## Examples | ||
|
|
||
| _Note the indentation style with outdented operators on their own line between | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.