fix: docker user mapping windows

Signed-off-by: Ivan Dagelic <[email protected]>
daytonaio · Sep 20, 2024 · 8a6635e · 8a6635e
1 parent 0842ec9
commit 8a6635e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/pkg/docker/create.go b/pkg/docker/create.go
@@ -10,6 +10,7 @@ import (
 	"os"
 	"os/user"
 	"path/filepath"
+	"runtime"
 	"strings"
 	"time"
 
@@ -135,7 +136,11 @@ func (d *DockerClient) cloneProjectRepository(opts *CreateProjectOptions) error
 		}
 	}()
 
-	containerUser, err := d.updateContainerUserUidGid(c.ID, opts)
+	containerUser := "daytona"
+
+	if runtime.GOOS != "windows" {
+		containerUser, err = d.updateContainerUserUidGid(c.ID, opts)
+	}
 
 	res, err := d.ExecSync(c.ID, container.ExecOptions{
 		User: containerUser,

diff --git a/pkg/docker/create_image.go b/pkg/docker/create_image.go
@@ -6,6 +6,7 @@ package docker
 import (
 	"context"
 	"fmt"
+	"runtime"
 	"time"
 
 	"github.com/daytonaio/daytona/pkg/workspace/project"
@@ -66,7 +67,10 @@ func (d *DockerClient) initProjectContainer(opts *CreateProjectOptions) error {
 		}
 	}()
 
-	_, err = d.updateContainerUserUidGid(c.ID, opts)
+	if runtime.GOOS != "windows" {
+		_, err = d.updateContainerUserUidGid(c.ID, opts)
+	}
+
 	err = d.apiClient.ContainerStop(ctx, c.ID, container.StopOptions{})
 	if err != nil {
 		return err