[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-04 Thread STINNER Victor
STINNER Victor added the comment: regrtest default timeout is now 30 minutes. I opened specific issues for each failure: * test_sendall_interrupted() of test_socket: issue #11753 * test_itimer_real() of test_signal: issue #11755 * test_threadsignals: issue #11768 * test_notify() of test_th

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-03 Thread Roundup Robot
Roundup Robot added the comment: New changeset 394f0ea0d29e by Victor Stinner in branch 'default': Issue #11727, issue #11753, issue #11755: disable regrtest timeout http://hg.python.org/cpython/rev/394f0ea0d29e -- ___ Python tracker

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread STINNER Victor
STINNER Victor added the comment: Timeout of 15 minutes on "x86 XP-4 3.x": ... [334/354] test_threading Thread 0x024c: File "D:\cygwin\home\db3l\buildarea\3.x.bolen-windows\build\lib\threading.py", line 235 in wait File "D:\cygwin\home\db3l\buildarea\3.x.b

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread STINNER Victor
STINNER Victor added the comment: For test_subprocess timeout, see also: http://bugs.python.org/issue8429#msg103368 For test_io timeout, see also: issue #8431 -- ___ Python tracker

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread STINNER Victor
STINNER Victor added the comment: Because the timeout was 30 minutes, we missed where test_ssl hangs on "x86 Windows7 3.x": -- ... [259/354] test_filecmp [260/354] test_ssl command timed out: 1200 seconds without output, killing pid 3012 SIGKILL failed to kill process using fake rc

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread STINNER Victor
STINNER Victor added the comment: (oh, "sparc solaris10 gcc 3.x" failed on test_subprocess, test_stdin_none is the function, not the file) With a timeout of 30 minutes: - "sparc solaris10 gcc 3.x" doesn't fail anymore. - "x86 FreeBSD 3.x" still fails on the same test: test_io (test_interru

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread STINNER Victor
STINNER Victor added the comment: Failures on the default branch with a timeout of 15 minutes: - test_io on "x86 FreeBSD 3.x" - test_signal on "x86 FreeBSD 7.2 3.x" - test_stdin_none on "sparc solaris10 gcc 3.x" I think that the tests failed because the buildbot is slow, not because of a h

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 053bc5ca199b by Victor Stinner in branch 'default': Issue #11727: set regrtest default timeout to 30 minutes http://hg.python.org/cpython/rev/053bc5ca199b -- ___ Python tracker

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread STINNER Victor
STINNER Victor added the comment: Great! The timeout works: - ... [ 25/354] test_threadsignals Thread 0xa000ed88: File "/Users/buildbot/buildarea/3.x.parc-tiger-1/build/Lib/test/test_threadsignals.py", line 46 in test_signals File "/Users/buildbot/builda

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-04-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 15f6fe139181 by Victor Stinner in branch 'default': Issue #11727: set regrtest default timeout to 15 minutes http://hg.python.org/cpython/rev/15f6fe139181 -- ___ Python tracker

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
STINNER Victor added the comment: Patch commited into 3.3 (bdc946dc512a, 0f3dbf93ac06). I opened a thread on python-dev to decide if we turn this on by default on buildbots. -- resolution: -> fixed status: open -> closed ___ Python tracker

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread Roundup Robot
Roundup Robot added the comment: New changeset 0f3dbf93ac06 by Victor Stinner in branch 'default': Issue #11727: Antoine and Arfrever don't like "0 < timeout" style (regrtest.py) http://hg.python.org/cpython/rev/0f3dbf93ac06 -- ___ Python tracker

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread Roundup Robot
Roundup Robot added the comment: New changeset bdc946dc512a by Victor Stinner in branch 'default': Issue #11727: add --timeout option to regrtest (disabled by default). http://hg.python.org/cpython/rev/bdc946dc512a -- nosy: +python-dev ___ Python tra

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread Charles-Francois Natali
Charles-Francois Natali added the comment: > There is something interesting in this output: the test uses a subprocess and > we only have the traceback of the parent. It may be nice to have the trace of > the child process. It might be possible by sending a signal to the child > process (but

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis : -- nosy: +Arfrever ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscri

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
STINNER Victor added the comment: Result on "x86 FreeBSD 7.2 custom": ... [148/354] test_fileinput [149/354] test_subprocess [36645 refs] [36645 refs] ... [36647 refs] [44034 refs] Current thread 0x28401040: File "/usr/home/db3l/buildarea/custom.bolen-freebsd7/build/Lib/s

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
STINNER Victor added the comment: I ran the full test suite on "x86 Tiger custom" on my features/regrtest_timeout repository: no failure (test_threadsignals failure was not reproduced). On "PPC Leopard custom", the test suite was interrupted (timeout) on test_io.test_basic_io: ... [ 53/3

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
STINNER Victor added the comment: >> "x86 Tiger custom" to learn more about test_threadsignals timeout >> (1 hour) > > Please read the comments. There is no timeout anymore. The last build failed with: --- ... [213/354] test_plistlib [214/354] test_minidom [215/354] test_threadsignals command

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread Antoine Pitrou
Antoine Pitrou added the comment: > "x86 Tiger custom" to learn more about test_threadsignals timeout > (1 hour) Please read the comments. There is no timeout anymore. -- ___ Python tracker _

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread Antoine Pitrou
Antoine Pitrou added the comment: Just a nit but: "if timeout and 0 < timeout" Could you use the usual coding style? ("if timeout and timeout > 0") Putting constants on the left-side of comparisons is pointless in Python, and even in C it's quite ugly and surprising. -- nosy: +pitrou

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
STINNER Victor added the comment: Try the branch on "PPC Leopard 3.x" (test_subprocess timeout): http://www.python.org/dev/buildbot/all/builders/PPC%20Leopard%20custom/builds/6 -- ___ Python tracker _

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
STINNER Victor added the comment: I created a new Mercurial repository to test this issue: features/regrtest_timeout. Let's try it with a timeout of 5 minutes on: - "x86 Tiger custom" to learn more about test_threadsignals timeout (1 hour) => issue #11223 - "x86 FreeBSD 7.2 3.x" to learn mo

[issue11727] Add a --timeout option to regrtest.py using the faulthandler module

2011-03-31 Thread STINNER Victor
New submission from STINNER Victor : I would be nice to add a timeout option to regrtest.py to be able to dump the traceback after TIMEOUT seconds and also exit the process. It would help debugging timeout issues (e.g. test_multiprocessing ), but also protect buildbots against evil (long) test