[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Change by STINNER Victor : -- resolution: -> out of date stage: -> resolved status: open -> closed ___ Python tracker ___ ___

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

STINNER Victor added the comment: As expected, the workaround adding a delay of 500 ms is not perfect, the test still fails on the very slow x86 Windows7 3.6 buildbot when tests are run in parallel: http://buildbot.python.org/all/#/builders/90/builds/383

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

STINNER Victor added the comment: Python 3.6 got the first change: commit 6aacc160a880de3b82c51193a4594b87fb7ace5f Author: Yury Selivanov Date: Tue May 29 14:57:38 2018 -0400 Fix ci for 3.6 / asyncio / test_stdin_broken_pipe (#7212) -- ___

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

STINNER Victor added the comment: Yury made 3 changes in the master branch for this change: * Workaround: await asyncio.sleep(0.5, loop=self.loop) * Use stdout to synchrone parent and child process * Revert the second change (stdout sync) commit bb9474f1fb2fc7c7ed9f826b78262d6a12b5f9e8

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Terry J. Reedy added the comment: The patch seems to have worked. The last AppVeyor failure was a day ago, when testing the 3.7 backport of the fix. https://ci.appveyor.com/project/python/cpython/history -- ___ Python tracker

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

STINNER Victor added the comment: > Fixed failed sendfile tests on Windows (at least I hope so). I guess that Andrew means the following fix: commit e2537521916c5bf88fcf54d4654ff1bcd332be4a Author: Andrew Svetlov Date: Mon May 21 12:03:45 2018

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Andrew Svetlov added the comment: Fixed failed sendfile tests on Windows (at least I hope so). -- ___ Python tracker ___

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Change by Christian Heimes : -- nosy: +christian.heimes ___ Python tracker ___ ___

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

STINNER Victor added the comment: Terry: " The Travis retest (for 3.7 backport) just failed with the same errors. Why can't we disable this tests. It now takes hours to do a merge with two backports." Usually, I prefer to try to understand a bug before disabling, even

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Terry J. Reedy added the comment: The Travis retest (for 3.7 backport) just failed with the same errors. Why can't we disable this tests. It now takes hours to do a merge with two backports. -- ___ Python tracker

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Terry J. Reedy added the comment: I just got the two broken pipe error on Travis. -- ___ Python tracker ___

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Terry J. Reedy added the comment: I have also gotten test_async errors on AppVeyor and MSTS Windows. ERROR: test__sock_sendfile_native_failure (test.test_asyncio.test_base_events.BaseLoopSockSendfileTests)

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

Change by Eitan Adler : -- nosy: +eitan.adler ___ Python tracker ___ ___

[issue33531] test_asyncio: test_subprocess test_stdin_broken_pipe() failure on Travis CI

New submission from STINNER Victor : https://travis-ci.org/python/cpython/jobs/379560387 == FAIL: test_stdin_broken_pipe (test.test_asyncio.test_subprocess.SubprocessFastWatcherTests)