Re: Failure in TAP tests of pg_ctl on Windows with parallel instance set

From: Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Failure in TAP tests of pg_ctl on Windows with parallel instance set
Date: 2019-12-02 12:57:31
Message-ID: 5c1c1e80-8ded-5d8a-be7d-05ee6a0368ca@2ndQuadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 12/1/19 10:14 PM, Michael Paquier wrote:
> Hi all,
>
> I have run the TAP tests with an instance of Postgres locally set at
> port 5432 on Windows, to notice that 001_start_stop.pl fails various
> tests because the test tries to use the default port for the node
> initialized with pg_ctl. The problem can get fixed easily by
> assigning a random port number to that instance.
>
> It could potentially become a problem if parallel TAP tests run in
> parallel on Windows while initializing the node because of a port
> conflict, but that's actually already a problem now for all the tests
> as all nodes listen to 127.0.0.1 in this case. This cannot happen on
> *nix simply because we use a unique unix domain path, so even if ports
> conflict things are able to work.
>
> Attached is a patch to fix this issue, that I would like to
> back-patch down to 9.4 where the issue can show up.
>
> Any objections?

Looks reasonable. I wonder if there are other test sets where we need to
set the port.

cheers

andrew

--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2019-12-02 13:00:28 Re: Collation versioning
Previous Message Peter Eisentraut 2019-12-02 12:27:19 Simplify passing of configure arguments to pg_config