pgsql: Switch TAP tests of pg_rewind to use non-superuser role,take tw - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Switch TAP tests of pg_rewind to use non-superuser role,take tw
Date
Msg-id E1hFboH-0002CN-1C@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Switch TAP tests of pg_rewind to use non-superuser role, take two

Up to now the tests of pg_rewind have been using a superuser for all its
tests (which is the default of many tests actually, and something that
ought to be reviewed) when involving an online source server, still it
is possible to use a non-superuser role to do that as long as this role
is granted permissions to execute all the source-side functions used for
the rewind.  This is possible since v11, and was already documented as
of bfc8068.

PostgresNode::init is extended so as callers of this routine can add
extra options to configure the authentication of a new node, which gets
used by this commit, and allows the tests to work properly on Windows
where SSPI is used.

This will allow to catch up easily any change in pg_rewind if the tool
begins to use more backend-side functions, so as the properties
introduced by v11 are kept.

Per suggestion from Peter Eisentraut.

Author: Michael Paquier
Reviewed-by: Magnus Hagander
Discussion: https://postgr.es/m/20190411041336.GM2728@paquier.xyz

Branch
------
master

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

Modified Files
--------------
src/bin/pg_rewind/t/RewindTest.pm | 23 ++++++++++++++++++++++-
src/test/perl/PostgresNode.pm     |  3 ++-
2 files changed, 24 insertions(+), 2 deletions(-)


pgsql-committers by date:

Previous
From: Noah Misch
Date:
Subject: pgsql: MSYS: Skip src/test/recovery/t/017_shm.pl.
Next
From: Michael Paquier
Date:
Subject: Re: pgsql: Switch TAP tests of pg_rewind to use a role with minimalpermiss