-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
getting timeout after streaming logs for some time #931
Comments
Hmm, I tested against 1.9.0 and the same thing happens. What's your version of |
docker-py is git master (c3a66cc)
Whole stack trace
|
I have this same issue with 1.4.0 of docker-py and 1.9.1 of docker-engine. It hits us when we try to push a large (>4gb) container. |
I had the same problem (requests = 2.3.0). Then I created a new virtual environment and installed the package(s) with the newest versions (requests = 2.9.1) and I had no problems anymore. |
I have the same problem, any answer about this? |
Fixes docker#931 Signed-off-by: Segev Finer <[email protected]>
I hit this (read timeout) error today while using the Python 3.6, requests==2.22.0 FWIW |
On a detached container,
This happens when the container has no console output for long enough to exceed the timeout. A couple of workarounds helped:
This sounds like the issue @TomasTomecek. It also makes sense that @jasonamyers and @akki might hit this with other parts of the API that can possibly wait a long time for the server to respond. While I'm not familiar with the code, ideally, these methods would set an infinite timeout, while other methods would set a reasonable default timeout. |
I think I made a PR for this. |
Indeed you did; I missed the pingback from #1959. That patch works for me in place of the workarounds in my previous comment. +1 |
Fixes docker#931 Signed-off-by: Segev Finer <[email protected]>
This is very likely an engine issue, first I would like to open here.
It looks like there is a change in behavior. When streaming logs with 1.10 I get timeout:
generator is
client.logs(c, stream=True)
This repro should be sufficient: #919 (comment)
I know this also happens when using compose.
The text was updated successfully, but these errors were encountered: