Reset no_more_notifications on filter mismatch correctly

Previously, if you enable flapping for a Checkable but the corresponding
`Notification` object does not have `FlappingStart` or `FlappingEnd`
types set, the `no_more_notifications` flag wasn't reset to false again.
This commit ensures that this flag is always reset on `Recovery` even
the type filter does not match including when we miss the `Recovery` due
to Flapping state.
This commit is contained in:
Yonas Habteab 2025-03-12 14:20:01 +01:00
parent 9166326876
commit 4596b44171

View File

@ -365,7 +365,7 @@ void Notification::BeginExecuteNotification(NotificationType type, const CheckRe
*/
{
ObjectLock olock(this);
if (type == NotificationRecovery && GetInterval() <= 0)
if (GetInterval() <= 0 && IsRecoveryOrFlappingEndAndCheckableIsOK(checkable, cr, type))
SetNoMoreNotifications(false);
}