From: | Craig Ringer <craig(dot)ringer(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
Cc: | Andres Freund <andres(at)anarazel(dot)de>, Daniel Gustafsson <daniel(at)yesql(dot)se>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Subject: | Re: A note about debugging TAP failures |
Date: | 2017-04-23 13:05:29 |
Message-ID: | CAMsr+YFyFU=+MVFZqhthfMW22x5-h517e6ck6ET+DT+X4bUO7g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 23 Apr. 2017 10:32, "Michael Paquier" <michael(dot)paquier(at)gmail(dot)com> wrote:
On Sun, Apr 23, 2017 at 7:48 AM, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> Skipping the tempdir and instead using ${testname}_data_${name} without a
> random suffix, we can achieve this with something along the lines of the
> attached PoC. It works as now (retain of failure, remove on success
unless
> overridden) but that logic can easily be turned around if we want that.
If
> it’s of interest I can pursue this after some sleep (tomorrow has become
today
> at this point).
Yes, something like that may make sense as well for readability.
Keeping folders in case of failures is something that I have been
advocating in favor of for some time, but this never got into the tree
:(
Huh? We do keep test temp datadirs etc in case of failure. Just not on
success.
Our definition of failure there sucks a bit though. We keep the datadirs if
any test fails in a script. If the script its self crashes we still blow
away the datadirs which is kind of counterintuitive.
I'd like to change the __DIE__ sig handler to only delete on clean script
exit code, tap reporting success, and if some env bar like PG_TESTS_NOCLEAN
is undefined. The later could also be used in pg_regress etc.
There is a patch from Horiguchi-san that allows actually such a thing,
have a look at patch 0006 on this email:
https://www.postgresql.org/message-id/CAMsr+YEc2Ek=
qJFJ2JeV7Spz69pOduoUJXBowZMppVYdeieRdA(at)mail(dot)gmail(dot)com
The same concept rebased gives the attached. I didn't double-check the
compatibility with past versions of Test:More though..
--
Michael
--
Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2017-04-23 14:01:16 | Re: Statistics "dependency" |
Previous Message | Simon Riggs | 2017-04-23 10:44:12 | Re: Statistics "dependency" |