Skip to content

Conversation

@demartinofra
Copy link
Contributor

Description of changes

Official Centos 7 AMIs listed here https://wiki.centos.org/Cloud/AWS have reached the 2 year EC2 default deprecation period. Allowing usage of deprecated AMIs for the build-image test while evaluating usage of marketplace AMIs

Tests

  • Locally executed the test with following command:
python -m test_runner --reports html junitxml json --key-name "fdm-jenkins" --key-path /Users/fdm/.ssh/fdm-jenkins.pem --tests-config configs/fdm.yaml --sequential -n 1 --no-delete --vpc-stack integ-tests-vpc-3dv0vxztw89rkh32 --use-default-iam-credentials

test-suites:
  createami:
    test_createami.py::test_build_image:
      dimensions:
      - instances:
        - c5.xlarge
        oss:
        - centos7
        regions:
        - eu-west-3

References

Checklist

  • Make sure you are pointing to the right branch and add a label in the PR title (i.e. 2.x vs 3.x)
  • Check all commits' messages are clear, describing what and why vs how.
  • Make sure to have added unit tests or integration tests to cover the new/modified code.
  • Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Official Centos 7 AMIs listed here https://wiki.centos.org/Cloud/AWS
have reached the 2 year EC2 default deprecation period.
Allowing usage of deprecated AMIs for the build-image test while evaluating
usage of marketplace AMIs

Signed-off-by: Francesco De Martino <fdm@amazon.com>
@demartinofra demartinofra requested review from a team as code owners November 14, 2022 12:14
@codecov
Copy link

codecov bot commented Nov 14, 2022

Codecov Report

Merging #4536 (323230b) into integ-tests-3.3.0 (0878600) will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           integ-tests-3.3.0    #4536   +/-   ##
==================================================
  Coverage              88.79%   88.79%           
==================================================
  Files                    159      159           
  Lines                  13741    13741           
==================================================
  Hits                   12202    12202           
  Misses                  1539     1539           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@eantonin eantonin added the skip-changelog-update Disables the check that enforces changelog updates in PRs label Nov 14, 2022
@demartinofra demartinofra merged commit 1c57504 into aws:integ-tests-3.3.0 Nov 14, 2022
@demartinofra demartinofra deleted the integ-tests-build-image-fix branch November 14, 2022 13:26
chenwany added a commit to chenwany/aws-parallelcluster that referenced this pull request Nov 14, 2022
Backport aws#4536

Signed-off-by: chenwany <chenwany@amazon.com>
chenwany added a commit that referenced this pull request Nov 14, 2022
Backport #4536

Signed-off-by: chenwany <chenwany@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog-update Disables the check that enforces changelog updates in PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants