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 1nAfxJ-0006dQ-MB for pgsql-committers@arkaria.postgresql.org; Thu, 20 Jan 2022 22:30:57 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1nAfxF-0006te-N6 for pgsql-committers@arkaria.postgresql.org; Thu, 20 Jan 2022 22:30:53 +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 1nAfxF-0006tR-Gh for pgsql-committers@lists.postgresql.org; Thu, 20 Jan 2022 22:30:53 +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 1nAfxD-000234-El for pgsql-committers@lists.postgresql.org; Thu, 20 Jan 2022 22:30:53 +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 20KMUiNq1075554; Thu, 20 Jan 2022 17:30:44 -0500 From: Tom Lane To: Andres Freund cc: Noah Misch , Thomas Munro , pgsql-committers Subject: Re: pgsql: TAP tests: check for postmaster.pid anyway when "pg_ctl start" f In-reply-to: <926839.1642708567@sss.pgh.pa.us> References: <855583.1642687119@sss.pgh.pa.us> <867290.1642689136@sss.pgh.pa.us> <20220120192736.msyesvdlljn6tptw@alap3.anarazel.de> <926839.1642708567@sss.pgh.pa.us> Comments: In-reply-to Tom Lane message dated "Thu, 20 Jan 2022 14:56:07 -0500" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <1075552.1642717844.1@sss.pgh.pa.us> Date: Thu, 20 Jan 2022 17:30:44 -0500 Message-ID: <1075553.1642717844@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk I wrote: > What I'm thinking of doing is inventing a "soft_stop" variant of > Cluster::stop that won't complain if pg_ctl stop fails, and then > having 017_shm's poll_start() call that before retrying the start > call. I did that, modeling the details on Cluster::start's fail_ok option. > It seems like it'd also be a good idea if the END block used > soft_stop instead of regular stop. I still like this idea, but I left it alone for the moment, figuring I've used up my quota of buildfarm destabilization for now. regards, tom lane