Skip to content

Commit c303657

Browse files
committed
[Subnet Prioritization] Set SingleAvailabilityZone as null if EnableSingleAvailabilityZone is not provided
Signed-off-by: Hanxuan Zhang <[email protected]>
1 parent 26eec74 commit c303657

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

cookbooks/aws-parallelcluster-slurm/files/default/head_node_slurm/slurm/pcluster_fleet_config_generator.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,9 @@ def generate_fleet_config_file(output_file: str, input_file: str):
9999
queue_capacity_reservation=queue_capacity_reservation,
100100
queue_capacity_type=queue_capacity_type,
101101
queue_subnets=queue_config["Networking"]["SubnetIds"],
102-
queue_single_availability_zone=queue_config["Networking"]["EnableSingleAvailabilityZone"],
102+
queue_single_availability_zone=queue_config.get("Networking", {}).get(
103+
"EnableSingleAvailabilityZone", None
104+
),
103105
)
104106
fleet_config[queue_name][compute_resource_name] = config_for_fleet
105107

test/unit/slurm/test_fleet_config_generator.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
}
6161
},
6262
CriticalError,
63-
"Unable to find key 'EnableSingleAvailabilityZone' in the configuration file. Queue: q1",
63+
"Unable to find key 'Name' in the configuration file. Queue: q1",
6464
),
6565
(
6666
{
@@ -239,8 +239,8 @@
239239
]
240240
}
241241
},
242-
CriticalError,
243-
"Unable to find key 'EnableSingleAvailabilityZone' in the configuration file. Queue: q1",
242+
None,
243+
None,
244244
),
245245
(
246246
{

test/unit/slurm/test_fleet_config_generator/test_generate_fleet_config_file/sample_input.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,6 @@ Scheduling:
253253
SecurityGroups: null
254254
SubnetIds:
255255
- subnet-0230367ab0e5123a4
256-
EnableSingleAvailabilityZone: null
257256
SlurmSettings:
258257
Dns:
259258
DisableManagedDns: false

0 commit comments

Comments
 (0)