pgsql: Handle timeout in PostgreSQL::Test::Cluster::is_alive()

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Handle timeout in PostgreSQL::Test::Cluster::is_alive()
Date: 2025-07-29 08:03:46
Message-ID: E1ugfJK-001HM6-0Q@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Handle timeout in PostgreSQL::Test::Cluster::is_alive()

This commit adds an extra --timeout=PG_TEST_TIMEOUT_DEFAULT to the call
of pg_isready done in is_alive(), so as it is possible to have more
leverage with the call on machines constrained on resources.

By default the timeout is 180s, and it can be changed depending on the
environment where the tests are run.

Per buildfarm member mamba, where the default timeout of 3s used by
pg_isready has proved that it may not be enough as the postmaster may
not have the time it needs to reply to a ping request.

Reported-by: Alexander Lakhin <exclusion(at)gmail(dot)com>
Reviewed-by: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
Discussion: https://postgr.es/m/29b637df-f818-4b52-986a-f11ba28300e9@gmail.com

Branch
------
master

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

Modified Files
--------------
src/test/perl/PostgreSQL/Test/Cluster.pm | 1 +
1 file changed, 1 insertion(+)

Browse pgsql-committers by date

  From Date Subject
Next Message Fujii Masao 2025-07-29 10:44:14 pgsql: Add regression test for background worker restart after crash.
Previous Message Alexander Korotkov 2025-07-29 07:44:16 pgsql: Clarify documentation for the initcap function