happyDomain/checker-ping
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

checker: report rules with no targets as StatusUnknown

Browse source 
Skipped tests that are not problematic should be UNKNOWN rather than
INFO; "no targets to ping" is a non-evaluation, not a finding.
This commit is contained in:
nemunaire 2026-04-26 01:25:33 +07:00
commit 9972ef512f
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
checker/rule.go
View file

@ -124,7 +124,7 @@ func (r *pingRule) Evaluate(ctx context.Context, obs sdk.ObservationGetter, opts
results := Evaluate(&data, warningRTT, criticalRTT, warningPacketLoss, criticalPacketLoss) results := Evaluate(&data, warningRTT, criticalRTT, warningPacketLoss, criticalPacketLoss)
if len(results) == 0 { if len(results) == 0 {
return []sdk.CheckState{{ return []sdk.CheckState{{
Status: sdk.StatusInfo, Status: sdk.StatusUnknown,
Message: "No targets to ping", Message: "No targets to ping",
Code: "ping_no_targets", Code: "ping_no_targets",
}} }}