MINOR: server: make srv_shutdown_sessions() call pendconn_redistribute()

When shutting down server sessions, the queue was not considered, which
is a problem if some element reached the queue at the moment the server
was going down, because there will be no more requests to kick them out
of it. Let's always make sure we scan the queue to kick these streams
out of it and that they can possibly find a more suitable server. This
may make a difference in the time it takes to shut down a server on the
CLI when lots of servers are in the queue.

It might be interesting to backport this to 3.0 but probably not much
further.

diff --git a/src/server.c b/src/server.c
index c0795c0a55a020f5b57fe7b26b63dcc344a3d2f6..58862c5a17b5f8ee3d0e731d05a1e8d7a93ab9a4 100644 (file)
--- a/src/server.c
+++ b/src/server.c
@@ -2027,6 +2027,9 @@ void srv_shutdown_streams(struct server *srv, int why)
                MT_LIST_FOR_EACH_ENTRY_LOCKED(stream, &srv->per_thr[thr].streams, by_srv, back)
                        if (stream->srv_conn == srv)
                                stream_shutdown(stream, why);
+
+       /* also kill the possibly pending streams in the queue */
+       pendconn_redistribute(srv);
 }
 
 /* Shutdown all connections of all backup servers of a proxy. The caller must