From 7542eadc9f073cbcb5e1cb1cb33fe451560bd31d Mon Sep 17 00:00:00 2001 From: David Steele Date: Mon, 2 Aug 2021 14:48:31 -0400 Subject: [PATCH] Increase timeouts in storage/remote test. There have been intermittent failures on f33 (with coverage) but not on u16 (without coverage). Reproducing this reliably has been very difficult, so just try increasing the timeouts. This is based on the observation that tests with coverage take longer than tests without, which may lead the f33 tests to fail if CI is running slower than usual. This will not increase the runtime of the test unless there is an error. --- test/src/module/storage/remoteTest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/src/module/storage/remoteTest.c b/test/src/module/storage/remoteTest.c index d5040407e..fe697c8c7 100644 --- a/test/src/module/storage/remoteTest.c +++ b/test/src/module/storage/remoteTest.c @@ -37,7 +37,7 @@ testRun(void) // ordering the second remote will never be sent an explicit exit and may not save coverage data. StringList *argList = strLstNew(); hrnCfgArgRawZ(argList, cfgOptStanza, "db"); - hrnCfgArgRawZ(argList, cfgOptProtocolTimeout, "10"); + hrnCfgArgRawZ(argList, cfgOptProtocolTimeout, "20"); hrnCfgArgRawZ(argList, cfgOptBufferSize, "16384"); hrnCfgArgKeyRawZ(argList, cfgOptPgPath, 1, TEST_PATH "/pg1"); hrnCfgArgKeyRawZ(argList, cfgOptPgHost, 2, "localhost"); @@ -51,7 +51,7 @@ testRun(void) // Load configuration and get repo remote storage argList = strLstNew(); hrnCfgArgRawZ(argList, cfgOptStanza, "db"); - hrnCfgArgRawZ(argList, cfgOptProtocolTimeout, "10"); + hrnCfgArgRawZ(argList, cfgOptProtocolTimeout, "20"); hrnCfgArgRawFmt(argList, cfgOptBufferSize, "%zu", ioBufferSize()); hrnCfgArgKeyRawZ(argList, cfgOptPgPath, 1, TEST_PATH "/pg"); hrnCfgArgRawZ(argList, cfgOptRepoHost, "localhost");