diff --git a/CHANGELOG.md b/CHANGELOG.md index 924e2382c2..74e8d39c9d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,7 +19,6 @@ CHANGELOG **CHANGES** - Upgrade Cinc Client to version to 18.4.12 from 18.2.7. -- Allow build-image to be run in an isolated network. 3.9.3 ------ diff --git a/cli/src/pcluster/constants.py b/cli/src/pcluster/constants.py index a644ab7460..73ee1e90f3 100644 --- a/cli/src/pcluster/constants.py +++ b/cli/src/pcluster/constants.py @@ -294,7 +294,16 @@ class Operation(Enum): VERSION = "version" -UNSUPPORTED_OPERATIONS_MAP = {} +UNSUPPORTED_OPERATIONS_MAP = { + Operation.BUILD_IMAGE: ["us-iso"], + Operation.DELETE_IMAGE: ["us-iso"], + Operation.DESCRIBE_IMAGE: ["us-iso"], + Operation.LIST_IMAGES: ["us-iso"], + Operation.EXPORT_IMAGE_LOGS: ["us-iso"], + Operation.GET_IMAGE_LOG_EVENTS: ["us-iso"], + Operation.GET_IMAGE_STACK_EVENTS: ["us-iso"], + Operation.LIST_IMAGE_LOG_STREAMS: ["us-iso"], +} MAX_TAGS_COUNT = 40 # Tags are limited to 50, reserve some tags for parallelcluster specified tags diff --git a/cli/src/pcluster/resources/imagebuilder/parallelcluster.yaml b/cli/src/pcluster/resources/imagebuilder/parallelcluster.yaml index caf1262588..ebccd777d3 100644 --- a/cli/src/pcluster/resources/imagebuilder/parallelcluster.yaml +++ b/cli/src/pcluster/resources/imagebuilder/parallelcluster.yaml @@ -53,7 +53,7 @@ phases: commands: - | set -v - CINC_URL="s3://${AWS::Region}-aws-parallelcluster/archives/cinc/cinc-install-1.3.0.sh" + CINC_URL="https://${AWS::Region}-aws-parallelcluster.s3.${AWS::Region}.${AWS::URLSuffix}/archives/cinc/cinc-install-1.3.0.sh" [ -n "${CfnParamCincInstaller}" ] && CINC_URL="${CfnParamCincInstaller}" echo "${!CINC_URL}" @@ -219,19 +219,13 @@ phases: apt-get -y --only-upgrade install ca-certificates fi - aws s3 --region ${AWS::Region} cp {{ build.CincUrl.outputs.stdout }} - | bash -s -- -v {{ ChefVersion }} + curl --retry 3 -L {{ build.CincUrl.outputs.stdout }} | bash -s -- -v {{ ChefVersion }} if [[ -e ${!CA_CERTS_FILE} ]]; then - mkdir -p /opt/cinc/embedded/ssl/certs ln -sf ${!CA_CERTS_FILE} /opt/cinc/embedded/ssl/certs/cacert.pem fi - - aws s3 cp s3://${AWS::Region}-aws-parallelcluster/archives/dependencies/ruby/gems.tgz gems.tgz --region ${AWS::Region} - tar -xf gems.tgz - - cd vendor/cache - /opt/cinc/embedded/bin/gem install --local --no-document berkshelf:{{ BerkshelfVersion }} + /opt/cinc/embedded/bin/gem install --no-document berkshelf:{{ BerkshelfVersion }} # Download and vendor Cookbook - name: DownloadCookbook @@ -241,14 +235,9 @@ phases: - | set -v mkdir -p /etc/chef && sudo chown -R root:root /etc/chef - - if [[ "{{ build.CookbookUrl.outputs.stdout }}" =~ "s3://" ]]; then - aws s3 cp "{{ build.CookbookUrl.outputs.stdout }}" /etc/chef/aws-parallelcluster-cookbook.tgz --region ${AWS::Region} - else - curl --retry 3 -L -o /etc/chef/aws-parallelcluster-cookbook.tgz "{{ build.CookbookUrl.outputs.stdout }}" - fi + curl --retry 3 -L -o /etc/chef/aws-parallelcluster-cookbook.tgz "{{ build.CookbookUrl.outputs.stdout }}" - mkdir -p /tmp/cookbooks + mkdir /tmp/cookbooks cd /tmp/cookbooks tar -xzf /etc/chef/aws-parallelcluster-cookbook.tgz diff --git a/tests/integration-tests/configs/build_image_iso.yaml b/tests/integration-tests/configs/build_image_iso.yaml deleted file mode 100644 index 25aecc8396..0000000000 --- a/tests/integration-tests/configs/build_image_iso.yaml +++ /dev/null @@ -1,22 +0,0 @@ -{%- import 'common.jinja2' as common with context -%} -{% if REGIONS %} -{%- set REGIONS = [ REGIONS ] -%} -{% else %} -{%- set REGIONS = ["us-isob-east-1","us-iso-east-1"] -%} -{% endif %} -{%- set INSTANCES = ["c5.xlarge"] -%} -{% if OSS %} -{%- set OSS = [ OSS ] -%} -{% else %} -{%- set OSS = ["alinux2"] -%} -{% endif %} -{%- set SCHEDULERS = ["slurm"] -%} ---- -test-suites: - createami: - test_createami.py::test_build_image: - dimensions: - - regions: {{ REGIONS }} - instances: {{ INSTANCES }} - schedulers: {{ SCHEDULERS }} - oss: {{ OSS }} \ No newline at end of file