Skip to content

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
Copy Markdown

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
@kgoutham93
Copy link
Copy Markdown
Author

Hey @antonbabenko 👋 could you have a look please.

@github-actions
Copy link
Copy Markdown

This PR has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this PR will be closed in 10 days

@github-actions github-actions bot added the stale label Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant