docker-py icon indicating copy to clipboard operation
docker-py copied to clipboard

implementing the revert suggested by @shin- ages ago

Open abrugh opened this issue 6 years ago • 4 comments
trafficstars

Fixes the issue described, tested and reproduced in https://github.com/docker/compose/issues/6018

abrugh avatar Mar 29 '19 17:03 abrugh

Please sign your commits following these rules: https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work The easiest way to do this is to amend the last commit:

$ git clone -b "6018" [email protected]:abrugh/docker-py.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

GordonTheTurtle avatar Mar 29 '19 17:03 GordonTheTurtle

This PR is not passing due to an already solved problem. Please rebase your PR with master

ulyssessouza avatar Apr 04 '19 12:04 ulyssessouza

Rebased, still failing AttachContainerTest::test_attach_stream_and_cancel but only 2 out of 4 times....

abrugh avatar Apr 04 '19 20:04 abrugh

Looks like it's failing pretty consistently on Python 2.7 (if you look at the test matrix here). To me that suggests that that condition does something helpful, and the fix is less trivial than it appears.

shin- avatar Apr 04 '19 21:04 shin-