BUG/MINOR: server: incorrect report for tracking servers leaving drain
authorAurelien DARRAGON <adarragon@haproxy.com>
Mon, 27 Mar 2023 08:17:31 +0000 (10:17 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 21 Apr 2023 12:36:45 +0000 (14:36 +0200)
commit9a1df02ccb0f40bf93ab7d654363e8706851a81e
tree4ebc920809a581f805728e60feaf8f6900f74781
parent2dac67af7d0213604f6bfb18e179fba711bb8cb1
BUG/MINOR: server: incorrect report for tracking servers leaving drain

Report message for tracking servers completely leaving drain is wrong:

The check for "leaving drain .. via" never evaluates because the
condition !(s->next_admin & SRV_ADMF_FDRAIN) is always true in the
current block which is guarded by !(s->next_admin & SRV_ADMF_DRAIN).

For tracking servers that leave inherited drain mode, this results in the
following message being emitted:
  "Server x/b is UP (leaving forced drain)"

Instead of:
  "Server x/b is UP (leaving drain) via x/a"

To this fix: we check if FDRAIN is currently set, else it means that the
drain status is inherited from the tracked server (IDRAIN)

This regression was introduced with 64cc49cf ("MAJOR: servers: propagate server
status changes asynchronously."), thus it may be backported to every stable
versions.
src/server.c