]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: resolvers: Add a task on servers to check SRV resolution status
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 15 Jun 2021 14:17:17 +0000 (16:17 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 17 Jun 2021 14:52:35 +0000 (16:52 +0200)
commitdcac41806239d4f3b2de5e4d1bacf9c03ca99efb
treece34bf83fabcfaae9cb7514e1377dc12b8ad0e3a
parent73001ab6e39fb25ff406bbcd3809d0a1fa0131b5
BUG/MEDIUM: resolvers: Add a task on servers to check SRV resolution status

When a server relies on a SRV resolution, a task is created to clean it up
(fqdn/port and address) when the SRV resolution is considered as outdated
(based on the resolvers 'timeout' value). It is only possible if the server
inherits outdated info from a state file and is no longer selected to be
attached to a SRV item. Note that most of time, a server is attached to a
SRV item. Thus when the item becomes obsolete, the server is cleaned
up.

It is important to have such task to be sure the server will be free again
to have a chance to be resolved again with fresh information. Of course,
this patch is a workaround to solve a design issue. But there is no other
obvious way to fix it without rewritting all the resolvers part. And it must
be backportable.

This patch relies on following commits:
 * MINOR: resolvers: Clean server in a dedicated function when removing a SRV item
 * MINOR: resolvers: Remove server from named_servers tree when removing a SRV item

All the series must be backported as far as 2.2 after some observation
period. Backports to 2.0 and 1.8 must be evaluated.
include/haproxy/server-t.h
src/resolvers.c
src/server.c
src/server_state.c