Fix #745: warning realert window uses repeat_alert, not time_warning_fail_length - #805
Fix #745: warning realert window uses repeat_alert, not time_warning_fail_length#805bmfmancini wants to merge 2 commits into
Conversation
…ning_fail_length The type 2 warning re-alert window was computed from time_warning_fail_length (the warning time-window length) instead of repeat_alert (the configured re-alert interval). The $ra guard's truthy test used the same wrong field. This caused warning re-alerts to fire at a cadence controlled by the wrong setting. The alert path (thold_functions.php:3136) already uses repeat_alert correctly; this aligns the warning path to match.
somethingwithproof
left a comment
There was a problem hiding this comment.
The repeat_alert correction is directionally correct, but this branch also carries three unrelated threshold_value edits from #789. Please remove those hunks so this PR contains only #745, then add Cacti-Composer/Pest coverage for initial warning, no early re-alert, and re-alert at the configured interval. A syntax check cannot validate this state machine, and the current integration matrix is red.
|
CI is failing during Composer dependency downloads with GitHub HTTP 429 responses. This branch also carries the older unpinned integration workflow. Maintainer edits are disabled, so I cannot apply the repository’s pinned-Cacti and bounded-retry workflow fix. Please enable maintainer edits or update/rebase the branch; I can rerun and finish the CI repair immediately afterward. |
Summary
Fixes #745.
The type 2 (time-based) warning re-alert window was computed from
time_warning_fail_length(the warning time-window length) instead ofrepeat_alert(the configured re-alert interval). The$raguard's truthy test used the same wrong field.Impact
Warning re-alerts fired at a cadence controlled by the wrong setting (
time_warning_fail_length) rather than the configured re-alert interval (repeat_alert).Fix
Both the
$realerttimecomputation and the$raguard now userepeat_alert, matching the alert path atthold_functions.php:3136.Testing
php -l thold_functions.php— no syntax errors.