From 0806cd29be01c837bc2f6b07bf14fb6aa0f4ad80 Mon Sep 17 00:00:00 2001 From: Noah Misch Date: Fri, 1 Jul 2022 18:27:18 -0700 Subject: [PATCH] Fix race condition in t/028_pitr_timelines.pl. Per buildfarm members sungazer and mylodon. Back-patch to v15, which introduced this test. Discussion: https://postgr.es/m/20220627070457.GA2176699@rfd.leadboat.com --- src/test/recovery/t/028_pitr_timelines.pl | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/test/recovery/t/028_pitr_timelines.pl b/src/test/recovery/t/028_pitr_timelines.pl index a8b12d9af6..bad02edf90 100644 --- a/src/test/recovery/t/028_pitr_timelines.pl +++ b/src/test/recovery/t/028_pitr_timelines.pl @@ -140,6 +140,13 @@ is($result, qq{1}, "check table contents after point-in-time recovery"); # back to this timeline. $node_pitr->safe_psql('postgres', "INSERT INTO foo VALUES(3);"); +# Wait for the archiver to be running. The startup process might have yet to +# exit, in which case the postmaster has not started the archiver. If we +# stop() without an archiver, the archive will be incomplete. +$node_pitr->poll_query_until('postgres', + "SELECT true FROM pg_stat_activity WHERE backend_type = 'archiver';") + or die "Timed out while waiting for archiver to start"; + # Stop the node. This archives the last segment. $node_pitr->stop();