Commit 6a34bb7b authored by Brad Davidson's avatar Brad Davidson Committed by Brad Davidson

Fix containerd hosts.toml path on windows

Signed-off-by: 's avatarBrad Davidson <brad.davidson@rancher.com> (cherry picked from commit bf97b8fa) Signed-off-by: 's avatarBrad Davidson <brad.davidson@rancher.com>
parent 1878a4c5
......@@ -53,7 +53,7 @@ func writeContainerdHosts(cfg *config.Node, containerdConfig templates.Container
// Write out new templates
for host, config := range hosts {
hostDir := filepath.Join(cfg.Containerd.Registry, host)
hostDir := filepath.Join(cfg.Containerd.Registry, hostDirectory(host))
hostsFile := filepath.Join(hostDir, "hosts.toml")
hostsTemplate, err := templates.ParseHostsTemplateFromConfig(templates.HostsTomlTemplate, config)
if err != nil {
......
......@@ -27,6 +27,12 @@ const (
runtimesPath = "/usr/local/nvidia/toolkit:/opt/kwasm/bin"
)
// hostDirectory returns the name of the host dir for a given registry.
// This is a no-op on linux, as all possible host:port strings are valid paths.
func hostDirectory(host string) string {
return host
}
func getContainerdArgs(cfg *config.Node) []string {
args := []string{
"containerd",
......
......@@ -1500,7 +1500,7 @@ func Test_UnitGetHostConfigs(t *testing.T) {
for host, config := range got {
hostsTemplate, err := templates.ParseHostsTemplateFromConfig(templates.HostsTomlTemplate, config)
assert.NoError(t, err, "ParseHostTemplateFromConfig for %s", host)
t.Logf("%s/hosts.d/%s/hosts.toml\n%s", tempDir, host, hostsTemplate)
t.Logf("%s/hosts.d/%s/hosts.toml\n%s", tempDir, hostDirectory(host), hostsTemplate)
}
// Confirm that the main containerd config.toml renders properly
......
......@@ -4,6 +4,8 @@
package containerd
import (
"net"
"github.com/containerd/containerd"
"github.com/k3s-io/k3s/pkg/agent/templates"
"github.com/k3s-io/k3s/pkg/daemons/config"
......@@ -13,6 +15,16 @@ import (
"k8s.io/kubernetes/pkg/kubelet/util"
)
// hostDirectory returns the name of the host dir for a given registry.
// Colons are not allowed in windows paths, so convert `:port` to `_port_`.
// Ref: https://github.com/containerd/containerd/blob/v1.7.25/remotes/docker/config/hosts.go#L291-L298
func hostDirectory(host string) string {
if host, port, err := net.SplitHostPort(host); err == nil && port != "" {
return host + "_" + port + "_"
}
return host
}
func getContainerdArgs(cfg *config.Node) []string {
args := []string{
"containerd",
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment