From be9088b25fb6cd02498626044afcfec241a53a1e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javier=20Jaramago=20Fern=C3=A1ndez?= Date: Fri, 9 Dec 2022 13:47:26 +0100 Subject: [PATCH] Increase sleeps for TAP tests reconfiguring ProxySQL interfaces Sleeps have been increased for tests 'test_clickhouse_server-t' and 'test_sqlite3_server-t' in order to prevent potentials 'EADDRINUSE', that could lead to client stalls when connecting to the interfaces. --- test/tap/tests/test_clickhouse_server-t.cpp | 3 ++- test/tap/tests/test_sqlite3_server-t.cpp | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/test/tap/tests/test_clickhouse_server-t.cpp b/test/tap/tests/test_clickhouse_server-t.cpp index 9280c7ce3..6f7028880 100644 --- a/test/tap/tests/test_clickhouse_server-t.cpp +++ b/test/tap/tests/test_clickhouse_server-t.cpp @@ -497,7 +497,8 @@ int main(int argc, char** argv) { // NOTE: Wait for ProxySQL to reconfigure, changing Clickhous interface. // Trying to perform a connection immediately after changing the // interface could lead to 'EADDRINUSE' in ProxySQL side. - sleep(1); + // UPDATE: Timeout increased to '5' seconds to avoid previously described issue. + sleep(5); // Connect to the new interface std::pair new_host_port {}; diff --git a/test/tap/tests/test_sqlite3_server-t.cpp b/test/tap/tests/test_sqlite3_server-t.cpp index cbeb578c7..79296649a 100644 --- a/test/tap/tests/test_sqlite3_server-t.cpp +++ b/test/tap/tests/test_sqlite3_server-t.cpp @@ -348,7 +348,8 @@ int main(int argc, char** argv) { // NOTE: Wait for ProxySQL to reconfigure, changing SQLite3 interface. // Trying to perform a connection immediately after changing the // interface could lead to 'EADDRINUSE' in ProxySQL side. - sleep(1); + // UPDATE: Timeout increased to '5' seconds to avoid previously described issue. + sleep(5); // Connect to the new interface std::pair new_host_port {};