Skip to content

Commit

Permalink
Merge pull request #379 from grafana/duologic/fix_cpu_irate
Browse files Browse the repository at this point in the history
fix(rules): upstream recording rule switched to sum_irate
  • Loading branch information
gouthamve authored Aug 25, 2021
2 parents 79144dd + 79c5f75 commit e7c4d89
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@
* [BUGFIX] Fixed rollout progress dashboard to correctly work when a Cortex service deployment spans across multiple zones (a zone is expected to have the `zone-[a-z]` suffix). #366
* [BUGFIX] Fixed rollout progress dashboard to include query-scheduler too. #376
* [BUGFIX] Fixed `-distributor.extend-writes` setting on ruler when `unregister_ingesters_on_shutdown` is disabled. #369
* [BUGFIX] Upstream recording rule `node_namespace_pod_container:container_cpu_usage_seconds_total:sum_irate` renamed. #379

## 1.9.0 / 2021-05-18

Expand Down
2 changes: 1 addition & 1 deletion cortex-mixin/recording_rules.libsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ local utils = import 'mixin-utils/utils.libsonnet';
sum by (cluster, namespace, deployment) (
label_replace(
label_replace(
node_namespace_pod_container:container_cpu_usage_seconds_total:sum_rate,
node_namespace_pod_container:container_cpu_usage_seconds_total:sum_irate,
"deployment", "$1", "pod", "(.*)-(?:([0-9]+)|([a-z0-9]+)-([a-z0-9]+))"
),
# The question mark in "(.*?)" is used to make it non-greedy, otherwise it
Expand Down

0 comments on commit e7c4d89

Please sign in to comment.