]> git.ipfire.org Git - thirdparty/haproxy.git/commit
MINOR: checks: simplify and improve reporting of state changes when using log-health...
authorWilly Tarreau <w@1wt.eu>
Tue, 13 May 2014 19:01:39 +0000 (21:01 +0200)
committerWilly Tarreau <w@1wt.eu>
Tue, 13 May 2014 20:01:28 +0000 (22:01 +0200)
commitbef1b32c4e9dab4eb7ec9c34ae37edc955df4c7c
tree13cece5b87413a978d86a1391d8399cf7dc8840d
parentd03fdf41ec966eabcdf71323d389d301541c6cca
MINOR: checks: simplify and improve reporting of state changes when using log-health-checks

Function set_server_check_status() is very weird. It is called at the
end of a check to update the server's state before the new state is even
calculated, and possibly to log status changes, only if the proxy has
"option log-health-checks" set.

In order to do so, it employs an exhaustive list of the combinations
which can lead to a state change, while in practice almost all of
them may simply be deduced from the change of check status. Better,
some changes of check status are currently not detected while they
can be very valuable (eg: changes between L4/L6/TOUT/HTTP 500 for
example).

The doc was updated to reflect this.

Also, a minor change was made to consider s->uweight and not s->eweight
as meaning "DRAIN" since eweight can be null without the DRAIN mode (eg:
throttle, NOLB, ...).
doc/configuration.txt
src/checks.c