pgsql: Further harden tests that might use not-so-compatible tar versio

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Further harden tests that might use not-so-compatible tar versio
Date: 2026-04-02 21:21:45
Message-ID: E1w8PU0-002f7J-1N@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Further harden tests that might use not-so-compatible tar versions.

Buildfarm testing shows that OpenSUSE (and perhaps related platforms?)
configures GNU tar in such a way that it'll archive sparse WAL files
by default, thus triggering the pax-extension detection code added by
bc30c704a. Thus, we need something similar to 852de579a but for
GNU tar's option set. "--format=ustar" seems to do the trick.

Moreover, the buildfarm shows that pg_verifybackup's 003_corruption.pl
test script is also triggering creation of pax-format tar files on
that platform. We had not noticed because those test cases all fail
(intentionally) before getting to the point of trying to verify WAL
data.

Since that means two TAP scripts need this option-selection logic, and
plausibly more will do so in future, factor it out into a subroutine
in Test::Utils. We also need to back-patch the 003_corruption.pl fix
into v18, where it's also failing.

While at it, clean up some places where guards for $tar being empty
or undefined were incomplete or even outright backwards. Presumably,
we missed noticing because the set of machines that run TAP tests
and don't have tar installed is empty. But if we're going to try
to handle that scenario, we should do it correctly.

Reported-by: Tomas Vondra <tomas(at)vondra(dot)me>
Author: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Discussion: https://postgr.es/m/02770bea-b3f3-4015-8a43-443ae345379c@vondra.me
Backpatch-through: 18

Branch
------
REL_18_STABLE

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

Modified Files
--------------
src/bin/pg_verifybackup/t/003_corruption.pl | 25 ++++++++++++----
src/test/perl/PostgreSQL/Test/Utils.pm | 44 +++++++++++++++++++++++++++++
2 files changed, 64 insertions(+), 5 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Fujii Masao 2026-04-03 00:45:20 Re: pgsql: Reduce log level of some logical decoding messages from LOG to D
Previous Message Robert Haas 2026-04-02 19:56:26 Re: pgsql: Reduce log level of some logical decoding messages from LOG to D