Skip to content

Conversation

simonpasquier
Copy link
Contributor

This PR reduces duplication for the KubeMemoryOvercommit and KubeCPUOvercommit alerting rules: the expressions are almost identical except for the resource label and the kube_pod_container_resource_requests recorded metrics.

cc @rexagod

Copy link
Collaborator

@skl skl left a comment

Choose a reason for hiding this comment

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

Thank you! There's good test covereage here already, so I'm happy with the queries. Left a suggestion around DRY'ing up the description fields.

@skl skl merged commit eb631d8 into kubernetes-monitoring:master Aug 1, 2025
9 checks passed
simonpasquier added a commit to simonpasquier/kubernetes-mixin that referenced this pull request Aug 4, 2025
Similar to kubernetes-monitoring#1087, this commit refactors the KubeCPUQuotaOvercommit and
KubeMemoryQuotaOvercommit alerting rules to reduce duplication.

Signed-off-by: Simon Pasquier <[email protected]>
skl pushed a commit that referenced this pull request Aug 11, 2025
Similar to #1087, this commit refactors the KubeCPUQuotaOvercommit and
KubeMemoryQuotaOvercommit alerting rules to reduce duplication.

Signed-off-by: Simon Pasquier <[email protected]>
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