Re: [PATCH] Fix pg_upgrade test from v10

From: "Anton A(dot) Melnikov" <aamelnikov(at)inbox(dot)ru>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] Fix pg_upgrade test from v10
Date: 2022-06-05 10:38:01
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers


On 02.06.2022 23:57, Andrew Dunstan wrote:

> 1. There is no mention of why there's a change w.r.t. Cygwin and
> permissions checks. Maybe it's ok, but it seems off topic and is
> certainly not referred to in the patch submission.
Thanks for the comments!
It was my error to change w.r.t. Cygwin. I've fixed it in the second
version of the patch. But change in permissons check is correct. If we
fix the error with initdb options, we've got the next one while testing
upgrade from v10:
"files in PGDATA with permission != 640"
and the will end immediately.
The thing is that the default permissions have changed in v11+ due to
this commit:;a=commit;h=c37b3d08ca6873f9d4eaf24c72a90a550970cbb8.
Changes of permissions checks in fix this error.

> On 2022-06-01 We 21:37, Michael Paquier wrote:
>> A node's pg_version is assigned via _set_pg_version() when creating it
>> using PostgreSQL::Test::Cluster::new(). In order to make the
>> difference with the set of initdb options to use when setting up the
>> old node, it would be simpler to rely on that, no? is able
>> to handle integer as well as string comparisons for the version
>> strings.
> 2. As Michael says, we should not be using perl's version module, we
> should be using the version object built into each
> PostgreSQL::Test::Cluster instance.
Sure, very valuable note. Fixed it in the 2nd version of the patch attached.

Also find that i forgot to adjust initdb keys for new node in v15. So
there was an error due to wal-segsize mismatch. Fixed it in the 2nd
version too. And added patches for other versions.

> The client buildfarm does not make use of the in-core facility, as it
> has its own module and logic to check after the case of cross-version
> upgrades (see PGBuild/Modules/

Michael, thanks a lot for your 2c.

With best regards,

Anton A. Melnikov
Postgres Professional:
The Russian Postgres Company

Attachment Content-Type Size
v2-0001-For-v15-Fix-upgrade-test-from-v10.patch text/x-patch 1.5 KB
v2-0001-For-v14-Fix-upgrade-test-from-v10.patch text/x-patch 2.3 KB
v2-0001-For-v12-13-Fix-upgrade-test-from-v10.patch text/x-patch 2.3 KB
v2-0001-For-v11-Fix-upgrade-test-from-v10.patch text/x-patch 2.4 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Matthias van de Meent 2022-06-05 19:12:36 Re: Improving btree performance through specializing by key shape, take 2
Previous Message Tomas Vondra 2022-06-05 09:51:32 Re: pgcon unconference / impact of block size on performance