Skip to content

Comments

fix: Updating AZ regex condition to support EU Sovereign Cloud AZ IDs.#1279

Open
kgoutham93 wants to merge 1 commit intoterraform-aws-modules:masterfrom
kgoutham93:feature/eusc-support
Open

fix: Updating AZ regex condition to support EU Sovereign Cloud AZ IDs.#1279
kgoutham93 wants to merge 1 commit intoterraform-aws-modules:masterfrom
kgoutham93:feature/eusc-support

Conversation

@kgoutham93
Copy link

Description

AWS EU Sovereign cloud has different conventions for zone name and zone ids. Instead of asserting the regex on entire AZ id or AZ name, we resort to a simple logic -- split the az by hypen (-), if length(splits) == 2 we assume its AZ id.

{
    "AvailabilityZones": [
        {
            "OptInStatus": "opt-in-not-required",
            "Messages": [],
            "RegionName": "eusc-de-east-1",
            "ZoneName": "eusc-de-east-1a",
            "ZoneId": "euscdee1-az1",
            "GroupName": "eusc-de-east-1-zg-1",
            "NetworkBorderGroup": "eusc-de-east-1",
            "ZoneType": "availability-zone",
            "GroupLongName": "EU (Germany) 1",
            "State": "available"
        }
 ...
}

Motivation and Context

AWS EUSC has different naming conventions for AZ name and id.

Breaking Changes

None, I can think off.

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects

I executed examples/simple for us-east-1 and eusc-de-east-1 by passing a mix of AZ names and AZ ids to azs list.

  • I have executed pre-commit run -a on my pull request

… az has 2 parts after splitting with hyphen
@kgoutham93 kgoutham93 changed the title Updating AZ regex condition to support EU Sovereign Cloud AZ IDs. fix: Updating AZ regex condition to support EU Sovereign Cloud AZ IDs. Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant