[Python-checkins] r61650 - python/trunk/Lib/test/test_socketserver.py

[Neal Norwitz]

On Wed, Mar 19, 2008 at 5:51 PM, trent.nelson
<python-checkins at python.org> wrote:
> Author: trent.nelson
>  Date: Wed Mar 19 23:51:42 2008
>  New Revision: 61650
>
>  Modified:
>    python/trunk/Lib/test/test_socketserver.py
>  Log:
>  Bump the SIGALM delay from 3 seconds to 20 seconds, mainly in an effort to see if it fixes the alarm failures in this test experienced by some of the buildbots.
>
>  Modified: python/trunk/Lib/test/test_socketserver.py
>  ==============================================================================
>  --- python/trunk/Lib/test/test_socketserver.py  (original)
>  +++ python/trunk/Lib/test/test_socketserver.py  Wed Mar 19 23:51:42 2008
>  @@ -253,4 +253,7 @@
>
>   if __name__ == "__main__":
>      test_main()
>  -    signal_alarm(3)  # Shutdown shouldn't take more than 3 seconds.
>  +    # Signal an alarm after 20 seconds if we haven't shut down.  This
>  +    # was originally 3 seconds, but has been bumped as numerous build
>  +    # slaves have been failing with SIGALMs on this test.
>  +    signal_alarm(20)

This won't have any effect on the buildbots since the tests are
started in regrtest.oy by calling test_main().
(There are a few tests that operate by side-effect of import, but
those are going away.)

n