opensource/cloudpods
3
0
Fork 0
mirror of https://github.com/yunionio/cloudpods.git synced 2026-09-19 10:46:58 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6147 from rainzm/fix/scaling_notification

Browse Source 
fix/monitor: AutoScalingNotifier.ShouldNotify should call NotifierBase.ShouldNotify
This commit is contained in:
Zexi Li
2020-04-30 14:41:27 +08:00
committed by GitHub
parent 647aec25ce e67effaf3f
commit 8e55ffa23e
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/monitor/alerting/notifiers/auto_scaling.go
+4 -3
View File
@@ -63,9 +63,10 @@ func (as *AutoScalingNotifier) Notify(ctx *alerting.EvalContext, data jsonutils.
return nil
}
func (as *AutoScalingNotifier) ShouldNotify(ctx context.Context, evalContext *alerting.EvalContext, notificationState *models.SAlertnotification) bool {
if evalContext.Rule.State == monitor.AlertStateOK || evalContext.NoDataFound {
func (as *AutoScalingNotifier) ShouldNotify(ctx context.Context, evalContext *alerting.EvalContext,
notificationState *models.SAlertnotification) bool {
if evalContext.NoDataFound {
return false
}
return true
return as.NotifierBase.ShouldNotify(ctx, evalContext, notificationState)
}