error exit if no clusters found

cli/cluster.go

@@ -199,14 +199,13 @@ func getKubeConfig(cluster string) (string, error) {
 }
 
 // printClusters prints the names of existing clusters
-func printClusters() {
+func printClusters() error {
 	clusters, err := getClusters(true, "")
 	if err != nil {
 		log.Fatalf("Couldn't list clusters\n%+v", err)
 	}
 	if len(clusters) == 0 {
-		log.Printf("No clusters found!")
-		return
+		return fmt.Errorf("No clusters found")
 	}
 
 	table := tablewriter.NewWriter(os.Stdout)
@@ -226,6 +225,7 @@ func printClusters() {
 	}
 
 	table.Render()
+	return nil
 }
 
 // Classify cluster state: Running, Stopped or Abnormal

cli/commands.go

@@ -246,6 +246,10 @@ func DeleteCluster(c *cli.Context) error {
 		return err
 	}
 
+	if len(clusters) == 0 {
+		return fmt.Errorf("No cluster(s) found")
+	}
+
 	// remove clusters one by one instead of appending all names to the docker command
 	// this allows for more granular error handling and logging
 	for _, cluster := range clusters {
@@ -365,7 +369,9 @@ func ListClusters(c *cli.Context) error {
 		log.Info("--all is on by default, thus no longer required. This option will be removed in v2.0.0")
 
 	}
-	printClusters()
+	if err := printClusters(); err != nil {
+		return err
+	}
 	return nil
 }
 
@@ -376,6 +382,10 @@ func GetKubeConfig(c *cli.Context) error {
 		return err
 	}
 
+	if len(clusters) == 0 {
+		return fmt.Errorf("No cluster(s) found")
+	}
+
 	for _, cluster := range clusters {
 		kubeConfigPath, err := getKubeConfig(cluster.name)
 		if err != nil {