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 1nAHnd-0007W6-OH for pgsql-hackers@arkaria.postgresql.org; Wed, 19 Jan 2022 20:43:21 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1nAHnc-0001sz-BO for pgsql-hackers@arkaria.postgresql.org; Wed, 19 Jan 2022 20:43:20 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nAHnc-0001sq-1s for pgsql-hackers@lists.postgresql.org; Wed, 19 Jan 2022 20:43:20 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1nAHnZ-0004ps-KK for pgsql-hackers@postgresql.org; Wed, 19 Jan 2022 20:43:19 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 20JKhAJV656025; Wed, 19 Jan 2022 15:43:10 -0500 From: Tom Lane To: Andres Freund cc: Justin Pryzby , Robert Haas , Andrew Dunstan , pgsql-hackers@postgresql.org, Thomas Munro , Melanie Plageman , Peter Eisentraut , Daniel Gustafsson Subject: Re: Adding CI to our tree In-reply-to: <20220119202926.nwbu5zfjts5qxbl4@alap3.anarazel.de> References: <20220117192510.txue5mihjaxlngep@alap3.anarazel.de> <85428.1642447853@sss.pgh.pa.us> <20220117201619.3ltudwhgk2krmoki@alap3.anarazel.de> <20220118210847.GC23027@telsasoft.com> <20220119011626.nw5cvugqfsy3fyob@alap3.anarazel.de> <440239.1642560607@sss.pgh.pa.us> <20220119041646.rhuo3youiqxqjmo2@alap3.anarazel.de> <450972.1642568052@sss.pgh.pa.us> <647439.1642622744@sss.pgh.pa.us> <20220119202926.nwbu5zfjts5qxbl4@alap3.anarazel.de> Comments: In-reply-to Andres Freund message dated "Wed, 19 Jan 2022 12:29:26 -0800" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <656023.1642624990.1@sss.pgh.pa.us> Date: Wed, 19 Jan 2022 15:43:10 -0500 Message-ID: <656024.1642624990@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Andres Freund writes: > I'm doubtful that it's good that we use BAIL_OUT so liberally, because it > prevents further tests from running (i.e. if 001 bails, 002+ doesn't run), > which doesn't generally seem like the right thing to do after a single test > fails. But that's really independent of the fix you make here. Agreed, that's a separate question. It does seem like "stop this script and move to the next one" would be better behavior, though. > I'd maybe do s/later/in the END block/ or such, so that one knows where to > look. Took me a minute to find it again. OK. regards, tom lane