Skip to content

Commit f0c1c73

Browse files
authored
test suite: clean server pids after server crashed (redis#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.
1 parent b784c53 commit f0c1c73

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

tests/support/server.tcl

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,10 @@ proc kill_server config {
6060
}
6161

6262
check_sanitizer_errors [dict get $config stderr]
63+
64+
# Remove this pid from the set of active pids in the test server.
65+
send_data_packet $::test_server_fd server-killed $pid
66+
6367
return
6468
}
6569

0 commit comments

Comments
 (0)