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
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 |