Skip to content

Commit

Permalink
[integ-test] fix usage of inject_additional_config_settings
Browse files Browse the repository at this point in the history
This commit fixes bug from aws@6fc6a02

Signed-off-by: Hanwen <[email protected]>
  • Loading branch information
hanwen-cluster committed Oct 24, 2024
1 parent b8a2956 commit 9df7b52
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -53,15 +53,15 @@


def test_pcluster_configure(
request, vpc_stack, key_name, region, os, instance, scheduler, clusters_factory, test_datadir
request, vpc_stack, key_name, region, os, instance, scheduler, clusters_factory, test_datadir, architecture
):
"""Verify that the config file produced by `pcluster configure` can be used to create a cluster."""
skip_if_unsupported_test_options_were_used(request)
config_path = test_datadir / "config.yaml"

_create_and_test_standard_configuration(request, config_path, region, key_name, scheduler, os, instance, vpc_stack)

inject_additional_config_settings(config_path, request, region)
inject_additional_config_settings(config_path, request, region, architecture)
clusters_factory(config_path)


Expand Down Expand Up @@ -142,6 +142,7 @@ def test_efa_and_placement_group(
region,
os,
instance,
architecture,
scheduler,
clusters_factory,
test_datadir,
Expand Down Expand Up @@ -187,7 +188,7 @@ def test_efa_and_placement_group(
efa_config=efa_config,
placement_group_config=placement_group_config["configuration"],
)
inject_additional_config_settings(config_path, request, region)
inject_additional_config_settings(config_path, request, region, architecture)
clusters_factory(config_path)


Expand Down

0 comments on commit 9df7b52

Please sign in to comment.