Merge pull request kubernetes#138 from justinsb/fix_136

Don't export kubecfg if we're only doing a --dryrun
klstr · Jul 13, 2016 · 25d1e97 · 25d1e97
2 parents 6c8d5d8 + 4b3b0ab
commit 25d1e97
Showing 1 changed file with 12 additions and 10 deletions.
diff --git a/cmd/kops/create_cluster.go b/cmd/kops/create_cluster.go
@@ -368,18 +368,20 @@ func (c *CreateClusterCmd) Run() error {
 		return err
 	}
 
-	glog.Infof("Exporting kubecfg for cluster")
+	if !isDryrun {
+		glog.Infof("Exporting kubecfg for cluster")
 
-	x := &kutil.CreateKubecfg{
-		ClusterName:      cluster.Name,
-		KeyStore:         clusterRegistry.KeyStore(cluster.Name),
-		MasterPublicName: cluster.Spec.MasterPublicName,
-	}
-	defer x.Close()
+		x := &kutil.CreateKubecfg{
+			ClusterName:      cluster.Name,
+			KeyStore:         clusterRegistry.KeyStore(cluster.Name),
+			MasterPublicName: cluster.Spec.MasterPublicName,
+		}
+		defer x.Close()
 
-	err = x.WriteKubecfg()
-	if err != nil {
-		return err
+		err = x.WriteKubecfg()
+		if err != nil {
+			return err
+		}
 	}
 
 	return nil