From 61d98edaece697d6f0713ccf40e33e830b5028d8 Mon Sep 17 00:00:00 2001 From: Xuanqi He Date: Tue, 23 Jul 2024 15:35:22 -0400 Subject: [PATCH] Add support for CGK region. --- CHANGELOG.md | 1 + cli/src/pcluster/resources/supported-regions | 3 ++- tests/integration-tests/configs/common.jinja2 | 2 +- tests/integration-tests/configs/develop.yaml | 8 ++++---- 4 files changed, 8 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index f866b8130f..6de944b0ff 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,7 @@ CHANGELOG - Add support for custom actions on login nodes. - Allow DCV connection on login nodes. +- Add support for ap-southeast-3 region. **BUG FIXES** - Fix validator `EfaPlacementGroupValidator` so that it does not suggest to configure a Placement Group when Capacity Blocks are used. diff --git a/cli/src/pcluster/resources/supported-regions b/cli/src/pcluster/resources/supported-regions index 68ab8abe50..08cbec0390 100644 --- a/cli/src/pcluster/resources/supported-regions +++ b/cli/src/pcluster/resources/supported-regions @@ -5,6 +5,7 @@ ap-northeast-2 ap-south-1 ap-southeast-1 ap-southeast-2 +ap-southeast-3 ca-central-1 cn-north-1 cn-northwest-1 @@ -24,4 +25,4 @@ us-isob-east-1 us-gov-east-1 us-gov-west-1 us-west-1 -us-west-2 \ No newline at end of file +us-west-2 diff --git a/tests/integration-tests/configs/common.jinja2 b/tests/integration-tests/configs/common.jinja2 index e8ff6aa265..2d73390d1f 100644 --- a/tests/integration-tests/configs/common.jinja2 +++ b/tests/integration-tests/configs/common.jinja2 @@ -1,4 +1,4 @@ -{%- set REGIONS_COMMERCIAL = ["us-east-1", "us-east-2", "us-west-1", "us-west-2", "ca-central-1", "eu-central-1", "eu-west-1", "eu-west-2", "eu-west-3", "sa-east-1", "ap-east-1", "ap-northeast-1", "ap-northeast-2", "ap-south-1", "ap-southeast-1", "ap-southeast-2", "eu-north-1", "me-south-1", "af-south-1", "eu-south-1", "il-central-1"] -%} +{%- set REGIONS_COMMERCIAL = ["us-east-1", "us-east-2", "us-west-1", "us-west-2", "ca-central-1", "eu-central-1", "eu-west-1", "eu-west-2", "eu-west-3", "sa-east-1", "ap-east-1", "ap-northeast-1", "ap-northeast-2", "ap-south-1", "ap-southeast-1", "ap-southeast-2", "ap-southeast-3", "eu-north-1", "me-south-1", "af-south-1", "eu-south-1", "il-central-1"] -%} {%- set REGIONS_CHINA = ["cn-north-1", "cn-northwest-1"] -%} {%- set REGIONS_GOVCLOUD = ["us-gov-west-1", "us-gov-east-1"] -%} {%- set REGIONS_ALL = REGIONS_COMMERCIAL + REGIONS_CHINA + REGIONS_GOVCLOUD -%} diff --git a/tests/integration-tests/configs/develop.yaml b/tests/integration-tests/configs/develop.yaml index 3ac313c27a..81fdbad379 100644 --- a/tests/integration-tests/configs/develop.yaml +++ b/tests/integration-tests/configs/develop.yaml @@ -388,7 +388,7 @@ test-suites: schedulers: ["slurm"] test_security_groups.py::test_overwrite_sg: dimensions: - - regions: ["eu-north-1"] + - regions: ["ap-southeast-3"] instances: {{ common.INSTANCES_DEFAULT_X86 }} oss: ["rocky9"] schedulers: ["slurm"] @@ -401,7 +401,7 @@ test-suites: scaling: test_scaling.py::test_job_level_scaling: dimensions: - - regions: ["us-west-2"] + - regions: ["ap-southeast-3"] instances: {{ common.INSTANCES_DEFAULT_ARM }} oss: ["ubuntu2204"] schedulers: ["slurm"] @@ -446,7 +446,7 @@ test-suites: schedulers: [ "slurm" ] test_slurm.py::test_error_handling: dimensions: - - regions: ["ca-central-1"] + - regions: ["ap-southeast-3"] instances: {{ common.INSTANCES_DEFAULT_X86 }} oss: ["rocky9"] schedulers: ["slurm"] @@ -512,7 +512,7 @@ test-suites: schedulers: ["slurm"] test_slurm.py::test_slurm_custom_config_parameters: dimensions: - - regions: ["ap-northeast-2"] + - regions: ["ap-southeast-3"] instances: {{ common.INSTANCES_DEFAULT_X86 }} oss: ["ubuntu2004"] schedulers: ["slurm"]