]> git.ipfire.org Git - thirdparty/haproxy.git/commit
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)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Tue, 10 Dec 2024 15:19:33 +0000 (16:19 +0100)
commit071ae8ce3d1a318d2227fad2ebf63e78a05815f0
tree929479ef3cfc78450a2773479e0a9cb632914e64
parenteafa8a32bb19ee567cb0a60a4042a8ef436e1e94
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
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