Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mnMX5-0004bt-HE for pgsql-hackers@arkaria.postgresql.org; Wed, 17 Nov 2021 15:07:31 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1mnMX3-0004NX-HS for pgsql-hackers@arkaria.postgresql.org; Wed, 17 Nov 2021 15:07:29 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mnMX3-0004ND-7a for pgsql-hackers@lists.postgresql.org; Wed, 17 Nov 2021 15:07:29 +0000 Received: from relay10.mail.gandi.net ([217.70.178.230]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mnMX0-000412-A5 for pgsql-hackers@lists.postgresql.org; Wed, 17 Nov 2021 15:07:27 +0000 Received: (Authenticated sender: adsend@dunslane.net) by relay10.mail.gandi.net (Postfix) with ESMTPSA id F14D624000D; Wed, 17 Nov 2021 15:07:18 +0000 (UTC) Message-ID: <0b8ebc97-829c-4a40-efcc-761f52bffb0d@dunslane.net> Date: Wed, 17 Nov 2021 10:07:17 -0500 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 Subject: Re: pg_upgrade test for binary compatibility of core data types Content-Language: en-US To: Michael Paquier , Justin Pryzby Cc: Jacob Champion , tgl@sss.pgh.pa.us, peter.eisentraut@enterprisedb.com, pgsql-hackers@lists.postgresql.org, buschmann@nidsa.net, noah@leadboat.com, tomas.vondra@2ndquadrant.com, bruce@momjian.us, andres@anarazel.de References: <20210112175356.GP4320@momjian.us> <66753439-83ae-7be5-6e41-9537a6041a39@dunslane.net> <1575064.1615060903@sss.pgh.pa.us> <20210430183348.GQ27406@telsasoft.com> <20210912005116.GF26465@telsasoft.com> <20211107192200.GK17618@telsasoft.com> From: Andrew Dunstan In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On 11/17/21 02:01, Michael Paquier wrote: > > The oldest version tested by the buildfarm is 9.2, so we could ignore > this part I guess? > > Andrew, what do you think about this part? Based on my read of this > thread, there is an agreement that this approach makes the buildfarm > code more manageable so as committers would not need to patch the > buildfarm code if their test fail. I agree with this conclusion, but > I wanted to double-check with you first. This would need a backpatch > down to 10 so as we could clean up a maximum of code in > TestUpgradeXversion.pm without waiting for an extra 5 years. Please > note that I am fine to send a patch for the buildfarm client. > > In general I'm in agreement with the direction here. If we can have a script that applies to back branches to make them suitable for upgrade testing instead of embedding this in the buildfarm client, so much the better. cheers andrew -- Andrew Dunstan EDB: https://www.enterprisedb.com