Skip to content

Commit

Permalink
[integ-tests] Fix test_create_wrong_pcluster_version
Browse files Browse the repository at this point in the history
The failure was not signaled to CloudFormation via cfn-signal with version 3.6.1 because of this change aws@aeb87fc. In particular, for whatever version<3.10.0 the envar $CFN_BOOTSTRAP_VIRTUALENV_PATH is not set and so the cfn-signal command is not resolved.

Therefore, we change the version to 3.10.0 vs 3.11.0 to make ``$CFN_BOOTSTRAP_VIRTUALENV_PATH` setup consistent.

Signed-off-by: Hanwen <[email protected]>
  • Loading branch information
hanwen-cluster committed Jul 10, 2024
1 parent fcb8ed4 commit 4473460
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/integration-tests/tests/create/test_create.py
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def test_create_wrong_pcluster_version(
):
"""Test error message when AMI provided was baked by a pcluster whose version is different from current version"""
current_version = get_installed_parallelcluster_version()
wrong_version = "3.6.1"
wrong_version = "3.10.0"
logging.info("Asserting wrong_version is different from current_version")
assert_that(current_version != wrong_version).is_true()
# Retrieve an AMI without 'aws-parallelcluster-<version>' in its name.
Expand Down

0 comments on commit 4473460

Please sign in to comment.