Re: pgsql: pg_upgrade: Check version of target cluster binaries

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: pg_upgrade: Check version of target cluster binaries
Date: 2021-03-03 21:58:09
Message-ID: 9df301a8-78bf-7fe4-c6bb-22ef7727385c@dunslane.net
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers


On 3/3/21 3:28 PM, Tom Lane wrote:
> Peter Eisentraut <peter(at)eisentraut(dot)org> writes:
>> pg_upgrade: Check version of target cluster binaries
> Crake says this patch broke its cross-version upgrade tests:
>
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=crake&dt=2021-03-03%2009%3A02%3A31
>
> where the important bit is
>
> check for "/home/andrew/bf/root/upgrade.crake/REL9_2_STABLE/inst/bin/postgres" failed: incorrect version: found "postgres (PostgreSQL) 9.2.24", expected "postgres (PostgreSQL) 14devel"
>
> It's not at all clear from here whether that's a bug in the patch
> or in the buildfarm's test harness.
>
>

This has worked for years ... here's the latest code:
<https://github.com/PGBuildFarm/client-code/blob/master/PGBuild/Modules/TestUpgradeXversion.pm>
The invocation of pg_upgrade is at line 560.

The log says:

check for
"/home/andrew/bf/root/upgrade.crake/REL9_2_STABLE/inst/bin/postgres"
failed: incorrect version: found "postgres (PostgreSQL) 9.2.24",
expected "postgres (PostgreSQL) 14devel"

But that makes no sense at all. Looks like we're confusing the source and the target.

cheers

andrew

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

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2021-03-03 22:04:41 Re: pgsql: pg_upgrade: Check version of target cluster binaries
Previous Message Tom Lane 2021-03-03 21:40:20 pgsql: Add trim_array() function.