-
Notifications
You must be signed in to change notification settings - Fork 386
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
[docker-in-docker] - Use blocking entrypoints to ensure dockerd startup #671
Comments
Seconding this! We are unfortunately still running into this issue. We're using DDEV and transitioned from using docker-in-docker, to the universal:2 package as per the conversation over in #634 yet any DDEV command failed with the following error message: Doing a partial and full rebuild is still netting us the same issue. The only thing we've found works is completely deleting the Codespace and recreating it from scratch. However after a single codespace inactivity timeout sends us back to the docker to no longer being recognized error. Staying tuned here to see if there's any activity on this fix 🤞 If it's helpful, here's a snapshot of our devcontainer.json
|
Hi @Christopholus 👋 We released new If you run Thanks for your patience, we will soon have the codespaces cache updated. However, you should get latest image with Update: You could pin the universal image, however, as it's not cached yet, it would lead to slower codespace creations. (unless you use prebuilds)
|
Thanks @samruddhikhandale! I just took a look with our team, and it when we use the So it does look like we're seeing v0.3.17 - and a release date of yesterday with lines up with your comment. Unfortunately, we are still seeing the Please let us know if you have any other ideas about what might be happening - and thanks for your help! |
Interesting, investigating with the provided devcontainer ^. In the meanwhile, @Christopholus if
Just curious, does pinning universal version to |
Thanks for this @samruddhikhandale. After some review with our team - it looks like pinning the universal version to We'll roll this tweak across the other project we're utilizing Codespaces for, and monitor to see if the issue crops back up. I'll be sure to poke back in here if we have any more Docker drama. I'm hopeful that in a few weeks time we can unpin the version (as maybe the cache may have been updated by then 🤞) Thanks again for your prompt responses here! |
hey, we noticed in recent startups that docker is not ready (again) when using the Any one else noticing this? Thanks for fixing this the first time! 🙏 (https://github.com/orgs/community/discussions/63776#discussioncomment-6745270) |
Reproduction with https://github.com/mandrasch/ddev-craftcms-vite
If I run |
We have seen flakiness in docker startups for
docker-in-docker
Feature where docker is not running in the container. See #634 and #660.#669 adds few retry mechanisms to attempt to fix this issue.
/usr/local/share/docker-init.sh
which starts/retries dockerd is added to theentrypoint
property for the Feature. This command runs in the background and is not a blocking script for the container startup. Since it's in the background,onCreateCommand/postCreateCommand/postStartCommand
could all start executing before docker is fully running. If it takes docker too long, that could introduce flakiness in those lifecycle scripts.Opened devcontainers/spec#299 which requests a new semantics to have "blocking" entrypoints that the CLI waits for. This way we can ensure that docker is already up and running for the mentioned ^ lifecycle scripts and is 💯 available in the container.
This issue tracks updates to dnd Feature, when blocking entrypoint is available.
Blocked by -
The text was updated successfully, but these errors were encountered: