Skip to content

[Test] Networking setup: honor the AZ allowlisting everywhere in the networking setup and exclude eu-west-2d. - #7603

Merged
gmarciani merged 1 commit into
aws:developfrom
gmarciani:wip/mgiacomo/3170/test-az-allow-0903-1
Sep 3, 2026
Merged

[Test] Networking setup: honor the AZ allowlisting everywhere in the networking setup and exclude eu-west-2d.#7603
gmarciani merged 1 commit into
aws:developfrom
gmarciani:wip/mgiacomo/3170/test-az-allow-0903-1

Conversation

@gmarciani

@gmarciani gmarciani commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description of changes

Honor the AZ allowlisting everywhere in the networking setup and exclude eu-west-2d.

Before this change, only the default AZ was honoring the allowlisting, so other components such as the ODCR stack had chances to select unexpected AZs.

With this change, by default all the region's AZs are usable, but if the region has an explicit allowlisting, only those allowlisted AZs will be considered.

As part of this change we set the allowlisting for eu-west-2, where we exclude eu-west-2d which is a new AZ that does not support our default instance type c5.xlarge.

Tests

SUCCESS

test-suites:
  update:
    test_update.py::test_multi_az_create_and_update:
      dimensions:
        - regions: ["eu-west-2"]
          schedulers: ["slurm"]
          oss: [{{ OS_X86_1 }}]
          instances: {{ common.INSTANCES_DEFAULT_X86 }}
  storage:
    test_fsx_lustre.py::test_multi_az_fsx:
      dimensions:
        - regions: ["eu-west-2"]
          instances: {{ common.INSTANCES_DEFAULT_X86 }}
          oss: [{{ LUSTRE_OS_X86_1 }}]
          schedulers: ["slurm"]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

…networking setup.

Before this change, only the default AZ was honoring the allowlisting, so other components such as the ODCR stack had chances to select unexpected AZs.

With this change, by default all the region's AZs are usable, but if the region has an explicit allowlisting, only those allowlisted AZs will be considered.

As part of this change we set the allowlisting for eu-west-2, where we exclude eu-west-2d which is a new AZ that does not support our default instance type c5.xlarge.
@gmarciani
gmarciani requested review from a team as code owners September 3, 2026 19:35
@gmarciani gmarciani added skip-changelog-update Disables the check that enforces changelog updates in PRs 3.x Test labels Sep 3, 2026
@gmarciani
gmarciani merged commit 014e913 into aws:develop Sep 3, 2026
28 of 29 checks passed
@gmarciani
gmarciani deleted the wip/mgiacomo/3170/test-az-allow-0903-1 branch September 3, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x skip-changelog-update Disables the check that enforces changelog updates in PRs Test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants