fix: gp rebuild

Author: Andrea Falzetti

components/gitpod-cli/cmd/rebuild.go

@@ -127,7 +127,9 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 		return err
 	}
 
-	err = os.WriteFile(filepath.Join(tmpDir, "Dockerfile"), []byte(baseimage), 0644)
+	tmpDockerfile := filepath.Join(tmpDir, "Dockerfile")
+
+	err = os.WriteFile(tmpDockerfile, []byte(baseimage), 0644)
 	if err != nil {
 		fmt.Println("Could not write the temporary Dockerfile")
 		event.Set("ErrorCode", utils.RebuildErrorCode_DockerfileCannotWirte)
@@ -143,7 +145,7 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 
 	tag := "gp-rebuild-temp-build"
 
-	dockerCmd := exec.Command(dockerPath, "build", "-t", tag, "--progress=tty", ".")
+	dockerCmd := exec.Command(dockerPath, "build", "-f", tmpDockerfile, "-t", tag, "--progress=tty", wsInfo.CheckoutLocation)
 	dockerCmd.Dir = tmpDir
 	dockerCmd.Stdout = os.Stdout
 	dockerCmd.Stderr = os.Stderr
@@ -183,26 +185,24 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 		"--label", "gp-rebuild=true",
 		"-it",
 		tag,
-		"bash",
+		"sh",
 		"-c",
-		fmt.Sprintf("echo '%s'; bash", welcomeMessage),
+		fmt.Sprintf("echo '%s'; sh", welcomeMessage),
 	)
 
 	dockerRunCmd.Stdout = os.Stdout
 	dockerRunCmd.Stderr = os.Stderr
 	dockerRunCmd.Stdin = os.Stdin
 
-	err = dockerRunCmd.Run()
-	if _, ok := err.(*exec.ExitError); ok {
-		fmt.Println("Docker Run Command Failed")
-		event.Set("ErrorCode", utils.RebuildErrorCode_DockerRunFailed)
-		return err
-	} else if err != nil {
+	err = dockerRunCmd.Start()
+	if err != nil {
 		fmt.Println("Docker error")
 		event.Set("ErrorCode", utils.RebuildErrorCode_DockerErr)
 		return err
 	}
 
+	_ = dockerRunCmd.Wait()
+
 	return nil
 }