Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres
Date
Msg-id YFqRGvGasd8RyltU@paquier.xyz
Whole thread Raw
In response to Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres  (Christoph Berg <myon@debian.org>)
Responses Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres  (Christoph Berg <myon@debian.org>)
List pgsql-hackers
On Tue, Mar 23, 2021 at 12:50:29PM +0100, Christoph Berg wrote:
> I'm working around the problem now by running the tests as user
> "postgres", but does completely break in environments where users want
> to run the testsuite from a separate compilation user but don't have root.
>
> Old code:
https://salsa.debian.org/postgresql/postgresql/-/blob/8b1217fcae3e64155bc35517acbd50c6f166d997/debian/tests/installcheck
> Workaround:
https://salsa.debian.org/postgresql/postgresql/-/blob/cbc0240bec738b6ab3b61c498825b82c8ff21a70/debian/tests/installcheck

So you basically mimicked the makefile rule that this commit removed
into your own test suite.  Reverting the change does not really help,
because we'd be back to square one where there would be problems in
parallel runs for developers.  Saying that, I would not mind adding an
option to pg_regress to control if this cleanup code is triggered or
not, say something like --no-tablespace-cleanup?  Then, you could just
pass down the option by yourself before creating your tablespace path
as you wish.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: PoC/WIP: Extended statistics on expressions
Next
From: David Rowley
Date:
Subject: Re: A reloption for partitioned tables - parallel_workers