Improve error messages for kind and minikube install (#585)

Alexander-Kita · web-flow · commit 7b2c52fd4cea · 2025-07-01T12:58:53.000Z
* improve minikube quickstart errors

* improve kind quickstart errors

* remove error from messages
diff --git a/pkg/kind/kind.go b/pkg/kind/kind.go
@@ -68,7 +68,7 @@ func SetUp(name, kVersion string, installServing, installEventing, installKindRe
 	}
 
 	if err := createKindCluster(installKindRegistry, installKindExtraMountHostPath, installKindExtraMountContainerPath); err != nil {
-		return fmt.Errorf("creating cluster: %w", err)
+		return fmt.Errorf("failed to create kind cluster: %w", err)
 	}
 	if installKnative {
 		if installServing {
@@ -80,18 +80,18 @@ func SetUp(name, kVersion string, installServing, installEventing, installKindRe
 				registries = fmt.Sprintf("localhost:%s", container_reg_port)
 			}
 			if err := install.Serving(registries); err != nil {
-				return fmt.Errorf("install serving: %w", err)
+				return fmt.Errorf("failed to install serving to kind cluster %s: %w", clusterName, err)
 			}
 			if err := install.Kourier(); err != nil {
-				return fmt.Errorf("install kourier: %w", err)
+				return fmt.Errorf("failed to install kourier to kind cluster %s: %w", clusterName, err)
 			}
 			if err := install.KourierKind(); err != nil {
-				return fmt.Errorf("configure kourier: %w", err)
+				return fmt.Errorf("failed while configuring kourier for kind cluster %s: %w", clusterName, err)
 			}
 		}
 		if installEventing {
 			if err := install.Eventing(); err != nil {
-				return fmt.Errorf("install eventing: %w", err)
+				return fmt.Errorf("failed to install eventing to king cluster %s: %w", clusterName, err)
 			}
 		}
 	}
@@ -109,7 +109,7 @@ func createKindCluster(registry bool, extraMountHostPath string, extraMountConta
 	}
 	fmt.Println("✅ Checking dependencies...")
 	if err := checkKindVersion(); err != nil {
-		return fmt.Errorf("kind version: %w", err)
+		return fmt.Errorf("unable to check kind version: %w", err)
 	}
 	if registry {
 		fmt.Println("💽 Installing local registry...")
@@ -127,7 +127,7 @@ func createKindCluster(registry bool, extraMountHostPath string, extraMountConta
 	}
 
 	if err := checkForExistingCluster(registry, extraMountHostPath, extraMountContainerPath); err != nil {
-		return fmt.Errorf("existing cluster: %w", err)
+		return fmt.Errorf("failed while handling or checking for existing kind cluster: %w", err)
 	}
 
 	return nil
@@ -237,13 +237,13 @@ func checkKindVersion() error {
 	versionCheck := exec.Command("kind", "version", "-q")
 	out, err := versionCheck.CombinedOutput()
 	if err != nil {
-		return fmt.Errorf("kind version: %w", err)
+		return fmt.Errorf("failed to get kind version: %w", err)
 	}
 	fmt.Printf("    Kind version is: %s", string(out))
 
 	userKindVersion, err := parseKindVersion(string(out))
 	if err != nil {
-		return fmt.Errorf("parsing kind version: %w", err)
+		return fmt.Errorf("unable to parse kind version: %w", err)
 	}
 	if userKindVersion < kindVersion {
 		var resp string
@@ -267,7 +267,7 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun
 	getClusters := exec.Command("kind", "get", "clusters", "-q")
 	out, err := getClusters.CombinedOutput()
 	if err != nil {
-		return fmt.Errorf("check cluster: %w", err)
+		return fmt.Errorf("unable to get kind clusters: %w", err)
 	}
 	// TODO Add tests for regex
 	r := regexp.MustCompile(fmt.Sprintf(`(?m)^%s\n`, clusterName))
@@ -278,7 +278,7 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun
 		fmt.Scanf("%s", &resp)
 		if resp == "y" || resp == "Y" {
 			if err := recreateCluster(registry, extraMountHostPath, extraMountContainerPath); err != nil {
-				return fmt.Errorf("new cluster: %w", err)
+				return fmt.Errorf("failed while recreating kind cluster %s: %w", clusterName, err)
 			}
 		} else {
 			fmt.Println("\n    Installation skipped")
@@ -287,14 +287,14 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun
 			namespaces := string(output)
 			if err != nil {
 				fmt.Println(string(output))
-				return fmt.Errorf("check existing cluster: %w", err)
+				return fmt.Errorf("unable to get kubernetes namspaces for kind cluster %s: %w", clusterName, err)
 			}
 			if strings.Contains(namespaces, "knative") {
 				fmt.Print("Knative installation already exists.\nDelete and recreate the cluster [y/N]: ")
 				fmt.Scanf("%s", &resp)
 				if resp == "y" || resp == "Y" {
 					if err := recreateCluster(registry, extraMountHostPath, extraMountContainerPath); err != nil {
-						return fmt.Errorf("new cluster: %w", err)
+						return fmt.Errorf("failed to recreate kind cluster: %w", err)
 					}
 				} else {
 					fmt.Println("Skipping installation")
@@ -307,11 +307,11 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun
 	} else {
 		dcli, err := dclient.NewClientWithOpts(dclient.FromEnv, dclient.WithAPIVersionNegotiation())
 		if err != nil {
-			return fmt.Errorf("new cluster: %w", err)
+			return fmt.Errorf("failed to initialize new api client: %w", err)
 		}
 
 		if err := createNewCluster(extraMountHostPath, extraMountContainerPath); err != nil {
-			return fmt.Errorf("new cluster: %w", err)
+			return fmt.Errorf("%w", err)
 		}
 		if registry {
 			if err := createLocalRegistry(dcli); err != nil {
@@ -332,25 +332,25 @@ func recreateCluster(registry bool, extraMountHostPath string, extraMountContain
 
 	dcli, err := dclient.NewClientWithOpts(dclient.FromEnv, dclient.WithAPIVersionNegotiation())
 	if err != nil {
-		return fmt.Errorf("delete cluster: %w", err)
+		return fmt.Errorf("failed to initialize new api client: %w", err)
 	}
 
 	deleteCluster := exec.Command("kind", "delete", "cluster", "--name", clusterName)
 	if err := deleteCluster.Run(); err != nil {
-		return fmt.Errorf("delete cluster: %w", err)
+		return fmt.Errorf("failed to delete kind cluster %s: %w", clusterName, err)
 	}
 	if err := deleteContainerRegistry(dcli); err != nil {
-		return fmt.Errorf("delete container registry: %w", err)
+		return fmt.Errorf("failed to delete container registry: %w", err)
 	}
 	if err := createNewCluster(extraMountHostPath, extraMountContainerPath); err != nil {
-		return fmt.Errorf("new cluster: %w", err)
+		return fmt.Errorf("%w", err)
 	}
 	if registry {
 		if err := createLocalRegistry(dcli); err != nil {
 			return fmt.Errorf("%w", err)
 		}
 		if err := connectLocalRegistry(dcli); err != nil {
-			return fmt.Errorf("local-registry: %w", err)
+			return fmt.Errorf("unable to connect local-registry: %w", err)
 		}
 	}
 	return nil
@@ -391,7 +391,7 @@ nodes:
 	createCluster := exec.Command("kind", "create", "cluster", "--wait=120s", "--config=-")
 	createCluster.Stdin = strings.NewReader(config)
 	if err := runCommandWithOutput(createCluster); err != nil {
-		return fmt.Errorf("kind create: %w", err)
+		return fmt.Errorf("failed to create kind cluster %s: %w", clusterName, err)
 	}
 
 	return nil
diff --git a/pkg/minikube/minikube.go b/pkg/minikube/minikube.go
@@ -59,7 +59,7 @@ func SetUp(name, kVersion string, installServing, installEventing bool) error {
 	}
 
 	if err := createMinikubeCluster(); err != nil {
-		return fmt.Errorf("creating cluster: %w", err)
+		return fmt.Errorf("failed to create minikube cluster: %w", err)
 	}
 	fmt.Print("\n")
 	fmt.Println("To finish setting up networking for minikube, run the following command in a separate terminal window:")
@@ -70,18 +70,18 @@ func SetUp(name, kVersion string, installServing, installEventing bool) error {
 	if installKnative {
 		if installServing {
 			if err := install.Serving(""); err != nil {
-				return fmt.Errorf("install serving: %w", err)
+				return fmt.Errorf("failed to install serving to minikube cluster %s: %w", clusterName, err)
 			}
 			if err := install.Kourier(); err != nil {
-				return fmt.Errorf("install kourier: %w", err)
+				return fmt.Errorf("failed to install kourier to minikube cluster %s: %w", clusterName, err)
 			}
 			if err := install.KourierMinikube(); err != nil {
-				return fmt.Errorf("configure kourier: %w", err)
+				return fmt.Errorf("failed while configuring kourier for minikube cluster %s: %w", clusterName, err)
 			}
 		}
 		if installEventing {
 			if err := install.Eventing(); err != nil {
-				return fmt.Errorf("install eventing: %w", err)
+				return fmt.Errorf("failed to install eventing to minikube cluster %s: %w", clusterName, err)
 			}
 		}
 	}
@@ -95,10 +95,10 @@ func SetUp(name, kVersion string, installServing, installEventing bool) error {
 
 func createMinikubeCluster() error {
 	if err := checkMinikubeVersion(); err != nil {
-		return fmt.Errorf("minikube version: %w", err)
+		return fmt.Errorf("unable to get minikube version: %w", err)
 	}
 	if err := checkForExistingCluster(); err != nil {
-		return fmt.Errorf("existing cluster: %w", err)
+		return fmt.Errorf("failure while handling or checking for existing minikube cluster: %w", err)
 	}
 	return nil
 }
@@ -109,13 +109,13 @@ func checkMinikubeVersion() error {
 	versionCheck := exec.Command("minikube", "version", "--short")
 	out, err := versionCheck.CombinedOutput()
 	if err != nil {
-		return fmt.Errorf("minikube version: %w", err)
+		return fmt.Errorf("unable to check minikube version: %w", err)
 	}
 	fmt.Printf("Minikube version is: %s\n", string(out))
 
 	userMinikubeVersion, err := parseMinikubeVersion(string(out))
 	if err != nil {
-		return fmt.Errorf("parsing minikube version: %w", err)
+		return fmt.Errorf("unable to parse minikube version: %w", err)
 	}
 	if userMinikubeVersion < minikubeVersion {
 		var resp string
@@ -143,7 +143,7 @@ func checkForExistingCluster() error {
 		// if there were no profiles, we simply want to create a new one and not stop the install
 		// so if the error is the "MK_USAGE_NO_PROFILE" error, we ignore it and continue onwards
 		if !strings.Contains(string(out), "MK_USAGE_NO_PROFILE") {
-			return fmt.Errorf("check cluster: %w", err)
+			return fmt.Errorf("failed to get existing minikube profiles: %w", err)
 		}
 	}
 	// TODO Add tests for regex
@@ -160,7 +160,7 @@ func checkForExistingCluster() error {
 			namespaces := string(output)
 			if err != nil {
 				fmt.Println(string(output))
-				return fmt.Errorf("check existing cluster: %w", err)
+				return fmt.Errorf("unable to get existing kubernetes namespaces for minikube cluster %s: %w", clusterName, err)
 			}
 			if strings.Contains(namespaces, "knative") {
 				fmt.Print("Knative installation already exists.\nDelete and recreate the cluster [y/N]: ")
@@ -171,20 +171,20 @@ func checkForExistingCluster() error {
 					return nil
 				} else {
 					if err := recreateCluster(); err != nil {
-						return fmt.Errorf("failed recreating cluster: %w", err)
+						return fmt.Errorf("failed while recreating minikube cluster: %w", err)
 					}
 				}
 			}
 			return nil
 		}
 		if err := recreateCluster(); err != nil {
-			return fmt.Errorf("failed recreating cluster: %w", err)
+			return fmt.Errorf("failed while recreating minikube cluster: %w", err)
 		}
 		return nil
 	}
 
 	if err := createNewCluster(); err != nil {
-		return fmt.Errorf("new cluster: %w", err)
+		return fmt.Errorf("%w", err)
 	}
 
 	return nil
@@ -223,7 +223,7 @@ func createNewCluster() error {
 		"--insecure-registry", "10.0.0.0/24",
 		"--addons=registry")
 	if err := runCommandWithOutput(createCluster); err != nil {
-		return fmt.Errorf("minikube create: %w", err)
+		return fmt.Errorf("failed to create new minikube cluster %s: %w", clusterName, err)
 	}
 
 	return nil
@@ -264,10 +264,10 @@ func recreateCluster() error {
 	fmt.Println("deleting cluster...")
 	deleteCluster := exec.Command("minikube", "delete", "--profile", clusterName)
 	if err := deleteCluster.Run(); err != nil {
-		return fmt.Errorf("delete cluster: %w", err)
+		return fmt.Errorf("failed to delete minikube cluster %s: %w", clusterName, err)
 	}
 	if err := createNewCluster(); err != nil {
-		return fmt.Errorf("new cluster: %w", err)
+		return fmt.Errorf("%w", err)
 	}
 	return nil
 }

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ func SetUp(name, kVersion string, installServing, installEventing, installKindRe`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`if err := createKindCluster(installKindRegistry, installKindExtraMountHostPath, installKindExtraMountContainerPath); err != nil {`
`71`		`- return fmt.Errorf("creating cluster: %w", err)`
	`71`	`+ return fmt.Errorf("failed to create kind cluster: %w", err)`
`72`	`72`	`}`
`73`	`73`	`if installKnative {`
`74`	`74`	`if installServing {`
`@@ -80,18 +80,18 @@ func SetUp(name, kVersion string, installServing, installEventing, installKindRe`
`80`	`80`	`registries = fmt.Sprintf("localhost:%s", container_reg_port)`
`81`	`81`	`}`
`82`	`82`	`if err := install.Serving(registries); err != nil {`
`83`		`- return fmt.Errorf("install serving: %w", err)`
	`83`	`+ return fmt.Errorf("failed to install serving to kind cluster %s: %w", clusterName, err)`
`84`	`84`	`}`
`85`	`85`	`if err := install.Kourier(); err != nil {`
`86`		`- return fmt.Errorf("install kourier: %w", err)`
	`86`	`+ return fmt.Errorf("failed to install kourier to kind cluster %s: %w", clusterName, err)`
`87`	`87`	`}`
`88`	`88`	`if err := install.KourierKind(); err != nil {`
`89`		`- return fmt.Errorf("configure kourier: %w", err)`
	`89`	`+ return fmt.Errorf("failed while configuring kourier for kind cluster %s: %w", clusterName, err)`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`	`if installEventing {`
`93`	`93`	`if err := install.Eventing(); err != nil {`
`94`		`- return fmt.Errorf("install eventing: %w", err)`
	`94`	`+ return fmt.Errorf("failed to install eventing to king cluster %s: %w", clusterName, err)`
`95`	`95`	`}`
`96`	`96`	`}`
`97`	`97`	`}`
`@@ -109,7 +109,7 @@ func createKindCluster(registry bool, extraMountHostPath string, extraMountConta`
`109`	`109`	`}`
`110`	`110`	`fmt.Println("✅ Checking dependencies...")`
`111`	`111`	`if err := checkKindVersion(); err != nil {`
`112`		`- return fmt.Errorf("kind version: %w", err)`
	`112`	`+ return fmt.Errorf("unable to check kind version: %w", err)`
`113`	`113`	`}`
`114`	`114`	`if registry {`
`115`	`115`	`fmt.Println("💽 Installing local registry...")`
`@@ -127,7 +127,7 @@ func createKindCluster(registry bool, extraMountHostPath string, extraMountConta`
`127`	`127`	`}`
`128`	`128`
`129`	`129`	`if err := checkForExistingCluster(registry, extraMountHostPath, extraMountContainerPath); err != nil {`
`130`		`- return fmt.Errorf("existing cluster: %w", err)`
	`130`	`+ return fmt.Errorf("failed while handling or checking for existing kind cluster: %w", err)`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`return nil`
`@@ -237,13 +237,13 @@ func checkKindVersion() error {`
`237`	`237`	`versionCheck := exec.Command("kind", "version", "-q")`
`238`	`238`	`out, err := versionCheck.CombinedOutput()`
`239`	`239`	`if err != nil {`
`240`		`- return fmt.Errorf("kind version: %w", err)`
	`240`	`+ return fmt.Errorf("failed to get kind version: %w", err)`
`241`	`241`	`}`
`242`	`242`	`fmt.Printf(" Kind version is: %s", string(out))`
`243`	`243`
`244`	`244`	`userKindVersion, err := parseKindVersion(string(out))`
`245`	`245`	`if err != nil {`
`246`		`- return fmt.Errorf("parsing kind version: %w", err)`
	`246`	`+ return fmt.Errorf("unable to parse kind version: %w", err)`
`247`	`247`	`}`
`248`	`248`	`if userKindVersion < kindVersion {`
`249`	`249`	`var resp string`
`@@ -267,7 +267,7 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun`
`267`	`267`	`getClusters := exec.Command("kind", "get", "clusters", "-q")`
`268`	`268`	`out, err := getClusters.CombinedOutput()`
`269`	`269`	`if err != nil {`
`270`		`- return fmt.Errorf("check cluster: %w", err)`
	`270`	`+ return fmt.Errorf("unable to get kind clusters: %w", err)`
`271`	`271`	`}`
`272`	`272`	`// TODO Add tests for regex`
`273`	`273`	r := regexp.MustCompile(fmt.Sprintf(`(?m)^%s\n`, clusterName))
`@@ -278,7 +278,7 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun`
`278`	`278`	`fmt.Scanf("%s", &resp)`
`279`	`279`	`if resp == "y" \|\| resp == "Y" {`
`280`	`280`	`if err := recreateCluster(registry, extraMountHostPath, extraMountContainerPath); err != nil {`
`281`		`- return fmt.Errorf("new cluster: %w", err)`
	`281`	`+ return fmt.Errorf("failed while recreating kind cluster %s: %w", clusterName, err)`
`282`	`282`	`}`
`283`	`283`	`} else {`
`284`	`284`	`fmt.Println("\n Installation skipped")`
`@@ -287,14 +287,14 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun`
`287`	`287`	`namespaces := string(output)`
`288`	`288`	`if err != nil {`
`289`	`289`	`fmt.Println(string(output))`
`290`		`- return fmt.Errorf("check existing cluster: %w", err)`
	`290`	`+ return fmt.Errorf("unable to get kubernetes namspaces for kind cluster %s: %w", clusterName, err)`
`291`	`291`	`}`
`292`	`292`	`if strings.Contains(namespaces, "knative") {`
`293`	`293`	`fmt.Print("Knative installation already exists.\nDelete and recreate the cluster [y/N]: ")`
`294`	`294`	`fmt.Scanf("%s", &resp)`
`295`	`295`	`if resp == "y" \|\| resp == "Y" {`
`296`	`296`	`if err := recreateCluster(registry, extraMountHostPath, extraMountContainerPath); err != nil {`
`297`		`- return fmt.Errorf("new cluster: %w", err)`
	`297`	`+ return fmt.Errorf("failed to recreate kind cluster: %w", err)`
`298`	`298`	`}`
`299`	`299`	`} else {`
`300`	`300`	`fmt.Println("Skipping installation")`
`@@ -307,11 +307,11 @@ func checkForExistingCluster(registry bool, extraMountHostPath string, extraMoun`
`307`	`307`	`} else {`
`308`	`308`	`dcli, err := dclient.NewClientWithOpts(dclient.FromEnv, dclient.WithAPIVersionNegotiation())`
`309`	`309`	`if err != nil {`
`310`		`- return fmt.Errorf("new cluster: %w", err)`
	`310`	`+ return fmt.Errorf("failed to initialize new api client: %w", err)`
`311`	`311`	`}`
`312`	`312`
`313`	`313`	`if err := createNewCluster(extraMountHostPath, extraMountContainerPath); err != nil {`
`314`		`- return fmt.Errorf("new cluster: %w", err)`
	`314`	`+ return fmt.Errorf("%w", err)`
`315`	`315`	`}`
`316`	`316`	`if registry {`
`317`	`317`	`if err := createLocalRegistry(dcli); err != nil {`
`@@ -332,25 +332,25 @@ func recreateCluster(registry bool, extraMountHostPath string, extraMountContain`
`332`	`332`
`333`	`333`	`dcli, err := dclient.NewClientWithOpts(dclient.FromEnv, dclient.WithAPIVersionNegotiation())`
`334`	`334`	`if err != nil {`
`335`		`- return fmt.Errorf("delete cluster: %w", err)`
	`335`	`+ return fmt.Errorf("failed to initialize new api client: %w", err)`
`336`	`336`	`}`
`337`	`337`
`338`	`338`	`deleteCluster := exec.Command("kind", "delete", "cluster", "--name", clusterName)`
`339`	`339`	`if err := deleteCluster.Run(); err != nil {`
`340`		`- return fmt.Errorf("delete cluster: %w", err)`
	`340`	`+ return fmt.Errorf("failed to delete kind cluster %s: %w", clusterName, err)`
`341`	`341`	`}`
`342`	`342`	`if err := deleteContainerRegistry(dcli); err != nil {`
`343`		`- return fmt.Errorf("delete container registry: %w", err)`
	`343`	`+ return fmt.Errorf("failed to delete container registry: %w", err)`
`344`	`344`	`}`
`345`	`345`	`if err := createNewCluster(extraMountHostPath, extraMountContainerPath); err != nil {`
`346`		`- return fmt.Errorf("new cluster: %w", err)`
	`346`	`+ return fmt.Errorf("%w", err)`
`347`	`347`	`}`
`348`	`348`	`if registry {`
`349`	`349`	`if err := createLocalRegistry(dcli); err != nil {`
`350`	`350`	`return fmt.Errorf("%w", err)`
`351`	`351`	`}`
`352`	`352`	`if err := connectLocalRegistry(dcli); err != nil {`
`353`		`- return fmt.Errorf("local-registry: %w", err)`
	`353`	`+ return fmt.Errorf("unable to connect local-registry: %w", err)`
`354`	`354`	`}`
`355`	`355`	`}`
`356`	`356`	`return nil`
`@@ -391,7 +391,7 @@ nodes:`
`391`	`391`	`createCluster := exec.Command("kind", "create", "cluster", "--wait=120s", "--config=-")`
`392`	`392`	`createCluster.Stdin = strings.NewReader(config)`
`393`	`393`	`if err := runCommandWithOutput(createCluster); err != nil {`
`394`		`- return fmt.Errorf("kind create: %w", err)`
	`394`	`+ return fmt.Errorf("failed to create kind cluster %s: %w", clusterName, err)`
`395`	`395`	`}`
`396`	`396`
`397`	`397`	`return nil`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ func SetUp(name, kVersion string, installServing, installEventing bool) error {`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`if err := createMinikubeCluster(); err != nil {`
`62`		`- return fmt.Errorf("creating cluster: %w", err)`
	`62`	`+ return fmt.Errorf("failed to create minikube cluster: %w", err)`
`63`	`63`	`}`
`64`	`64`	`fmt.Print("\n")`
`65`	`65`	`fmt.Println("To finish setting up networking for minikube, run the following command in a separate terminal window:")`
`@@ -70,18 +70,18 @@ func SetUp(name, kVersion string, installServing, installEventing bool) error {`
`70`	`70`	`if installKnative {`
`71`	`71`	`if installServing {`
`72`	`72`	`if err := install.Serving(""); err != nil {`
`73`		`- return fmt.Errorf("install serving: %w", err)`
	`73`	`+ return fmt.Errorf("failed to install serving to minikube cluster %s: %w", clusterName, err)`
`74`	`74`	`}`
`75`	`75`	`if err := install.Kourier(); err != nil {`
`76`		`- return fmt.Errorf("install kourier: %w", err)`
	`76`	`+ return fmt.Errorf("failed to install kourier to minikube cluster %s: %w", clusterName, err)`
`77`	`77`	`}`
`78`	`78`	`if err := install.KourierMinikube(); err != nil {`
`79`		`- return fmt.Errorf("configure kourier: %w", err)`
	`79`	`+ return fmt.Errorf("failed while configuring kourier for minikube cluster %s: %w", clusterName, err)`
`80`	`80`	`}`
`81`	`81`	`}`
`82`	`82`	`if installEventing {`
`83`	`83`	`if err := install.Eventing(); err != nil {`
`84`		`- return fmt.Errorf("install eventing: %w", err)`
	`84`	`+ return fmt.Errorf("failed to install eventing to minikube cluster %s: %w", clusterName, err)`
`85`	`85`	`}`
`86`	`86`	`}`
`87`	`87`	`}`
`@@ -95,10 +95,10 @@ func SetUp(name, kVersion string, installServing, installEventing bool) error {`
`95`	`95`
`96`	`96`	`func createMinikubeCluster() error {`
`97`	`97`	`if err := checkMinikubeVersion(); err != nil {`
`98`		`- return fmt.Errorf("minikube version: %w", err)`
	`98`	`+ return fmt.Errorf("unable to get minikube version: %w", err)`
`99`	`99`	`}`
`100`	`100`	`if err := checkForExistingCluster(); err != nil {`
`101`		`- return fmt.Errorf("existing cluster: %w", err)`
	`101`	`+ return fmt.Errorf("failure while handling or checking for existing minikube cluster: %w", err)`
`102`	`102`	`}`
`103`	`103`	`return nil`
`104`	`104`	`}`
`@@ -109,13 +109,13 @@ func checkMinikubeVersion() error {`
`109`	`109`	`versionCheck := exec.Command("minikube", "version", "--short")`
`110`	`110`	`out, err := versionCheck.CombinedOutput()`
`111`	`111`	`if err != nil {`
`112`		`- return fmt.Errorf("minikube version: %w", err)`
	`112`	`+ return fmt.Errorf("unable to check minikube version: %w", err)`
`113`	`113`	`}`
`114`	`114`	`fmt.Printf("Minikube version is: %s\n", string(out))`
`115`	`115`
`116`	`116`	`userMinikubeVersion, err := parseMinikubeVersion(string(out))`
`117`	`117`	`if err != nil {`
`118`		`- return fmt.Errorf("parsing minikube version: %w", err)`
	`118`	`+ return fmt.Errorf("unable to parse minikube version: %w", err)`
`119`	`119`	`}`
`120`	`120`	`if userMinikubeVersion < minikubeVersion {`
`121`	`121`	`var resp string`
`@@ -143,7 +143,7 @@ func checkForExistingCluster() error {`
`143`	`143`	`// if there were no profiles, we simply want to create a new one and not stop the install`
`144`	`144`	`// so if the error is the "MK_USAGE_NO_PROFILE" error, we ignore it and continue onwards`
`145`	`145`	`if !strings.Contains(string(out), "MK_USAGE_NO_PROFILE") {`
`146`		`- return fmt.Errorf("check cluster: %w", err)`
	`146`	`+ return fmt.Errorf("failed to get existing minikube profiles: %w", err)`
`147`	`147`	`}`
`148`	`148`	`}`
`149`	`149`	`// TODO Add tests for regex`
`@@ -160,7 +160,7 @@ func checkForExistingCluster() error {`
`160`	`160`	`namespaces := string(output)`
`161`	`161`	`if err != nil {`
`162`	`162`	`fmt.Println(string(output))`
`163`		`- return fmt.Errorf("check existing cluster: %w", err)`
	`163`	`+ return fmt.Errorf("unable to get existing kubernetes namespaces for minikube cluster %s: %w", clusterName, err)`
`164`	`164`	`}`
`165`	`165`	`if strings.Contains(namespaces, "knative") {`
`166`	`166`	`fmt.Print("Knative installation already exists.\nDelete and recreate the cluster [y/N]: ")`
`@@ -171,20 +171,20 @@ func checkForExistingCluster() error {`
`171`	`171`	`return nil`
`172`	`172`	`} else {`
`173`	`173`	`if err := recreateCluster(); err != nil {`
`174`		`- return fmt.Errorf("failed recreating cluster: %w", err)`
	`174`	`+ return fmt.Errorf("failed while recreating minikube cluster: %w", err)`
`175`	`175`	`}`
`176`	`176`	`}`
`177`	`177`	`}`
`178`	`178`	`return nil`
`179`	`179`	`}`
`180`	`180`	`if err := recreateCluster(); err != nil {`
`181`		`- return fmt.Errorf("failed recreating cluster: %w", err)`
	`181`	`+ return fmt.Errorf("failed while recreating minikube cluster: %w", err)`
`182`	`182`	`}`
`183`	`183`	`return nil`
`184`	`184`	`}`
`185`	`185`
`186`	`186`	`if err := createNewCluster(); err != nil {`
`187`		`- return fmt.Errorf("new cluster: %w", err)`
	`187`	`+ return fmt.Errorf("%w", err)`
`188`	`188`	`}`
`189`	`189`
`190`	`190`	`return nil`
`@@ -223,7 +223,7 @@ func createNewCluster() error {`
`223`	`223`	`"--insecure-registry", "10.0.0.0/24",`
`224`	`224`	`"--addons=registry")`
`225`	`225`	`if err := runCommandWithOutput(createCluster); err != nil {`
`226`		`- return fmt.Errorf("minikube create: %w", err)`
	`226`	`+ return fmt.Errorf("failed to create new minikube cluster %s: %w", clusterName, err)`
`227`	`227`	`}`
`228`	`228`
`229`	`229`	`return nil`
`@@ -264,10 +264,10 @@ func recreateCluster() error {`
`264`	`264`	`fmt.Println("deleting cluster...")`
`265`	`265`	`deleteCluster := exec.Command("minikube", "delete", "--profile", clusterName)`
`266`	`266`	`if err := deleteCluster.Run(); err != nil {`
`267`		`- return fmt.Errorf("delete cluster: %w", err)`
	`267`	`+ return fmt.Errorf("failed to delete minikube cluster %s: %w", clusterName, err)`
`268`	`268`	`}`
`269`	`269`	`if err := createNewCluster(); err != nil {`
`270`		`- return fmt.Errorf("new cluster: %w", err)`
	`270`	`+ return fmt.Errorf("%w", err)`
`271`	`271`	`}`
`272`	`272`	`return nil`
`273`	`273`	`}`