Issue2708

classification
Title: test_urllib2net fails (on Travis CI)
Type: Severity: normal
Components: Versions:
Milestone:
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: darjus, jamesmudd, jeff.allen, stefan.richthofer
Priority: Keywords:

Created on 2018-10-13.11:23:54 by stefan.richthofer, last changed 2018-10-17.20:18:23 by stefan.richthofer.

Messages
msg12135 (view) Author: Stefan Richthofer (stefan.richthofer) Date: 2018-10-13.11:23:52
I noticed that rather recently (some months ago) test_urllib2net started to fail on TravisCI. I went through the github commit history to identity the satrting point of failure.
It seems to be this commit:
https://github.com/jythontools/jython/commit/2f4c2928b617a10aa99bb5de98926015500481d8
where it consequently starts failing.
One commit earlier (https://github.com/jythontools/jython/commit/52eb2fbae077036f9488a2ee9b78ed1630f2d306) the test still went well.

Routinely breaking tests make TravisCI less useful for us, so this should be cleaned up.

Given that the "breaking" PR is titled "Refactor PyJavaType.init for readability (no material change)." I suspect this issue might be induced some change in Travis CI. Traditionally, networking tests were not so reliable anyway. However also in this case we should consider to skip this test or better only the broken bit.

Any ideas what actually goes wrong?
msg12136 (view) Author: Jeff Allen (jeff.allen) Date: 2018-10-13.16:43:33
It is a fairly big refactor ... and it does not quite live up to the description. But the thing I know I broke here (sorry):
https://hg.python.org/jython/rev/623eaa3d7834
I fixed here:
https://hg.python.org/jython/rev/2adcf0e29407

test_urllib2net passes for me at both these revisions on Java 7 & 8. Also at the tip on 11. All on Windows.

I'd been attributing those failures to the network environbment in which the CI is run. I agree it is useless if it just shows red all the time for reasons we think are spurious, but there is a bit to do to fix it. I can't say I really follow the scripts. Our CI needs revisiting on quite a few points.
msg12138 (view) Author: Stefan Richthofer (stefan.richthofer) Date: 2018-10-17.20:15:58
The CI (Travis, not looking at the others) performs some fatally failing runs due to some config issues. However, there are also some okay runs and for these the only failure is test_urllib2net since some time.
So. fixing (or muting) this issue, turns at least some runs green, providing a better look than currently.

     [exec] 379 tests OK.
     [exec] 2 tests skipped:
     [exec]     test_codecmaps_hk test_curses
     [exec] 1 test failed:
     [exec]     test_urllib2net
     [exec] 1 fail unexpected:
     [exec]     test_urllib2net
     [exec] Re-running failed tests in verbose mode
     [exec] Re-running test 'test_urllib2net' in verbose mode
     [exec] test test_urllib2net failed -- multiple errors occurred

Unfortunately the "verbose" rerun of test_urllib2net is not very enlightening. I suspect it will be tedious to find the cause.
msg12139 (view) Author: Stefan Richthofer (stefan.richthofer) Date: 2018-10-17.20:18:23
In another run I found this:

     [exec] Re-running test 'test_urllib2net' in verbose mode
     [exec] test test_urllib2net failed -- Traceback (most recent call last):
     [exec]   File "/home/travis/build/jythontools/jython/dist/Lib/test/test_urllib2net.py", line 294, in test_ftp_basic
     [exec]     u = _urlopen_with_retry(self.FTP_HOST)
     [exec]   File "/home/travis/build/jythontools/jython/dist/Lib/test/test_urllib2net.py", line 27, in wrapped
     [exec]     return _retry_thrice(func, exc, *args, **kwargs)
     [exec]   File "/home/travis/build/jythontools/jython/dist/Lib/test/test_urllib2net.py", line 23, in _retry_thrice
     [exec]     raise last_exc
     [exec] URLError: <urlopen error ftp error: 425 Security: Bad IP connecting.>
     [exec]
History
Date User Action Args
2018-10-17 20:18:23stefan.richthofersetmessages: + msg12139
2018-10-17 20:15:59stefan.richthofersetmessages: + msg12138
2018-10-13 16:43:34jeff.allensetnosy: + darjus
messages: + msg12136
2018-10-13 11:23:54stefan.richthofercreate