fix(eks): clusters in a FAILED state are not detected (#9553)

Fixes #9529

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

Author: iliapolo

packages/@aws-cdk/aws-eks/lib/cluster-resource-handler/cluster.ts

@@ -196,7 +196,11 @@ export class ClusterResourceHandler extends ResourceHandler {
     // if cluster is undefined (shouldnt happen) or status is not ACTIVE, we are
     // not complete. note that the custom resource provider framework forbids
     // returning attributes (Data) if isComplete is false.
-    if (cluster?.status !== 'ACTIVE') {
+    if (cluster?.status === 'FAILED') {
+      // not very informative, unfortunately the response doesn't contain any error
+      // information :\
+      throw new Error('Cluster is in a FAILED status');
+    } else if (cluster?.status !== 'ACTIVE') {
       return {
         IsComplete: false,
       };

packages/@aws-cdk/aws-eks/test/test.cluster-resource-provider.ts

@@ -88,6 +88,30 @@ export = {
       test.done();
     },
 
+    async 'isCreateComplete throws if cluster is FAILED'(test: Test) {
+      const handler = new ClusterResourceHandler(mocks.client, mocks.newRequest('Create'));
+      mocks.simulateResponse.describeClusterResponseMockStatus = 'FAILED';
+      try {
+        await handler.isComplete();
+        test.ok(false, 'expected error to be thrown');
+      } catch (err) {
+        test.equal(err.message, 'Cluster is in a FAILED status');
+      }
+      test.done();
+    },
+
+    async 'isUpdateComplete throws if cluster is FAILED'(test: Test) {
+      const handler = new ClusterResourceHandler(mocks.client, mocks.newRequest('Update'));
+      mocks.simulateResponse.describeClusterResponseMockStatus = 'FAILED';
+      try {
+        await handler.isComplete();
+        test.ok(false, 'expected error to be thrown');
+      } catch (err) {
+        test.equal(err.message, 'Cluster is in a FAILED status');
+      }
+      test.done();
+    },
+
     async 'isCreateComplete is complete when cluster is ACTIVE'(test: Test) {
       const handler = new ClusterResourceHandler(mocks.client, mocks.newRequest('Create'));
       mocks.simulateResponse.describeClusterResponseMockStatus = 'ACTIVE';