tree 9edeb677aac8464f1360f044074e64c258854a79
parent b7e226eabf84e0dbafbdb4ffc68d7d6363eb6c16
author Amaury Denoyelle <adenoyelle@haproxy.com> 1623683065 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1624006862 +0200

BUG/MINOR: stick-table: insert srv in used_name tree even with fixed id

If the server id is fixed in the configuration, it is immediately
inserted in the 'used_server_id' backend tree via srv_parse_id. On
check_config_validity, the dynamic id generation is thus skipped for
fixed-id servers. However, it must nevertheless be inserted in the
'used_server_name' backend tree.

This bug seems to be not noticeable for the user. Indeed, before the
fix, the search in sticking_rule_find_target always returned NULL for
the name, then the fallback search with server id succeeded, so the
persistence is properly applied. However with the fix the fallback
search is not executed anymore, which saves from the locking of
STK_SESS.

This should be backported up to 2.0.

(cherry picked from commit 077c6b8d29ba63a625648ebb73bf151a793f0096)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 50779aefca2b685bf6e2c922d02bfab59195534a)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 7189da52e78eebe115b2845400dc596966cef7c0)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit dc02994ef2431345011b8b37de34d6b074f94b67)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>