tree 066ef8f064199b26e9a62e23001c908ae9854063
parent affe0cb7d29e9a625c114b2113152eeb6bdf9344
author Emeric Brun <ebrun@haproxy.com> 1594388831 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1594975494 +0200

BUG/MEDIUM: log: issue mixing sampled to not sampled log servers.

A boolean was mistakenly declared 'static THREAD_LOCAL' causing
the probe of a log to a 'not sampled' log server conditionned by
the last evaluated 'sampled log' server test on the same thread.

This results to unpredictable drops of logs on 'not sampled'
log servers as soon a 'sampled' log server is declared.

This patch removes the static THREAD_LOCAL attribute from this
boolean, fixing the issue and allowing to mix 'sampled' and
'not sampled' servers.

This fix should be backported in any branches which includes
the log sampling feature.

(cherry picked from commit 2f4cc28e0f1c8c3deca8b1b7fa446c6523dcc93c)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 9ce61f6d578876f1eda258d4cfad997e09c140a4)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit b2bb526fa46b5e2119683d60e9a7ddefad801965)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>