❗ NOTICE: Firehose CIDR block support issue for few regions(eu-central-2, eu-south-2, ap-south-2)

[jstoneham]

Please add your +1 👍 to let us know you have encountered this

Status: IN-PROGRESS

Overview:

Commit 1604cf4b88af1239253a1ecb1c411ca66f8f57b2 (1604cf) removed the Firehose CIDR blocks in packages/aws-cdk-lib/region-info/build-tools/fact-tables.ts for eu-central-2, eu-south-2, and ap-south-2, so these constructs cannot be synthesized in those regions in 2.95+.

Expected Behavior

Constructs synth in these regions

Current Behavior

They don't!

Reproduction Steps

Synth your normal constructs in one of these regions.

Workaround:

Downgrade to 2.94.0 of aws-cdk-lib if encountering this error. Also with this version we added il-central-1 and ap-southeast-4 to CDK Metadata which we don't provide analytical support to and got a patch release out to handle them. The probable solution for that version would be to clean/comment out those two regions from within CDK Metadata and should work.

Solution:

Adding the Kinesis firehose CIDR block for those regions to provide support.
#27672

Related issue:

#27518

Additional Information/Context

Error: Mapping doesn't contain top-level key 'eu-south-2'
    at CfnMapping.findInMap (/workspace/jgstone/UpgradeCDK/src/MyCDK/node_modules/aws-cdk-lib/core/lib/cfn-mapping.js:1:1419)
    at setConnections (/workspace/jgstone/UpgradeCDK/src/MyCDK/node_modules/@aws-cdk/aws-kinesisfirehose-alpha/lib/delivery-stream.ts:407:32)
    at new DeliveryStreamBase (/workspace/jgstone/UpgradeCDK/src/MyCDK/node_modules/@aws-cdk/aws-kinesisfirehose-alpha/lib/delivery-stream.ts:108:24)
    at new DeliveryStream (/workspace/jgstone/UpgradeCDK/src/MyCDK/node_modules/@aws-cdk/aws-kinesisfirehose-alpha/lib/delivery-stream.ts:317:5)
...

CDK CLI Version

2.99.1 (build b2a895e)

Framework Version

No response

Node.js Version

18

OS

MacOS

Language

TypeScript

Language Version

No response

Other information

No response

[khushail]

Hi @jstoneham , thanks for reporting this. I will consult with team on why this change was implemented and get back with updates.

[khushail]

@jstoneham , Team is working on this issue and a fix would be provided soon

[jstoneham]

Thank you for your prompt attention!

[SankyRed]

@jstoneham Working on getting the changes added in for eu-central-2 and ap-south-2. Also wanted to call out that at the moment we do not support eu-south-2(Europe (Spain)). Did check the documentation for the Kinesis Data Firehose CIDR block and seems like we do not support it. Thank you for calling the issue out.

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.