gsadmin/pulse
1
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-09-25 04:33:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert "fix: properly handle 100% thresholds to disable alerts (addresses #434)"

Browse Source 
This reverts commit ffb744d711.
This commit is contained in:
Pulse Monitor
2025-09-09 21:27:29 +00:00
parent ffb744d711
commit 9f09c981c7
1 changed files with 0 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/alerts/alerts.go
-29
View File
@@ -923,35 +923,6 @@ func (m *Manager) checkMetric(resourceID, resourceName, node, instance, resource
return
}
// Special case: 100% threshold effectively disables alerts for this metric
if threshold.Trigger >= 100 {
log.Debug().
Str("resource", resourceName).
Str("metric", metricType).
Float64("value", value).
Float64("threshold", threshold.Trigger).
Msg("Alert disabled (100% threshold)")
// Clear any existing alert for this metric
alertID := fmt.Sprintf("%s-%s", resourceID, metricType)
m.mu.Lock()
if _, exists := m.activeAlerts[alertID]; exists {
delete(m.activeAlerts, alertID)
log.Info().
Str("alertID", alertID).
Str("resource", resourceName).
Str("metric", metricType).
Msg("Cleared alert - threshold set to 100%")
// Notify resolution
if m.onResolved != nil {
go m.onResolved(alertID)
}
}
m.mu.Unlock()
return
}
log.Debug().
Str("resource", resourceName).
Str("metric", metricType).