add validation

aws · Nov 2, 2024 · 6896b65 · 6896b65
1 parent 7986dcd
commit 6896b65
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/packages/@aws-cdk/aws-redshift-alpha/lib/cluster.ts b/packages/@aws-cdk/aws-redshift-alpha/lib/cluster.ts
@@ -81,7 +81,7 @@ export enum ClusterType {
 }
 
 /**
- * The Amazon Redshift operation to be performed
+ * The Amazon Redshift operation
  */
 export enum ResourceAction {
   /**
@@ -613,6 +613,10 @@ export class Cluster extends ClusterBase {
       }
     }
 
+    if (props.resourceAction === ResourceAction.FAILOVER_PRIMARY_COMPUTE && !props.multiAz) {
+      throw new Error('ResourceAction.FAILOVER_PRIMARY_COMPUTE can only be used with multi-AZ clusters.');
+    }
+
     this.cluster = new CfnCluster(this, 'Resource', {
       // Basic
       allowVersionUpgrade: true,

diff --git a/packages/@aws-cdk/aws-redshift-alpha/test/cluster.test.ts b/packages/@aws-cdk/aws-redshift-alpha/test/cluster.test.ts
@@ -491,9 +491,23 @@ test.each([
     },
     vpc,
     resourceAction,
+    multiAz: true,
   });
 });
 
+test.each([false, undefined])('throw error for failover primary compute action with single AZ cluster', (multiAz) => {
+  expect(() => {
+    new Cluster(stack, 'Redshift', {
+      masterUser: {
+        masterUsername: 'admin',
+      },
+      vpc,
+      multiAz,
+      resourceAction: ResourceAction.FAILOVER_PRIMARY_COMPUTE,
+    });
+  }).toThrow('ResourceAction.FAILOVER_PRIMARY_COMPUTE can only be used with multi-AZ clusters.')
+});
+
 test('throws when trying to add rotation to a cluster without secret', () => {
   // WHEN
   const cluster = new Cluster(stack, 'Redshift', {