]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: checks: servers must not start in slowstart mode
authorWilly Tarreau <w@1wt.eu>
Mon, 16 Dec 2013 17:08:36 +0000 (18:08 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 16 Dec 2013 17:08:36 +0000 (18:08 +0100)
commit02541e8be22c212493ceeb88a54ccd32190db36d
treefbc85efa996489252e74bb8fae1cb2ac08bdd59d
parente38feed966070142e1fd5d3033c9e999a9ffe83c
BUG/MEDIUM: checks: servers must not start in slowstart mode

In 1.5-dev20, commit bb9665e (BUG/MEDIUM: checks: ensure we can enable
a server after boot) tried to fix a side effect of having both regular
checks and agent checks condition the up state propagation to servers.

Unfortunately it was still not fine because after this fix, servers
which make use of slowstart start in this mode. We must not check
the agent's health if agent checks are not enabled, and likewise,
we must not check the regular check's health if they are not enabled.

Reading the code, it seems like we could avoid entering this function
at all if (s->state & SRV_RUNNING) is not satisfied. Let's reserve
this for a later patch if needed.

Thanks to Sander Klein for reporting this abnormal situation.
src/checks.c