> On Dec 23, 2017, at 2:24 AM, Willy Tarreau <w...@1wt.eu> wrote: > > Hi guys, > > On Sat, Dec 23, 2017 at 08:58:43AM +0100, Cyril Bonté wrote: >> It looks to be a code regression. >> >> Emeric, can you have a look at commit 5a1335110c ? It seems there was an >> unwanted change in the function call : srv_set_stopping() was replaced by >> srv_set_running() >> [...] >> /* Marks the check <check> as valid and tries to set its server into >> stopping mode >> @@ -406,7 +371,7 @@ static void check_notify_stopping(struct check *check) >> if ((s->agent.state & CHK_ST_ENABLED) && (s->agent.health < >> s->agent.rise)) >> return; >> >> - srv_set_stopping(s, (!s->track && !(s->proxy->options2 & >> PR_O2_LOGHCHKS)) >> ? check_reason_string(check) : NULL); >> + srv_set_running(s, NULL, (!s->track && !(s->proxy->options2 & >> PR_O2_LOGHCHKS)) ? check : NULL); > > Nice catch! Thanks Paul for your helpful report and Cyril for spotting > the bug. Now fixed, I can prepare 1.8.2 :-) > > Willy
Thanks all for getting this fixed so dang quickly on a Friday night before a holiday weekend. It looks good to me now. Much appreciated. -Paul