github/containerd
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

Merge pull request #7577 from dcantah/maintenance-cri-winns

Browse Source
This commit is contained in:
Maksym Pavlenko
2022-10-23 14:32:02 -07:00
committed by GitHub
parent 52025b5d67 9a0331c477
commit 908be16858
5 changed files with 4 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
pkg/cri/opts/spec_windows.go
View File

@@ -32,21 +32,6 @@ import (
osinterface "github.com/containerd/containerd/pkg/os" osinterface "github.com/containerd/containerd/pkg/os"
) )
// WithWindowsNetworkNamespace sets windows network namespace for container.
// TODO(windows): Move this into container/containerd.
func WithWindowsNetworkNamespace(path string) oci.SpecOpts {
return func(ctx context.Context, client oci.Client, c *containers.Container, s *runtimespec.Spec) error {
if s.Windows == nil {
s.Windows = &runtimespec.Windows{}
}
if s.Windows.Network == nil {
s.Windows.Network = &runtimespec.WindowsNetwork{}
}
s.Windows.Network.NetworkNamespace = path
return nil
}
}
// namedPipePath returns true if the given path is to a named pipe. // namedPipePath returns true if the given path is to a named pipe.
func namedPipePath(p string) bool { func namedPipePath(p string) bool {
return strings.HasPrefix(p, `\\.\pipe\`) return strings.HasPrefix(p, `\\.\pipe\`)

2
pkg/cri/sbserver/container_create_windows.go
View File

@@ -85,7 +85,7 @@ func (c *criService) containerSpec(
// Clear the root location since hcsshim expects it. // Clear the root location since hcsshim expects it.
// NOTE: readonly rootfs doesn't work on windows. // NOTE: readonly rootfs doesn't work on windows.
customopts.WithoutRoot, customopts.WithoutRoot,
customopts.WithWindowsNetworkNamespace(netNSPath), oci.WithWindowsNetworkNamespace(netNSPath),
oci.WithHostname(sandboxConfig.GetHostname()), oci.WithHostname(sandboxConfig.GetHostname()),
) )

2
pkg/cri/sbserver/podsandbox/sandbox_run_windows.go
View File

@@ -51,7 +51,7 @@ func (c *Controller) sandboxContainerSpec(id string, config *runtime.PodSandboxC
// Clear the root location since hcsshim expects it. // Clear the root location since hcsshim expects it.
// NOTE: readonly rootfs doesn't work on windows. // NOTE: readonly rootfs doesn't work on windows.
customopts.WithoutRoot, customopts.WithoutRoot,
customopts.WithWindowsNetworkNamespace(nsPath), oci.WithWindowsNetworkNamespace(nsPath),
) )
specOpts = append(specOpts, customopts.WithWindowsDefaultSandboxShares) specOpts = append(specOpts, customopts.WithWindowsDefaultSandboxShares)

2
pkg/cri/server/container_create_windows.go
View File

@@ -85,7 +85,7 @@ func (c *criService) containerSpec(
// Clear the root location since hcsshim expects it. // Clear the root location since hcsshim expects it.
// NOTE: readonly rootfs doesn't work on windows. // NOTE: readonly rootfs doesn't work on windows.
customopts.WithoutRoot, customopts.WithoutRoot,
customopts.WithWindowsNetworkNamespace(netNSPath), oci.WithWindowsNetworkNamespace(netNSPath),
oci.WithHostname(sandboxConfig.GetHostname()), oci.WithHostname(sandboxConfig.GetHostname()),
) )

2
pkg/cri/server/sandbox_run_windows.go
View File

@@ -51,7 +51,7 @@ func (c *criService) sandboxContainerSpec(id string, config *runtime.PodSandboxC
// Clear the root location since hcsshim expects it. // Clear the root location since hcsshim expects it.
// NOTE: readonly rootfs doesn't work on windows. // NOTE: readonly rootfs doesn't work on windows.
customopts.WithoutRoot, customopts.WithoutRoot,
customopts.WithWindowsNetworkNamespace(nsPath), oci.WithWindowsNetworkNamespace(nsPath),
) )
specOpts = append(specOpts, customopts.WithWindowsDefaultSandboxShares) specOpts = append(specOpts, customopts.WithWindowsDefaultSandboxShares)