pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Michael Paquier <michael@paquier.xyz>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix stability issue with new TAP test of pg_createsubscriber
Date: Fri, 16 Jan 2026 03:13:21 +0000
Message-ID: <E1vgaH2-000jLd-12@gemulon.postgresql.org> (raw)

Fix stability issue with new TAP test of pg_createsubscriber

The test introduced in 639352d904c8 has added a direct pg_ctl command to
start a node, a method that is incompatible with the teardown() routine
used at the end of the test as the PID saved in the Cluster object would
prevent the node to be shut down.  This can ultimately prevent the test
to perform its cleanup, failing on timeout.

Like pg_ctl's 001_start_stop or ssl_passphrase_callback's 001_testfunc,
this commit changes the test so a direct pg_ctl command is used to stop
the rogue node.  That should be hopefully enough to cool down the
buildfarm.

Per report from buildfarm member fairywren, which is the only animal
that is showing this issue.

Author: Hayato Kuroda <kuroda.hayato@fujitsu.com>
Discussion: https://postgr.es/m/TY7PR01MB1455452AE9053DD2B77B74FEAF58CA@TY7PR01MB14554.jpnprd01.prod.outlook.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a7c63e486050636925f81123b1427f1cb49644a2

Modified Files
--------------
src/bin/pg_basebackup/t/040_pg_createsubscriber.pl | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)



Message-ID: <E1vgaH2-000jLd-12@gemulon.postgresql.org>
Permalink:  ../E1vgaH2-000jLd-12@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1vgaH2-000jLd-12@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: michael@paquier.xyz, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix stability issue with new TAP test of pg_createsubscriber
  In-Reply-To: <E1vgaH2-000jLd-12@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox