better error message

dkoshkin · dlipovetsky · web-flow · commit c88c19e375dd · 2022-10-18T16:28:30.000-07:00
Co-authored-by: Daniel Lipovetsky &lt;3445370+dlipovetsky@users.noreply.github.com&gt;
diff --git a/pkg/cloud/services/eks/cluster.go b/pkg/cloud/services/eks/cluster.go
@@ -69,7 +69,7 @@ func (s *Service) reconcileCluster(ctx context.Context) error {
 		oldOwnedTag := cluster.Tags[oldTagKey]
 
 		if ownedTag == nil && oldOwnedTag == nil {
-			return fmt.Errorf("cluster is not tagged with neither %s nor %s", tagKey, oldTagKey)
+			return fmt.Errorf("EKS cluster resource %q must have a tag with key %q or %q", eksClusterName, oldTagKey, tagKey)
 		}
 
 		s.scope.V(2).Info("Found owned EKS cluster in AWS", "cluster", klog.KRef("", eksClusterName))

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ func (s *Service) reconcileCluster(ctx context.Context) error {`
`69`	`69`	`oldOwnedTag := cluster.Tags[oldTagKey]`
`70`	`70`
`71`	`71`	`if ownedTag == nil && oldOwnedTag == nil {`
`72`		`- return fmt.Errorf("cluster is not tagged with neither %s nor %s", tagKey, oldTagKey)`
	`72`	`+ return fmt.Errorf("EKS cluster resource %q must have a tag with key %q or %q", eksClusterName, oldTagKey, tagKey)`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`s.scope.V(2).Info("Found owned EKS cluster in AWS", "cluster", klog.KRef("", eksClusterName))`