When doing a soft shutdown, we won't be making new connections anymore so there's no point in keeping the namespace file descriptors open anymore.
Keeping these open effectively makes it impossible to properly clean up namespaces which are no longer used in the new configuration until all previously opened connections are closed in the old worker process. This change introduces a cleanup function that is called during soft shutdown that closes all namespace file descriptors by iterating over the namespace ebtree. --- src/namespace.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/src/namespace.c b/src/namespace.c index cfb81ba0f..e3c46e67e 100644 --- a/src/namespace.c +++ b/src/namespace.c @@ -15,6 +15,7 @@ #include <common/hash.h> #include <common/errors.h> #include <proto/log.h> +#include <proto/signal.h> #include <types/global.h> /* Opens the namespace <ns_name> and returns the FD or -1 in case of error @@ -39,6 +40,24 @@ static int init_default_namespace() static struct eb_root namespace_tree_root = EB_ROOT; +static void netns_sig_stop(struct sig_handler *sh) { + struct ebpt_node *node, *next; + struct netns_entry *entry; + + /* close namespace file descriptors and remove registered namespaces from the + * tree when stopping */ + node = ebpt_first(&namespace_tree_root); + while (node) { + next = ebpt_next(node); + ebpt_delete(node); + entry = container_of(node, struct netns_entry, node); + ha_warning("Closing namespace %s.\n", (char *)entry->node.key); + free(entry->node.key); + close(entry->fd); + node = next; + } +} + int netns_init(void) { int err_code = 0; @@ -55,6 +74,8 @@ int netns_init(void) } } + signal_register_fct(0, netns_sig_stop, 0); + return err_code; } -- 2.23.0