-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add Linux containers support #8
Comments
For the reference, here's how Docker Desktop works:
Some logs. This is how proxy is launched for Windows containers:
This is how proxy is launched for Linux containers:
Neither proxy nor backend is running:
Proxy is running but backend is not (Linux containers):
Note that Linux dockerd is only listening |
|
It might be possible to open Unfortunately, there's no way to pass custom transport to docker client: https://github.com/moby/moby/blob/v20.10.12/client/client.go#L120 See https://devblogs.microsoft.com/commandline/windowswsl-interop-with-af_unix/ UPD: forget about AF_UNIX. It doesn't work in WSL2: microsoft/WSL#5961 |
I'm currently working on docker-wsl-proxy It is supposed to become the missing puzzle piece that converts Windows <-> WSL2 paths and routes docker requests into WSL2. When docker-wsl-proxy is ready, the work will continue in linux-containers branch of current repo. |
After several days of messing with docker-wsl-proxy, I start to think that using moby routes is a dead-end for several reasons:
New plan: we go one level lower and instead implement a HTTP proxy. Currently, it looks like the only place that requires paths conversions is container creation. We could intercept that API call in the proxy and rewrite JSON. I suddenly understood that paths is not the only thing we need to do. We also need to support container port bindings to Windows host ports. This possibly can wait though. |
HTTP proxy ended up being so much simpler: slonopotamus/docker-wsl-proxy@7c75d2d Missing bits are:
|
That's what was committed to Desktop_220214_1355.mp4 |
No description provided.
The text was updated successfully, but these errors were encountered: