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
fix race condition, waiting for containers when one exit #10725
Conversation
e15772a
to
123587b
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## v2 #10725 +/- ##
==========================================
+ Coverage 58.94% 58.99% +0.04%
==========================================
Files 112 112
Lines 9749 9752 +3
==========================================
+ Hits 5747 5753 +6
+ Misses 3413 3410 -3
Partials 589 589
☔ View full report in Codecov by Sentry. |
123587b
to
853c219
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
853c219
to
0de121f
Compare
Hello there! It took me 3 days of troubleshooting/purging/reinstalling docker-desktop to finally find this issue because compose also stopped throwing error messages which could have shortened my odyssey: compose-not-responding+stuck-forever.mp4@ndeloof Could we please somehow trigger a release of Docker Desktop whenever a new Compose version is released so that always recent versions get shipped?? |
What I did
as a container exits with error, we must cancel but the goroutine waiting for expected container to be started used to get stuck waiting. Need to stop waiting when context is cancelled
Related issue
fixes #10707
fixes #10718
(not mandatory) A picture of a cute animal, if possible in relation to what you did