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

Alert StatefulsetNotSatisfiedAtlas alert #1393

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Alert StatefulsetNotSatisfiedAtlas alert #1393

merged 1 commit into from
Oct 14, 2024

Conversation

hervenicol
Copy link
Contributor

@hervenicol hervenicol commented Oct 14, 2024

Towards: giantswarm/roadmap#3719

This PR an alert when a deployment has not all the expected pods running.

Related opsrecipe change here: https://github.com/giantswarm/giantswarm/pull/31903

Checklist

@hervenicol hervenicol self-assigned this Oct 14, 2024
@hervenicol hervenicol requested a review from a team as a code owner October 14, 2024 14:28
description: '{{`Statefulset {{ $labels.namespace}}/{{ $labels.statefulset }} is not satisfied.`}}'
opsrecipe: deployment-not-satisfied/
expr: |-
kube_statefulset_status_replicas{cluster_type="management_cluster", statefulset=~"loki.*|mimir.*"}
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we use namespace instead lf statefulset? The values would be the same but we would not need to use regex?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to stick to the current deploymentnotsatisfied alert's behaviour.

I guess the reason why it's based on the pod name is for apps running in kube-system, where we have to filter by pod name to then route to the proper team.

So, for today/current case, filtering by namespace would work, but if we want to add a kube-system statefulset to the alert we would have to revert.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sure :)

@hervenicol hervenicol merged commit f21c17e into main Oct 14, 2024
7 checks passed
@hervenicol hervenicol deleted the sts-not-ready branch October 14, 2024 14:55
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