BUG/MEDIUM: stats/server: use watcher to track server during stats dump
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Wed, 23 Oct 2024 09:33:34 +0000 (11:33 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 11 Dec 2024 12:57:48 +0000 (13:57 +0100)
commit4e932f79bea653157516a4c41ee26b32f22c470c
treea90f489a19db79727c891e6e958358d286d06c4e
parentb4501697a6321e990a94243c0274b4294dcbbd6b
BUG/MEDIUM: stats/server: use watcher to track server during stats dump

If a server A is deleted while a stats dump is currently on it, deletion
is delayed thanks to reference counting. Server A is nonetheless removed
from the proxy list. However, this list is a single linked list. If the
next server B is deleted and freed immediately, server A would still
point to it. This problem has been solved by the prev_deleted list in
servers.

This model seems correct, but it is difficult to ensure completely its
validity. In particular, it implies when stats dump is resumed, server A
elements will be accessed despite the server being in a half-deleted
state.

Thus, it has been decided to completely ditch the refcount mechanism for
stats dump. Instead, use the watcher element to register every stats
dump currently tracking a server instance. Each time a server is deleted
on the CLI, each stats dump element which may points to it are updated
to access the next server instance, or NULL if this is the last server.
This ensures that a server which was deleted via CLI but not completely
freed is never accessed on stats dump resumption.

Currently, no race condition related to dynamic servers and stats dump
is known. However, as described above, the previous model is deemed too
fragile, as such this patch is labelled as bug-fix. It should be
backported up to 2.6, after a reasonable period of observation. It
relies on the following patch :
  MINOR: list: define a watcher type

(cherry picked from commit 071ae8ce3d1a318d2227fad2ebf63e78a05815f0)
Signed-off-by: Willy Tarreau <w@1wt.eu>
include/haproxy/applet-t.h
include/haproxy/server-t.h
include/haproxy/stats-t.h
src/http_ana.c
src/server.c
src/stats-html.c
src/stats-proxy.c
src/stats.c