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

Update the default module requirements from python-2.6/boto to python-3.6/botocore-1.16 (breaking change) #298

Merged
merged 5 commits into from
May 19, 2021

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Mar 10, 2021

Depends-On: #302

SUMMARY

The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.

Switch the defaults over to reduce the copy&paste and reflect that

  • boto v2 is deprecated
  • python 2.6 is no longer supported by the AWS collections
ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

plugins/doc_fragments/aws.py

ADDITIONAL INFORMATION

Should wait for the 2.0 release so we can cleanly update everything.

@tremble
Copy link
Contributor Author

tremble commented Mar 10, 2021

@jillr in your presentation I think you suggested dropping 2.7 support as a part of the 2.0.0 release since boto3 is going to drop support soon. If we're going to do that I guess we update the python requirement in here to 3.6 rather than 2.7.

@ansibullbot ansibullbot added lookup lookup plugin module module labels Mar 10, 2021
@goneri
Copy link
Member

goneri commented Mar 12, 2021

recheck

@jillr
Copy link
Collaborator

jillr commented Apr 19, 2021

@tremble Yeah 1.5 will be the last 2.x supporting release, so we can bump this to py3.6 for collection version 2.0

@ansibullbot
Copy link

@ansibullbot ansibullbot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR and removed community_review labels Apr 19, 2021
@ansibullbot ansibullbot added community_review and removed needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Apr 28, 2021
Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

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

I'm very excited about this PR. :) LGTM once the merge conflict is resolved.
@tremble do you have a PR queued up to add the boto requirement to community modules that are still boto2 or shall I take care of that?

@tremble tremble changed the title Update the default module requirements from python2.6/boto to python2.7/boto3 (breaking change) Update the default module requirements from python2.6/boto to python3.6/boto3 (breaking change) May 5, 2021
@tremble tremble force-pushed the docs/require-boto3 branch 2 times, most recently from b07bd5c to 38d10ac Compare May 6, 2021 07:46
@tremble tremble changed the title Update the default module requirements from python2.6/boto to python3.6/boto3 (breaking change) Update the default module requirements from python2.6/boto to python3.6/botocore-1.16 (breaking change) May 6, 2021
@tremble tremble changed the title Update the default module requirements from python2.6/boto to python3.6/botocore-1.16 (breaking change) Update the default module requirements from python-2.6/boto to python-3.6/botocore-1.16 (breaking change) May 6, 2021
@tremble
Copy link
Contributor Author

tremble commented May 6, 2021

@jillr I don't currently have a PR lined up for community.aws but I can get that lined up in the next couple of days, was mostly waiting on this to land (and decisions from your side on version support policies)

I've also added botocore >= 1.16.0 based on the policy in ansible/ansible#74585 (in theory we could bump this to something around 1.16.5 or even 1.17.0 but now there's a general policy outlined I'd be inclined to go slightly older (and with .0 release) knowing we can bump it with the next 'major' release.

@jillr
Copy link
Collaborator

jillr commented May 14, 2021

recheck

@jillr jillr added the gate label May 19, 2021
@ansible-zuul ansible-zuul bot merged commit 33c7196 into ansible-collections:main May 19, 2021
@tremble tremble deleted the docs/require-boto3 branch May 28, 2021 07:07
ansible-zuul bot added a commit to ansible-collections/community.aws that referenced this pull request Jul 5, 2021
Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 15, 2021
…s/require-boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@f186c9a
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
…s/require-boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@f186c9a
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 17, 2021
…s/require-boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@3692d16
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections/amazon.aws#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…boto3

Update the default module requirements from python 2.6/boto to python 3.6/boto3

SUMMARY
The vast bulk of modules are now boto3 based, but some even list python2.6/boto as their requirements because they've picked the default from the aws docs fragment.
Switch the defaults over to reduce the copy&paste and reflect that

boto v2 is deprecated
python 2.6 is no longer supported by the AWS collections

ISSUE TYPE

Docs Pull Request

COMPONENT NAME
lots of them
ADDITIONAL INFORMATION
Depends-On: ansible-collections#298

Reviewed-by: Jill R <None>
Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review docs lookup lookup plugin module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants