Prefer exitWithError to os.Exit

cmd/kops/create_cluster.go

@@ -47,7 +47,7 @@ func init() {
 		Run: func(cmd *cobra.Command, args []string) {
 			err := createCluster.Run(args)
 			if err != nil {
-				glog.Exitf("%v", err)
+				exitWithError("%v", err)
 			}
 		},
 	}
@@ -209,8 +209,7 @@ func (c *CreateClusterCmd) Run(args []string) error {
 			}
 		} else {
 			// This is hard, because of the etcd cluster
-			glog.Errorf("Cannot change master-zones from the CLI")
+			return fmt.Errorf("Cannot change master-zones from the CLI")
-			os.Exit(1)
 		}
 	}
 
@@ -298,8 +297,7 @@ func (c *CreateClusterCmd) Run(args []string) error {
 	}
 
 	if cluster.SharedVPC() && cluster.Spec.NetworkCIDR == "" {
-		glog.Errorf("Must specify NetworkCIDR when VPC is set")
+		return fmt.Errorf("Must specify NetworkCIDR when VPC is set")
-		os.Exit(1)
 	}
 
 	if cluster.Spec.CloudProvider == "" {

cmd/kops/export_kubecfg.go

@@ -24,8 +24,7 @@ func init() {
 		Run: func(cmd *cobra.Command, args []string) {
 			err := exportKubecfgCommand.Run(args)
 			if err != nil {
-				fmt.Fprintf(os.Stderr, "%v\n", err)
+				exitWithError(err)
-				os.Exit(1)
 			}
 		},
 	}

cmd/kops/root.go

@@ -37,8 +37,7 @@ It allows you to create, destroy, upgrade and maintain clusters.`,
 func Execute() {
 	goflag.CommandLine.Parse([]string{})
 	if err := rootCommand.cobraCommand.Execute(); err != nil {
-		fmt.Println(err)
+		exitWithError(err)
-		os.Exit(-1)
 	}
 }