-
Notifications
You must be signed in to change notification settings - Fork 109
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
Populate the exhaustion time in the Alerts #788
Comments
hello! i just need these changes too. I think they are necessary to improve the alerts and give a better support. when could it be ready? thanks! |
This is exactly what I need! Hope it will pass 🙏 🚀 |
+1 👍 |
Hey @metalmatze 😄 invoking some maintainer here 🙏🏼 |
Closing the issue as merged here: #787 Thank you @metalmatze fr reviewing it :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello there @metalmatze,
I have seen that the alerts to Alertmanager are not having a description, or a summary. It can be interesting to have deeper data for them just to bring more light to the devs that rely on that information to hurry when the error budget is burning.
IMHO, an interesting approach is populating the exhaustion time for crafting a meaningful message with it, because depending on it, more developers can focus on the problem.
Proposed code is implemented in this PR
The text was updated successfully, but these errors were encountered: