Re: pgsql: Migrate logical slots to the new node during an upgrade.

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>, Amit Kapila <akapila(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Migrate logical slots to the new node during an upgrade.
Date: 2023-11-06 14:43:21
Message-ID: 7b8a9460-5668-b372-04e6-7b52e9308493@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


On 2023-11-06 Mo 05:31, Peter Eisentraut wrote:
> On 26.10.23 04:14, Amit Kapila wrote:
>> Migrate logical slots to the new node during an upgrade.
>
> Something in this patch is causing the pg_upgrade test to be run in
> the source directory and leaving files such as delete_old_cluster.sh
> around there.  (This is under meson; there shouldn't be any new files
> appearing in the source directory during the build.)  Check please.
>
>
>

The same probably applies to VPATH builds using Makefiles.

002_pg_upgrade.pl has this, which I think should cure the new script:

# In a VPATH build, we'll be started in the source directory, but we want
# to run pg_upgrade in the build directory so that any files generated
finish
# in it, like delete_old_cluster.{sh,bat}.
chdir ${PostgreSQL::Test::Utils::tmp_check};

cheers

andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2023-11-06 15:04:38 pgsql: Remove distprep
Previous Message Noah Misch 2023-11-06 14:17:12 pgsql: Set GUC "is_superuser" in all processes that set AuthenticatedUs