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: Align log_line_prefix in CI and TAP tests with pg_regress.c
Date: Mon, 30 Jun 2025 05:10:59 +0000
Message-ID: <E1uW6nD-003zIn-1s@gemulon.postgresql.org> (raw)

Align log_line_prefix in CI and TAP tests with pg_regress.c

log_line_prefix is changed to include "%b", the backend type in the TAP
test configuration.  %v and %x are removed from the CI configuration,
with the format around %b changed.

The lack of backend type in postgresql.conf set by Cluster.pm for the
TAP test configuration was something that has been bugging me, beginning
the discussion that has led to this change.  The change in the CI has
come up during the discussion, to become consistent with pg_regress.c,
%v and %x not being that useful to have.

Reviewed-by: Andres Freund <andres@anarazel.de>
Reviewed-by: Fujii Masao <masao.fujii@gmail.com>
Reviewed-by: Daniel Gustafsson <daniel@yesql.se>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/aC0VaIWAXLgXcHVP@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5ba00e175a4eaefa4dc38ea14c667bbeb13af305

Modified Files
--------------
src/test/perl/PostgreSQL/Test/Cluster.pm | 2 +-
src/tools/ci/pg_ci_base.conf             | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)



Message-ID: <E1uW6nD-003zIn-1s@gemulon.postgresql.org>
Permalink:  ../E1uW6nD-003zIn-1s@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1uW6nD-003zIn-1s@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: Align log_line_prefix in CI and TAP tests with pg_regress.c
  In-Reply-To: <E1uW6nD-003zIn-1s@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