-
Notifications
You must be signed in to change notification settings - Fork 574
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
[WIP] Feature/notification rework #7236
Conversation
Please also add function header docs which allows to better understand the purpose. |
Please update the current status, and provide a list of open TODOs. |
Currently the Component is in a state where a lot works and a lot dosen't. A few things are unclear yet but right now I'll be focusing on UnitTests. TODOs:
|
Update: Currently Problem, Recovery, Downtime and Flapping notifications work as far as I can tell. There is an issue with Acknowledgements and Custom Notifications are still TODO. |
This didn't go as planned, unfortunately I have to remove this. |
This is an attempt in rewriting the NotificationComponent and aims to squash multiple issues we have with the current implementation.
The first step was adding a Scheduler for ReNotifications similar to that used in the CHeckerComponent.
In its current state this PR works, except for one or two Notifications Types, like the current one. The next Task is to find a solution for guaranteeing the execution of Notifications in a HA setup, moving more Notification related code from the lib/icinga to the NotificationComponent and writing Unit Tests.
refs #7161
Note: I rebased this on the master, squashing and reordering commits. It currently does not work.