BUG/MINOR: checks: properly handle wrapping time in __health_adjust()
authorWilly Tarreau <w@1wt.eu>
Wed, 17 Feb 2021 14:15:15 +0000 (15:15 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 18 Feb 2021 09:06:45 +0000 (10:06 +0100)
commit64ba5ebadcd5d98e00989d08dfaa3c94c15196c9
tree3e5ce7374c93e4c9c239cbff9daea9b546e1c871
parent36441f46c42de1297398051457c53baa95fae11e
BUG/MINOR: checks: properly handle wrapping time in __health_adjust()

There's an issue when a server state changes, we use an integer comparison
to decide whether or not to reschedule a test instead of using a wrapping
timer comparison. This will cause some health-checks not to be immediately
triggered half of the time, and some unneeded calls to task_queue() to be
performed in other cases.

This bug has always been there as it was introduced with the commit that
added the feature, 97f07b832 ("[MEDIUM] Decrease server health based on
http responses / events, version 3"). This may be backported everywhere.
src/check.c