From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pg_rewind tests |
Date: | 2015-04-13 15:32:31 |
Message-ID: | 552BE18F.5080704@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 04/02/2015 04:56 AM, Michael Paquier wrote:
> On Tue, Mar 31, 2015 at 4:37 PM, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
> wrote:
>
>> While looking at that I noticed two additional issues:
>> - In remote mode, the connection string to the promoted standby was
>> incorrect when running pg_rewind, leading to connection errors
>> - At least in my environment, a sleep of 1 after the standby promotion was
>> not sufficient to make the tests work.
>
> While working on another patch for TAP tests, I noticed that relying on
> environment variables to run tests is a bad idea as well, as other tests do
> not do it, so instead is a patch that refactors the tests so as they do not
> use environment variables and so as it is not necessary to pass arguments
> to prove.
> The trick is to use sub-routines in each test, and invoke this subroutine
> for both 'local' and 'remote'. This changes the order the tests are run,
> but I guess that's not a big deal as long as the tests are run, and this
> approach looks more solid to me as it makes pg_rewind's tests more
> consistent with the rest. The log files of each test are still separated
> the same way as before.
Applied, thanks!
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2015-04-13 23:22:22 | Re: pg_rewind tests |
Previous Message | Heikki Linnakangas | 2015-04-13 15:32:13 | Re: pg_rewind tests |