From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Improve handling and logging of TAP tests for pg_upgrade |
Date: | 2022-04-01 03:47:31 |
Message-ID: | E1na8G2-000FU1-ME@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Improve handling and logging of TAP tests for pg_upgrade
This commit includes a set of improvements to help with the debugging of
failures in these new TAP tests:
- Instead of a plain diff command to compare the dumps generated, use
File::Compare::compare for the same effect. diff is still used to
provide more context in the event of an error.
- Log the contents of regression.diffs if the pg_regress command fails.
- Unify the format of the logs generated, getting inspiration from the
style used in 027_stream_regress.pl.
wrasse is the only buildfarm member that has reported a failure until
now after the introduction of 322becb, complaining that the dumps
generated do not match, and I am lacking information to understand what
is going in this environment.
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/73db8f4d17ed4efb7709f1cafd5b1dd0285b0842
Modified Files
--------------
src/bin/pg_upgrade/t/002_pg_upgrade.pl | 40 ++++++++++++++++++++++++++--------
1 file changed, 31 insertions(+), 9 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2022-04-01 04:30:40 | pgsql: Add macros in hash and btree AMs to get the special area of thei |
Previous Message | Michael Paquier | 2022-04-01 01:15:24 | pgsql: Switch the regression tests of pg_upgrade to use TAP tests |