Merge pull request #41 from weaveworks/bug/40-empty-output

Adding the proper field value for cluster status
eksctl-io · Jun 6, 2018 · 835d2c2 · 835d2c2
2 parents 3c8826e + c708f82
commit 835d2c2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.gitignore b/.gitignore
@@ -10,3 +10,6 @@
 # Release config file backups
 .goreleaser.floating.yml.bak
 .goreleaser.permalink.yml.bak
+
+# Locally built binary
+eksctl
diff --git a/pkg/eks/eks.go b/pkg/eks/eks.go
@@ -145,6 +145,7 @@ func (c *ClusterProvider) ListClusters() error {
 	if err != nil {
 		return errors.Wrap(err, "listing control planes")
 	}
+	logger.Debug("clusters = %#v", output)
 	for _, clusterName := range output.Clusters {
 		if err := c.doListCluster(clusterName); err != nil {
 			return err
@@ -161,7 +162,8 @@ func (c *ClusterProvider) doListCluster(clusterName *string) error {
 	if err != nil {
 		return errors.Wrapf(err, "unable to describe control plane %q", *clusterName)
 	}
-	if *output.Cluster.Status == "Ready" {
+	logger.Debug("cluster = %#v", output)
+	if *output.Cluster.Status == eks.ClusterStatusActive {
 		logger.Info("cluster = %#v", *output.Cluster)
 		stacks, err := c.ListReadyStacks(fmt.Sprintf("^EKS-%s-.*$", *clusterName))
 		if err != nil {