Skip to content
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

fix(rules): upstream recording rule switched to sum_irate #379

Merged
merged 2 commits into from
Aug 25, 2021

Conversation

Duologic
Copy link
Member

@Duologic Duologic commented Aug 25, 2021

ref: kubernetes-monitoring/kubernetes-mixin#619

What this PR does:
Upstream recording rule switched to sum_irate, this was reflected in the recording rule name.

Checklist

  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@Duologic Duologic requested a review from a team as a code owner August 25, 2021 12:36
@@ -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,
Copy link
Member Author

@Duologic Duologic Aug 25, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if we should reflect that in the name of this recording rule too (line 211).

Copy link
Member

@gouthamve gouthamve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gouthamve gouthamve merged commit e7c4d89 into main Aug 25, 2021
@gouthamve gouthamve deleted the duologic/fix_cpu_irate branch August 25, 2021 13:22
simonswine pushed a commit to grafana/mimir that referenced this pull request Oct 18, 2021
…ix_cpu_irate

fix(rules): upstream recording rule switched to sum_irate
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.

2 participants