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

[PR #1064/d0c9f1a6 backport][stable-3] ec2_asg: Change purge_tags default value to False #1069

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Apr 14, 2022

This is a backport of PR #1064 as merged into main (d0c9f1a).

SUMMARY

Changed default value of purge_tags to False.

With the addition of purge_tags to ec2_asg module #960, the default value was kept to True similar to many other modules in this collection and also as specified in ansible dev guide.

With this addition, the issue was discovered that there is a possibility of this change breaking existing playbooks for users if they don't update their playbooks and specify purge_tags: false where required.

This PR's change will prevent accidental breakage.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_asg

ec2_asg: Change purge_tags default value to False

SUMMARY

Changed default value of purge_tags to False.

With the addition of purge_tags to ec2_asg module #960, the default value was kept to True similar to many other modules in this collection and also as specified in ansible dev guide.
With this addition, the issue was discovered that there is a possibility of this change breaking existing playbooks for users if they don't update their playbooks and specify purge_tags: false where required.
This PR's change will prevent accidental breakage.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ec2_asg

Reviewed-by: Mark Chappell <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Mark Woolley <mw@marknet15.com>
Reviewed-by: Alina Buzachis <None>
(cherry picked from commit d0c9f1a)
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 14, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 7b32ef0 into stable-3 Apr 14, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/d0c9f1a6ee6198358a4ab2e59266e17ac10bfe25/pr-1064 branch April 14, 2022 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants