diff options
author | Oran Agra <oran@redislabs.com> | 2020-11-22 21:21:42 +0200 |
---|---|---|
committer | Oran Agra <oran@redislabs.com> | 2020-12-06 14:54:34 +0200 |
commit | 5b44631397787a65327fcab77f7df37862286ed9 (patch) | |
tree | b824f5b2d98be660cc076929b06906bbbec34b78 /tests/test_helper.tcl | |
parent | c31055db617a7fedb20c28c245ddad8ed2b11931 (diff) | |
download | redis-5b44631397787a65327fcab77f7df37862286ed9.tar.gz |
testsuite: fix fd leak, prevent port clashing when using --baseport
when using --baseport to run two tests suite in parallel (different
folders), we need to also make sure the port used by the testsuite to
communicate with it's workers is unique. otherwise the attept to find a
free port connects to the other test suite and messes it.
maybe one day we need to attempt to bind, instead of connect when tring
to find a free port.
Diffstat (limited to 'tests/test_helper.tcl')
-rw-r--r-- | tests/test_helper.tcl | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/test_helper.tcl b/tests/test_helper.tcl index a1fae8d75..29ebdd7bb 100644 --- a/tests/test_helper.tcl +++ b/tests/test_helper.tcl @@ -180,6 +180,10 @@ proc reconnect {args} { set port [dict get $srv "port"] set config [dict get $srv "config"] set client [redis $host $port 0 $::tls] + if {[dict exists $srv "client"]} { + set old [dict get $srv "client"] + $old close + } dict set srv "client" $client # select the right db when we don't have to authenticate @@ -257,7 +261,7 @@ proc test_server_main {} { set tclsh [info nameofexecutable] # Open a listening socket, trying different ports in order to find a # non busy one. - set clientport [find_available_port 11111 32] + set clientport [find_available_port [expr {$::baseport - 32}] 32] if {!$::quiet} { puts "Starting test server at port $clientport" } |