diff options
| -rw-r--r-- | internal/html.go | 9 | ||||
| -rw-r--r-- | internal/state.go | 3 |
2 files changed, 7 insertions, 5 deletions
diff --git a/internal/html.go b/internal/html.go index 753a5b0..1ccb894 100644 --- a/internal/html.go +++ b/internal/html.go @@ -173,10 +173,11 @@ func (s state) htmlReportUnhandledContent(sb *strings.Builder) { } // htmlReportStaleAlerts generates HTML for stale checks. -// Mirrors state.reportStaleAlerts() from state.go:216-220. +// Only reports stale alerts that are not OK, since stale OK alerts aren't concerning. +// Mirrors state.reportStaleAlerts() from state.go. func (s state) htmlReportStaleAlerts(sb *strings.Builder) int { return s.htmlReportBy(sb, false, true, func(cs checkState) bool { - return cs.Epoch < s.staleEpoch + return cs.Epoch < s.staleEpoch && cs.Status != nagiosOk }) } @@ -228,11 +229,11 @@ func (s state) htmlReportBy(sb *strings.Builder, showStatusChange, isStaleReport return } -// countStale counts the number of stale checks. +// countStale counts the number of stale checks (excluding OK status). // Helper function for generating summary counts. func (s state) countStale() int { return s.countBy(func(cs checkState) bool { - return cs.Epoch < s.staleEpoch + return cs.Epoch < s.staleEpoch && cs.Status != nagiosOk }) } diff --git a/internal/state.go b/internal/state.go index d71c71c..516e63a 100644 --- a/internal/state.go +++ b/internal/state.go @@ -214,8 +214,9 @@ func (s state) reportUnhandled(sb *strings.Builder) (numCriticals, numWarnings, } func (s state) reportStaleAlerts(sb *strings.Builder) int { + // Only report stale alerts that are not OK, since stale OK alerts aren't concerning return s.reportBy(sb, false, true, func(cs checkState) bool { - return cs.Epoch < s.staleEpoch + return cs.Epoch < s.staleEpoch && cs.Status != nagiosOk }) } |