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

route53 module_utils not ported to amazon.aws #1064

Closed
1 task done
tremble opened this issue Sep 23, 2022 · 3 comments
Closed
1 task done

route53 module_utils not ported to amazon.aws #1064

tremble opened this issue Sep 23, 2022 · 3 comments
Labels
bug This issue/PR relates to a bug module module plugins plugin (any type) python3

Comments

@tremble
Copy link
Contributor

tremble commented Sep 23, 2022

Summary

https://b7d126e9ce1bb4ebcac2-5137bb838e27bff384ce99c5e92105c7.ssl.cf1.rackcdn.com/1063/8671cbbc48875c782c9090a1de56ba527fe9248e/check/ansible-test-sanity-aws-ansible-2.12-python38/2184f29/job-output.txt

2022-09-23 14:23:11.496260 | controller | ERROR: plugins/modules/route53_health_check.py:0:0: import-error: Exception attempting to import module for argument_spec introspection, 'No module named 'ansible_collections.community.aws.plugins.module_utils.route53''
2022-09-23 14:23:11.496280 | controller | ERROR: plugins/modules/route53_zone.py:0:0: import-error: Exception attempting to import module for argument_spec introspection, 'No module named 'ansible_collections.community.aws.plugins.module_utils.route53''

Issue Type

Bug Report

Component Name

route53

Ansible Version

CI

Collection Versions

Devel

AWS SDK versions

n/a

Configuration

n/a

OS / Environment

CI

Steps to Reproduce

Run sanity tests against devel

Expected Results

Tests pass

Actual Results

ERROR: plugins/modules/route53_health_check.py:0:0: import-error: Exception attempting to import module for argument_spec introspection, 'No module named 'ansible_collections.community.aws.plugins.module_utils.route53''

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@tremble
Copy link
Contributor Author

tremble commented Sep 23, 2022

CC: @alinabuzachis

@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug module module needs_triage plugins plugin (any type) python3 labels Sep 23, 2022
@alinabuzachis
Copy link
Collaborator

@tremble I guess we can close this one now.

abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Sep 18, 2023
…s#1064)

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 ansible-collections#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 <[email protected]>
Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Sep 18, 2023
…s#1064)

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 ansible-collections#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 <[email protected]>
Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Oct 24, 2023
…s#1064)

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 ansible-collections#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 <[email protected]>
Reviewed-by: Mark Woolley <[email protected]>
Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module plugins plugin (any type) python3
Projects
None yet
Development

No branches or pull requests

3 participants