Repository navigation
Increase the podman driver's API timeout - #4820
Conversation
|
All contributors have signed the DCO ✍️ ✅ |
|
I have read the DCO document and I hereby sign the DCO. |
|
/ok to test 9c091d9 |
|
@sruggier please see the failing lint jobs |
Experience has shown that ordinary podman container creation can take longer than 30 seconds for some users, so for now, this change doubles the timeout, until the source of the slowness is found and accounted for. Signed-off-by: Simon Ruggier <Simon.Ruggier@gmail.com>
9c091d9 to
d31866d
Compare
Thanks, sorry I didn't catch that locally. I've pushed a fix (and slightly updated the commit description). |
|
/ok to test d31866d |
|
@sruggier thanks! I edited the description slightly to not auto close. Please let me know further how we can help investigate the root cause issue further! |
From this document, it looks like it's a known issue:
My setup definitely ticks all of these boxes. I'm not sure about the fourth one, though - I tried creating a second sandbox, and it was still slow enough that it might not be any faster than the first one. I can see in top that most of the slowness seems to involve a couple of From my perspective as a user, bumping the timeout is enough to solve the problem for me, the actual container creation time is acceptable as it is now. I'm able to try switching to fuse-overlayfs at some point, but it requires me to run |
Summary
This PR increases the podman driver's API timeout: Experience has shown that normal container creation can take longer than 30 seconds, for some users, so this change doubles the timeout, for now, until the source of the slowness is found and fixed.
Related Issue
Relates to #2913. (Edited to not auto-close)
This only partially addresses the issue, though, so it would probably make sense to keep it open, for now.
Release Notes
The hard-coded API timeout in the podman driver has been bumped from 30s to 60s, to reduce the likelihood of timeouts during sandbox creation (until the cause is investigated).
Testing
None yet.
Checklist