Use a longer connection timeout in pg_isready test.
authorTom Lane
Sun, 1 Oct 2017 16:43:47 +0000 (12:43 -0400)
committerTom Lane
Sun, 1 Oct 2017 16:43:47 +0000 (12:43 -0400)
Buildfarm members skink and sungazer have both recently failed this
test, with symptoms indicating that the default 3-second timeout
isn't quite enough for those very slow systems.  There's no reason
to be miserly with this timeout, so boost it to 60 seconds.

Back-patch to all versions containing this test.  That may be overkill,
because the failure has only been observed in the v10 branch, but
I don't feel like having to revisit this later.

src/bin/scripts/t/080_pg_isready.pl

index f432505d5cd390a549351244a0c6665ea8864aae..2ddb03d7c24b3160514a686bb8688dd991b80dfc 100644 (file)
@@ -12,4 +12,5 @@ command_fails(['pg_isready'], 'fails with no server running');
 my $tempdir = tempdir;
 start_test_server $tempdir;
 
-command_ok(['pg_isready'], 'succeeds with server running');
+# use a long timeout for the benefit of very slow buildfarm machines
+command_ok([qw(pg_isready --timeout=60)], 'succeeds with server running');