-
-
Notifications
You must be signed in to change notification settings - Fork 580
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
codespaces - remove docker-in-docker since it's installed via universal image #5267
codespaces - remove docker-in-docker since it's installed via universal image #5267
Conversation
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.
Would you mind adding these changes to the DDEV devcontainer.json as well?
Sure thing, very happy to try it! :-) I'll try with the universal image (and without
|
Updated the Creating a new codespace instance with it ran through successfully, I could also open the example project in /workspaces/d9simple/. ✅ One slight hiccup occured when the codespace instance was automatically paused, after the restart docker was not up and running. After turning it off and on again everything worked again. Most probably not a DDEV-related issue, just a codespace one. Tried another manual restart as well, all worked fine. |
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.
Thanks!
@@ -3,9 +3,8 @@ | |||
{ | |||
"name": "Go", | |||
// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile | |||
"image": "mcr.microsoft.com/devcontainers/go:0-1-bullseye", | |||
"image": "mcr.microsoft.com/devcontainers/universal:2", |
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.
Universal also contains golang and make, all we need.
The Issue
it was explained here, that the universal image already installs
docker-in-docker
Docker-in-docker: Add retry mechanism into the docker init script (Failed to connect to Docker) devcontainers/features#634 (comment)docker-in-docker
feature had no effect and lead to irritating output, since the outputted version info in codespaces creation log had no real effect (since docker was already installed)updated my demo repo https://github.com/mandrasch/ddev-craftcms-vite/blob/main/.devcontainer/devcontainer.json#L4, first tests worked
We might discuss whether ddev itself should be tested with universal image as well?
ddev/.devcontainer/devcontainer.json
Lines 2 to 14 in 56afe4b
How This PR Solves The Issue
Manual Testing Instructions
Automated Testing Overview
Related Issue Link(s)
Release/Deployment Notes
no