go-runc icon indicating copy to clipboard operation
go-runc copied to clipboard

Need help to understand non-linux (Windows) branch in go-runc

Open ningziwen opened this issue 2 years ago • 3 comments

Hi, I'm new to go-runc and trying to learn go-runc from reading the docs and source code.

I noticed there is code for non-linux (Windows) branch in multiple places.

https://github.com/containerd/go-runc/blob/main/command_other.go https://github.com/containerd/go-runc/blob/main/io_windows.go

I'm wondering what is the use case of having non-linux branch in go-runc. My understanding is go-runc is a wrapper for runc and runc is linux-specific. Windows container runtime has a separated repo hcsshim to be used by containerd. So why does go-runc need windows branch? Does go-runc/runc have the plan to support Windows container?

ningziwen avatar Dec 30 '22 06:12 ningziwen

Seems it was for hcsshim.

cc @jterry75 because of https://github.com/containerd/go-runc/pull/45

kzys avatar Jan 13 '23 22:01 kzys

Lolz its march and I have now seen this. Still need help?

jterry75 avatar Mar 29 '23 20:03 jterry75

Its just because the client is built into some generic containerd calls and so rather than forking all of those go files we just have it return unsupported for windows

jterry75 avatar Mar 29 '23 20:03 jterry75