Skip to content

Commit

Permalink
[integ-test] Fix test_cli_commands in iso regions
Browse files Browse the repository at this point in the history
This commit fixes a bug from 7323fae

Signed-off-by: Hanwen <hanwenli@amazon.com>
  • Loading branch information
hanwen-cluster committed Jan 21, 2025
1 parent bd55b86 commit c6402a0
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ def test_slurm_cli_commands(
# Use long scale down idle time so we know nodes are terminated by pcluster stop
cluster_config = pcluster_config_reader(scaledown_idletime=60)

if "alinux" not in os and "us-iso" in region: # The code does not know non-amazon vanilla AMIs IDs in iso regions
if "alinux" in os or "us-iso" not in region: # The code does not know non-amazon vanilla AMIs IDs in iso regions
# Using custom AMI not tagged by pcluser will generate a warning
custom_ami = retrieve_latest_ami(region, os, ami_type="official", architecture="x86_64")
config_file = "pcluster.config.with.warnings.yaml"
Expand All @@ -59,7 +59,7 @@ def test_slurm_cli_commands(
_test_describe_cluster(cluster)
_test_list_cluster(cluster.name, "CREATE_COMPLETE")

if "alinux" not in os and "us-iso" in region:
if "alinux" in os or "us-iso" not in region:
_test_update_with_warnings(cluster_config_with_warning, cluster)
check_status(cluster, "CREATE_COMPLETE", "running", "RUNNING")

Expand Down

0 comments on commit c6402a0

Please sign in to comment.