fix: Fix error checking ordering for DescribeInstances (#7381)

aws · Nov 14, 2024 · 992689c · 992689c
1 parent 569c06d
commit 992689c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/providers/instance/instance.go b/pkg/providers/instance/instance.go
@@ -157,7 +157,7 @@ func (p *DefaultProvider) List(ctx context.Context) ([]*Instance, error) {
 				Values: []string{v1.NodeClassTagKey},
 			},
 			{
-				Name:   aws.String(v1.EKSClusterNameTagKey),
+				Name:   aws.String(fmt.Sprintf("tag:%s", v1.EKSClusterNameTagKey)),
 				Values: []string{options.FromContext(ctx).ClusterName},
 			},
 			instanceStateFilter,
@@ -166,10 +166,10 @@ func (p *DefaultProvider) List(ctx context.Context) ([]*Instance, error) {
 
 	for paginator.HasMorePages() {
 		page, err := paginator.NextPage(ctx)
-		out.Reservations = append(out.Reservations, page.Reservations...)
 		if err != nil {
 			return nil, fmt.Errorf("describing ec2 instances, %w", err)
 		}
+		out.Reservations = append(out.Reservations, page.Reservations...)
 	}
 	instances, err := instancesFromOutput(out)
 	return instances, cloudprovider.IgnoreNodeClaimNotFoundError(err)