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

aws_ec2: remove py2 compatiblity header #505

Conversation

goneri
Copy link
Member

@goneri goneri commented Sep 15, 2021

@ansibullbot
Copy link

@goneri: Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information.

Here are the items we could not find in your description:

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot ansibullbot added inventory inventory plugin needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly needs_triage plugins plugin (any type) small_patch Hopefully easy to review labels Sep 15, 2021
mdellweg pushed a commit to mdellweg/amazon.aws that referenced this pull request Sep 16, 2021
…le-collections#505)

* ec2_instance don't change termination protection in check mode

Fixes: ansible/ansible/issues/67716
Extend termination protection tests

* Set the path for the aws CLI tool - setting ansible_python_interpreter updates the python search path but not the shell search path

* changelog

Co-authored-by: Mark Chappell <mchappel@redhat.com>
@tremble
Copy link
Contributor

tremble commented Sep 17, 2021

recheck

1 similar comment
@alinabuzachis
Copy link
Collaborator

recheck

@tremble
Copy link
Contributor

tremble commented Sep 23, 2021

Closing this. Until the containers support disabling the python2 compatability headers we'll still need to add them. Filling ignore.txt with skip/ignore entries just makes the ignore.txt files unreadable.

@tremble tremble closed this Sep 23, 2021
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…le-collections#505)

* ec2_instance don't change termination protection in check mode

Fixes: ansible/ansible/issues/67716
Extend termination protection tests

* Set the path for the aws CLI tool - setting ansible_python_interpreter updates the python search path but not the shell search path

* changelog

Co-authored-by: Mark Chappell <mchappel@redhat.com>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…le-collections#505)

* ec2_instance don't change termination protection in check mode

Fixes: ansible/ansible/issues/67716
Extend termination protection tests

* Set the path for the aws CLI tool - setting ansible_python_interpreter updates the python search path but not the shell search path

* changelog

Co-authored-by: Mark Chappell <mchappel@redhat.com>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…le-collections#505)

* ec2_instance don't change termination protection in check mode

Fixes: ansible/ansible/issues/67716
Extend termination protection tests

* Set the path for the aws CLI tool - setting ansible_python_interpreter updates the python search path but not the shell search path

* changelog

Co-authored-by: Mark Chappell <mchappel@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inventory inventory plugin needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants