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

Add alerts to monitor the HelmReleases for cilium and coredns #1433

Merged
merged 4 commits into from
Nov 20, 2024

Conversation

fiunchinho
Copy link
Member

@fiunchinho fiunchinho commented Nov 19, 2024

Towards https://github.com/giantswarm/giantswarm/issues/32121

We currently deploy cilium and coredns using HelmReleases but we don't have alerts to notify us when something is wrong. What do you think @giantswarm/team-cabbage ? The alerts won't page outside of business hours.

Checklist

@fiunchinho fiunchinho self-assigned this Nov 19, 2024
@fiunchinho fiunchinho requested a review from a team as a code owner November 19, 2024 21:26
@fiunchinho fiunchinho force-pushed the cilium-helmrelease-alert branch 3 times, most recently from ca72f98 to 14b403c Compare November 19, 2024 22:01
@fiunchinho fiunchinho force-pushed the cilium-helmrelease-alert branch from 14b403c to ded0373 Compare November 19, 2024 22:14
Copy link
Member

@ubergesundheit ubergesundheit left a comment

Choose a reason for hiding this comment

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

Not sure whats going on with the tests, but I like the idea 👍

@fiunchinho fiunchinho merged commit 92f843f into main Nov 20, 2024
7 checks passed
@fiunchinho fiunchinho deleted the cilium-helmrelease-alert branch November 20, 2024 14:03
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