From f0c1c730d495107c22bf722fa37d592f6c535feb Mon Sep 17 00:00:00 2001 From: Oran Agra Date: Fri, 13 Oct 2023 16:28:52 +0300 Subject: [PATCH] test suite: clean server pids after server crashed (#12639) when a server in the test suite crashes and is restarted by redstart_server, we didn't clean it's pid from the list. we can see that when the corrupt-dump-fuzzer hangs, it has a long list of servers to lean, but in fact they're all already dead. --- tests/support/server.tcl | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/support/server.tcl b/tests/support/server.tcl index 79bc6b5b2..018c659e9 100644 --- a/tests/support/server.tcl +++ b/tests/support/server.tcl @@ -60,6 +60,10 @@ proc kill_server config { } check_sanitizer_errors [dict get $config stderr] + + # Remove this pid from the set of active pids in the test server. + send_data_packet $::test_server_fd server-killed $pid + return }