Add Windows Support

Signed-off-by: Olli Janatuinen <[email protected]>

Author: olljanat

.gitignore

@@ -1,2 +1,3 @@
 .vagrant/
 /containerd-driver
+/containerd-driver.exe

Makefile

@@ -16,6 +16,7 @@ clean:
 .PHONY: build
 build:
 	$(GOLANG) build -o $(BINARY) .
+	GOOS=windows $(GOLANG) build -o $(BINARY).exe .
 
 .PHONY: test
 test:

containerd/driver.go

@@ -38,7 +38,6 @@ import (
 	"github.com/hashicorp/nomad/plugins/drivers"
 	"github.com/hashicorp/nomad/plugins/shared/hclspec"
 	"github.com/hashicorp/nomad/plugins/shared/structs"
-	"github.com/opencontainers/runc/libcontainer/cgroups"
 )
 
 const (
@@ -263,16 +262,7 @@ type Driver struct {
 func NewPlugin(logger log.Logger) drivers.DriverPlugin {
 	ctx, cancel := context.WithCancel(context.Background())
 	logger = logger.Named(PluginName)
-
-	// Calls to containerd API are namespaced.
-	// "nomad" is the namespace that will be used for all nomad-driver-containerd
-	// related containerd API calls.
-	namespace := "nomad"
-	// Unless we are operating in cgroups.v2 mode, in which case we use the
-	// name "nomad.slice", which ends up being the cgroup parent.
-	if cgroups.IsCgroup2UnifiedMode() {
-		namespace = "nomad.slice"
-	}
+	namespace := getNamespaceName()
 	ctxContainerd := namespaces.WithNamespace(context.Background(), namespace)
 
 	return &Driver{
@@ -451,13 +441,7 @@ func (d *Driver) StartTask(cfg *drivers.TaskConfig) (*drivers.TaskHandle, *drive
 	handle := drivers.NewTaskHandle(taskHandleVersion)
 	handle.Config = cfg
 
-	// Use Nomad's docker naming convention for the container name
-	// https://www.nomadproject.io/docs/drivers/docker#container-name
-	containerName := cfg.Name + "-" + cfg.AllocID
-	if cgroups.IsCgroup2UnifiedMode() {
-		// In cgroup.v2 mode, the name is slightly different.
-		containerName = fmt.Sprintf("%s.%s.scope", cfg.AllocID, cfg.Name)
-	}
+	containerName := getContainerName(cfg.Name, cfg.AllocID)
 	containerConfig.ContainerName = containerName
 
 	var err error

containerd/driver_linux.go

@@ -0,0 +1,31 @@
+package containerd
+
+import (
+	"fmt"
+
+	"github.com/opencontainers/runc/libcontainer/cgroups"
+)
+
+func getNamespaceName() string {
+	// Calls to containerd API are namespaced.
+	// "nomad" is the namespace that will be used for all nomad-driver-containerd
+	// related containerd API calls.
+	namespace := "nomad"
+	// Unless we are operating in cgroups.v2 mode, in which case we use the
+	// name "nomad.slice", which ends up being the cgroup parent.
+	if cgroups.IsCgroup2UnifiedMode() {
+		namespace = "nomad.slice"
+	}
+	return namespace
+}
+
+func getContainerName(name, allocID string) string {
+	// Use Nomad's docker naming convention for the container name
+	// https://www.nomadproject.io/docs/drivers/docker#container-name
+	containerName := name + "-" + allocID
+	if cgroups.IsCgroup2UnifiedMode() {
+		// In cgroup.v2 mode, the name is slightly different.
+		containerName = fmt.Sprintf("%s.%s.scope", allocID, name)
+	}
+	return containerName
+}

containerd/driver_windows.go

@@ -0,0 +1,10 @@
+package containerd
+
+func getNamespaceName() string {
+	return "nomad"
+}
+
+func getContainerName(name, allocID string) string {
+	containerName := name + "-" + allocID
+	return containerName
+}