Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add source_version param to ec2_launch_template module #239

Conversation

luiseterc
Copy link
Contributor

SUMMARY

Add support for Boto3.create_launch_template_version's source_version parameter.

Accepted values:

  • int (specific version) : Creates a new launch template using this version as the base, hence keeping all its parameters
  • string ( 'latest') : Uses the latest found in list template_versions
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ec2_launch_template.py

ADDITIONAL INFORMATION

Sanity tests passed for this module.

@luiseterc luiseterc force-pushed the feature/ec2_launch_template_source_version branch from cd0623d to 027fb6c Compare September 28, 2020 06:52
@ansibullbot
Copy link

@ansibullbot ansibullbot added affects_2.10 community_review feature This issue/PR relates to a feature request module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) stale_ci CI is older than 7 days, rerun before merging labels Nov 16, 2020
@goneri
Copy link
Member

goneri commented Feb 19, 2021

Hi @luiseterc,

Thank you for the PR! Can you please add a changelog (https://docs.ansible.com/ansible/latest/community/development_process.html#changelogs) entry and extend the existing functional tests.

@tremble tremble force-pushed the feature/ec2_launch_template_source_version branch from 027fb6c to 2ea4c14 Compare July 1, 2022 18:33
@ansibullbot
Copy link

@ansibullbot ansibullbot added integration tests/integration tests tests labels Jul 1, 2022
@github-actions
Copy link

github-actions bot commented Jul 1, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@tremble tremble requested a review from markuman July 1, 2022 18:46
@tremble tremble removed the stale_ci CI is older than 7 days, rerun before merging label Jul 1, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 37s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 20s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 26s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 57s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 57s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 10s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 57s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 53s
✔️ ansible-test-splitter SUCCESS in 2m 40s
✔️ integration-community.aws-1 SUCCESS in 10m 13s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@markuman markuman added the backport-4 PR should be backported to the stable-4 branch label Jul 1, 2022
Copy link
Contributor

@jatorcasso jatorcasso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jul 1, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 06s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 12s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 25s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 37s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 22s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 31s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 19s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 56s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 6m 56s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 279bbc9 into ansible-collections:main Jul 1, 2022
@patchback
Copy link

patchback bot commented Jul 1, 2022

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/279bbc980befe043745e47480857358e307a2215/pr-239

Backported as #1295

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Jul 1, 2022
Add source_version param to ec2_launch_template module

SUMMARY
Add support for Boto3.create_launch_template_version's source_version parameter.
Accepted values:

int (specific version) : Creates a new launch template using this version as the base, hence keeping all its parameters
string ( 'latest') : Uses the latest found in list template_versions

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_launch_template.py
ADDITIONAL INFORMATION
Sanity tests passed for this module.

Reviewed-by: Mark Chappell <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit 279bbc9)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jul 2, 2022
[PR #239/279bbc98 backport][stable-4] Add source_version param to ec2_launch_template module

This is a backport of PR #239 as merged into main (279bbc9).
SUMMARY
Add support for Boto3.create_launch_template_version's source_version parameter.
Accepted values:

int (specific version) : Creates a new launch template using this version as the base, hence keeping all its parameters
string ( 'latest') : Uses the latest found in list template_versions

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_launch_template.py
ADDITIONAL INFORMATION
Sanity tests passed for this module.

Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects_2.10 backport-4 PR should be backported to the stable-4 branch community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants