fix(msk-alpha): cluster deployment fails in ap-southeast-1 (#28112)

The reason is that the `AwsCustomResource` that this construct library uses calls `describeCluster`, which passes back all responses to CloudFormation, even if they are unused. This PR restricts the return value to only the outputs that are actually read. Fixes #28108 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Nov 27, 2023 · 0ee4199 · 0ee4199
1 parent 030782a
commit 0ee4199
Show file tree

Hide file tree

Showing 10 changed files with 25 additions and 18 deletions.
diff --git a/packages/@aws-cdk/aws-msk-alpha/lib/cluster.ts b/packages/@aws-cdk/aws-msk-alpha/lib/cluster.ts
@@ -721,6 +721,11 @@ export class Cluster extends ClusterBase {
           physicalResourceId: cr.PhysicalResourceId.of(
             'ZooKeeperConnectionString',
           ),
+          // Limit the output of describeCluster that is otherwise too large
+          outputPaths: [
+            'ClusterInfo.ZookeeperConnectString',
+            'ClusterInfo.ZookeeperConnectStringTls',
+          ],
         },
         policy: cr.AwsCustomResourcePolicy.fromSdkCalls({
           resources: [this.clusterArn],

diff --git a/...-zookeeper.js.snapshot/KafkaZookeeperIntegTestDefaultTestDeployAssert731E1402.assets.json b/...-zookeeper.js.snapshot/KafkaZookeeperIntegTestDefaultTestDeployAssert731E1402.assets.json
diff --git a/...ws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/KafkaZookeeperTestStack.assets.json b/...ws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/KafkaZookeeperTestStack.assets.json
diff --git a/...-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/KafkaZookeeperTestStack.template.json b/...-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/KafkaZookeeperTestStack.template.json
@@ -477,7 +477,7 @@
        {
         "Ref": "ClusterZookeeper29ECF2D7"
        },
-       "\"},\"physicalResourceId\":{\"id\":\"ZooKeeperConnectionString\"}}"
+       "\"},\"physicalResourceId\":{\"id\":\"ZooKeeperConnectionString\"},\"outputPaths\":[\"ClusterInfo.ZookeeperConnectString\",\"ClusterInfo.ZookeeperConnectStringTls\"]}"
       ]
      ]
     },
@@ -489,7 +489,7 @@
        {
         "Ref": "ClusterZookeeper29ECF2D7"
        },
-       "\"},\"physicalResourceId\":{\"id\":\"ZooKeeperConnectionString\"}}"
+       "\"},\"physicalResourceId\":{\"id\":\"ZooKeeperConnectionString\"},\"outputPaths\":[\"ClusterInfo.ZookeeperConnectString\",\"ClusterInfo.ZookeeperConnectStringTls\"]}"
       ]
      ]
     },
@@ -562,7 +562,7 @@
      "S3Bucket": {
       "Fn::Sub": "cdk-hnb659fds-assets-${AWS::AccountId}-${AWS::Region}"
      },
-     "S3Key": "e5178afc49b7c6a85127a67856ce958e4f0879ce6aad5e974cac2a088bf939db.zip"
+     "S3Key": "b0de6372e1c756b7afa1a8a1d0a21ee4cd330150290abffc9bc163f285150514.zip"
     },
     "Handler": "index.handler",
     "Role": {

diff --git a/....snapshot/asset.b0de6372e1c756b7afa1a8a1d0a21ee4cd330150290abffc9bc163f285150514/index.js b/....snapshot/asset.b0de6372e1c756b7afa1a8a1d0a21ee4cd330150290abffc9bc163f285150514/index.js
diff --git a/....snapshot/asset.e5178afc49b7c6a85127a67856ce958e4f0879ce6aad5e974cac2a088bf939db/index.js b/....snapshot/asset.e5178afc49b7c6a85127a67856ce958e4f0879ce6aad5e974cac2a088bf939db/index.js
diff --git a/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/cdk.out b/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/cdk.out
diff --git a/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/integ.json b/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/integ.json
diff --git a/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/manifest.json b/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/manifest.json
diff --git a/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/tree.json b/packages/@aws-cdk/aws-msk-alpha/test/integ.cluster-zookeeper.js.snapshot/tree.json