add outcome to event

Author: Andrea Falzetti

components/gitpod-cli/cmd/rebuild.go

@@ -49,17 +49,15 @@ func TerminateExistingContainer(ctx context.Context) error {
 	return nil
 }
 
-func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClient, event *utils.EventTracker) error {
+func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClient, event *utils.EventTracker) (string, error) {
 	wsInfo, err := supervisorClient.Info.WorkspaceInfo(ctx, &api.WorkspaceInfoRequest{})
 	if err != nil {
-		event.Set("ErrorCode", utils.SystemErrorCode)
-		return err
+		return utils.Outcome_SystemErr, err
 	}
 
 	tmpDir, err := os.MkdirTemp("", "gp-rebuild-*")
 	if err != nil {
-		event.Set("ErrorCode", utils.SystemErrorCode)
-		return err
+		return utils.Outcome_SystemErr, err
 	}
 	defer os.RemoveAll(tmpDir)
 
@@ -70,7 +68,7 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 		fmt.Println("For help check out the reference page:")
 		fmt.Println("https://www.gitpod.io/docs/references/gitpod-yml#gitpodyml")
 		event.Set("ErrorCode", utils.RebuildErrorCode_MalformedGitpodYaml)
-		return err
+		return utils.Outcome_UserErr, err
 	}
 
 	if gitpodConfig == nil {
@@ -81,7 +79,7 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 		fmt.Println("Alternatively, check out the following docs for getting started configuring your project")
 		fmt.Println("https://www.gitpod.io/docs/configure#configure-gitpod")
 		event.Set("ErrorCode", utils.RebuildErrorCode_MissingGitpodYaml)
-		return nil
+		return utils.Outcome_UserErr, nil
 	}
 
 	var baseimage string
@@ -95,13 +93,11 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 
 		if _, err := os.Stat(dockerfilePath); os.IsNotExist(err) {
 			fmt.Println("Your .gitpod.yml points to a Dockerfile that doesn't exist: " + dockerfilePath)
-			event.Set("ErrorCode", utils.RebuildErrorCode_DockerfileNotFound).Send(ctx)
-			return err
+			return utils.Outcome_UserErr, err
 		}
 		dockerfile, err := os.ReadFile(dockerfilePath)
 		if err != nil {
-			event.Set("ErrorCode", utils.RebuildErrorCode_DockerfileCannotRead)
-			return err
+			return utils.Outcome_SystemErr, err
 		}
 		if string(dockerfile) == "" {
 			fmt.Println("Your Gitpod's Dockerfile is empty")
@@ -110,14 +106,13 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 			fmt.Println("https://www.gitpod.io/docs/configure/workspaces/workspace-image#use-a-custom-dockerfile")
 			fmt.Println("")
 			fmt.Println("Once you configure your Dockerfile, re-run this command to validate your changes")
-			event.Set("ErrorCode", utils.RebuildErrorCode_DockerfileEmpty)
-			return nil
+			return utils.Outcome_UserErr, nil
 		}
 		baseimage = "\n" + string(dockerfile) + "\n"
 	default:
 		fmt.Println("Check your .gitpod.yml and make sure the image property is configured correctly")
 		event.Set("ErrorCode", utils.RebuildErrorCode_MalformedGitpodYaml)
-		return err
+		return utils.Outcome_UserErr, nil
 	}
 
 	if baseimage == "" {
@@ -126,23 +121,22 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 		fmt.Println("")
 		fmt.Println("https://www.gitpod.io/docs/configure/workspaces/workspace-image#use-a-public-docker-image")
 		event.Set("ErrorCode", utils.RebuildErrorCode_NoCustomImage)
-		return nil
+		return utils.Outcome_UserErr, nil
 	}
 
 	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_DockerfileCannotWrite)
-		return err
+		return utils.Outcome_SystemErr, err
 	}
 
 	dockerPath, err := exec.LookPath("docker")
 	if err != nil {
 		fmt.Println("Docker is not installed in your workspace")
 		event.Set("ErrorCode", utils.RebuildErrorCode_DockerNotFound)
-		return err
+		return utils.Outcome_SystemErr, err
 	}
 
 	tag := "gp-rebuild-temp-build"
@@ -155,20 +149,19 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 	err = dockerCmd.Run()
 	if _, ok := err.(*exec.ExitError); ok {
 		fmt.Println("Image Build Failed")
-		event.Set("ErrorCode", utils.RebuildErrorCode_DockerBuildFailed)
-		return nil
+		event.Set("ErrorCode", utils.RebuildErrorCode_ImageBuildFailed)
+		return utils.Outcome_UserErr, nil
 	} else if err != nil {
 		fmt.Println("Docker error")
 		event.Set("ErrorCode", utils.RebuildErrorCode_DockerErr)
-		return err
+		return utils.Outcome_SystemErr, err
 	}
 	ImageBuildDuration := time.Since(imageBuildStartTime).Milliseconds()
 	event.Set("ImageBuildDuration", ImageBuildDuration)
 
 	err = TerminateExistingContainer(ctx)
 	if err != nil {
-		event.Set("ErrorCode", utils.SystemErrorCode)
-		return err
+		return utils.Outcome_SystemErr, err
 	}
 
 	welcomeMessage := strings.Join([]string{
@@ -209,12 +202,12 @@ func runRebuild(ctx context.Context, supervisorClient *supervisor.SupervisorClie
 	if err != nil {
 		fmt.Println("Failed to run docker container")
 		event.Set("ErrorCode", utils.RebuildErrorCode_DockerRunFailed)
-		return err
+		return utils.Outcome_UserErr, err
 	}
 
 	_ = dockerRunCmd.Wait()
 
-	return nil
+	return utils.Outcome_Success, nil
 }
 
 var buildCmd = &cobra.Command{
@@ -240,10 +233,18 @@ var buildCmd = &cobra.Command{
 			Command: cmd.Name(),
 		})
 
-		err = runRebuild(ctx, supervisorClient, event)
-		if err != nil && event.Data.ErrorCode == "" {
-			event.Set("ErrorCode", utils.SystemErrorCode)
+		outcome, err := runRebuild(ctx, supervisorClient, event)
+		event.Set("Outcome", outcome)
+
+		if outcome != utils.Outcome_Success && event.Data.ErrorCode == "" {
+			switch outcome {
+			case utils.Outcome_UserErr:
+				event.Set("ErrorCode", utils.UserErrorCode)
+			case utils.Outcome_SystemErr:
+				event.Set("ErrorCode", utils.SystemErrorCode)
+			}
 		}
+
 		event.Send(ctx)
 
 		if err != nil {

components/gitpod-cli/pkg/utils/trackEvent.go

@@ -15,22 +15,29 @@ import (
 	log "github.com/sirupsen/logrus"
 )
 
+const (
+	Outcome_Success   = "success"
+	Outcome_UserErr   = "user_error"
+	Outcome_SystemErr = "system_error"
+)
+
 const (
 	// System
 	SystemErrorCode = "system_error"
+	UserErrorCode   = "user_error"
 
 	// Rebuild
-	RebuildErrorCode_DockerBuildFailed     = "rebuild_docker_build_failed"
-	RebuildErrorCode_DockerErr             = "rebuild_docker_err"
-	RebuildErrorCode_DockerfileCannotRead  = "rebuild_dockerfile_cannot_read"
-	RebuildErrorCode_DockerfileCannotWrite = "rebuild_dockerfile_cannot_write"
-	RebuildErrorCode_DockerfileEmpty       = "rebuild_dockerfile_empty"
-	RebuildErrorCode_DockerfileNotFound    = "rebuild_dockerfile_not_found"
-	RebuildErrorCode_DockerNotFound        = "rebuild_docker_not_found"
-	RebuildErrorCode_DockerRunFailed       = "rebuild_docker_run_failed"
-	RebuildErrorCode_MalformedGitpodYaml   = "rebuild_malformed_gitpod_yaml"
-	RebuildErrorCode_MissingGitpodYaml     = "rebuild_missing_gitpod_yaml"
-	RebuildErrorCode_NoCustomImage         = "rebuild_no_custom_image"
+	RebuildErrorCode_ImageBuildFailed = "rebuild_image_build_failed"
+	RebuildErrorCode_DockerErr        = "rebuild_docker_err"
+	// RebuildErrorCode_DockerfileCannotRead  = "rebuild_dockerfile_cannot_read"
+	// RebuildErrorCode_DockerfileCannotWrite = "rebuild_dockerfile_cannot_write"
+	// RebuildErrorCode_DockerfileEmpty       = "rebuild_dockerfile_empty"
+	// RebuildErrorCode_DockerfileNotFound    = "rebuild_dockerfile_not_found"
+	RebuildErrorCode_DockerNotFound      = "rebuild_docker_not_found"
+	RebuildErrorCode_DockerRunFailed     = "rebuild_docker_run_failed"
+	RebuildErrorCode_MalformedGitpodYaml = "rebuild_malformed_gitpod_yaml"
+	RebuildErrorCode_MissingGitpodYaml   = "rebuild_missing_gitpod_yaml"
+	RebuildErrorCode_NoCustomImage       = "rebuild_no_custom_image"
 )
 
 type TrackCommandUsageParams struct {
@@ -41,6 +48,7 @@ type TrackCommandUsageParams struct {
 	InstanceId         string `json:"instanceId,omitempty"`
 	Timestamp          int64  `json:"timestamp,omitempty"`
 	ImageBuildDuration int64  `json:"imageBuildDuration,omitempty"`
+	Outcome            string `json:"outcome,omitempty"`
 }
 
 type EventTracker struct {
@@ -96,6 +104,8 @@ func (t *EventTracker) Set(key string, value interface{}) *EventTracker {
 		t.Data.InstanceId = value.(string)
 	case "ImageBuildDuration":
 		t.Data.ImageBuildDuration = value.(int64)
+	case "Outcome":
+		t.Data.Outcome = value.(string)
 	}
 	return t
 }