run git command as gitpod user when using pvc

Author: sagor999

components/content-service/pkg/git/git.go

@@ -92,6 +92,9 @@ type Client struct {
 
 	// UpstreamCloneURI is the fork upstream of a repository
 	UpstreamRemoteURI string
+
+	// if true will run git command as gitpod user (should be executed as root that has access to sudo in this case)
+	RunAsGitpodUser bool
 }
 
 // Status describes the status of a Git repo/working copy akin to "git status"
@@ -197,7 +200,12 @@ func (c *Client) GitWithOutput(ctx context.Context, ignoreErr *string, subcomman
 
 	span.LogKV("args", fullArgs)
 
-	cmd := exec.Command("git", fullArgs...)
+	cmdName := "git"
+	if c.RunAsGitpodUser {
+		cmdName = "sudo"
+		fullArgs = append([]string{"-u", "gitpod", "git"}, fullArgs...)
+	}
+	cmd := exec.Command(cmdName, fullArgs...)
 	cmd.Dir = c.Location
 	cmd.Env = env
 

components/content-service/pkg/initializer/git.go

@@ -76,6 +76,23 @@ func (ws *GitInitializer) Run(ctx context.Context, mappings []archive.IDMapping)
 			return err
 		}
 
+		// make sure that folder itself is owned by gitpod user prior to doing git clone
+		// this is needed as otherwise git clone will fail if the folder is owned by root
+		if ws.RunAsGitpodUser {
+			args := []string{"gitpod", ws.Location}
+			cmd := exec.Command("chown", args...)
+			res, cerr := cmd.CombinedOutput()
+			if cerr != nil && !process.IsNotChildProcess(cerr) {
+				err = git.OpFailedError{
+					Args:       args,
+					ExecErr:    cerr,
+					Output:     string(res),
+					Subcommand: "chown",
+				}
+				return err
+			}
+		}
+
 		log.WithField("stage", "init").WithField("location", ws.Location).Debug("Running git clone on workspace")
 		err = ws.Clone(ctx)
 		if err != nil {

components/content-service/pkg/initializer/initializer.go

@@ -261,10 +261,11 @@ func newGitInitializer(ctx context.Context, loc string, req *csapi.GitInitialize
 			Config:            req.Config.CustomConfig,
 			AuthMethod:        authMethod,
 			AuthProvider:      authProvider,
+			RunAsGitpodUser:   forceGitpodUser,
 		},
 		TargetMode:  targetMode,
 		CloneTarget: req.CloneTaget,
-		Chown:       forceGitpodUser,
+		Chown:       false,
 	}, nil
 }