Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

[Associated vote ended on 2023-08-30] Update Python requirements for collections included in Ansible #251

Closed
felixfontein opened this issue Jul 10, 2023 · 13 comments
Labels
being_implemented This is currently being implemented discussion_topic

Comments

@felixfontein
Copy link
Contributor

Summary

The current Ansbile inclusion requirements have a session on Python requirements (https://docs.ansible.com/ansible/latest/community/collection_contributors/collection_requirements.html#python-requirements) that's somewhat outdated. Python 2.7 and 3.5/3.6/3.7/3.8 are no longer supported on the controller for recent ansible-core releases, and Python 2.6 is no longer supported on the target for recent ansible-core releases. Python 3.5 support will also be dropped on the target side soon.

We should update these, and in particular try to rephrase these conditions so we don't mention explicit Python versions that have to be updated regularly. There is already a proposal - see ansible/ansible-documentation#68 by the core team - which we should discuss, and possibly amend.

@felixfontein
Copy link
Contributor Author

(A useful link is https://docs.ansible.com/ansible/devel/reference_appendices/release_and_maintenance.html#support-life, which shows which Python versions are supported by which ansible-core versions. This is the link the reference in the PR points to.)

@gotmax23 gotmax23 added the next_meeting Topics that needs to be discussed in the next Community Meeting label Jul 12, 2023
@samccann samccann removed the next_meeting Topics that needs to be discussed in the next Community Meeting label Jul 19, 2023
@mariolenz
Copy link
Contributor

This might be relevant, at least in the future: ansible-collections/news-for-maintainers#54

@felixfontein
Copy link
Contributor Author

How about:

  1. Python content in a collection MUST support all Python versions that are supported on the controller side by at least one ansible-core version supported by the collection. The only exception are Python versions not supported by libraries the Python content depends on. Other exceptions need to be approved on a case-by-case basis by the Steering Committee.
  2. Python content in a collection SHOULD support all Python versions that are supported on the target side by at least one ansible-core version supported by the collection.

@felixfontein
Copy link
Contributor Author

As discussed in today's community meeting: I'll convert the above post into a PR during the next days, then we can word-smith it, and start a vote once everyone's happy with the wording.

@felixfontein
Copy link
Contributor Author

I tried writing this up in ansible/ansible-documentation#281.

@felixfontein felixfontein added the active-vote-discussion These are proposals that are currently voted on label Aug 16, 2023
@felixfontein
Copy link
Contributor Author

There is now an active vote for this in #268.

@felixfontein felixfontein moved this from Backlog to In Progress in Community Topics TODO Aug 16, 2023
@felixfontein felixfontein changed the title Update Python requirements for collections included in Ansible [vote ends on 2023-08-30] Update Python requirements for collections included in Ansible Aug 16, 2023
@felixfontein felixfontein changed the title [vote ends on 2023-08-30] Update Python requirements for collections included in Ansible [Associated vote ends on 2023-08-30] Update Python requirements for collections included in Ansible Aug 16, 2023
@felixfontein
Copy link
Contributor Author

@ansible-community/steering-committee please do not forget about this vote. So far only four SC members voted, and the two-weeks voting period ends tomorrow.

@felixfontein
Copy link
Contributor Author

@acozine @cidrblock @briantist @thaumos @jamescassell @gundalow @ssbarnea the vote currently doesn't reach quorum (which is 7; so far only 6 SC members voted). Since the vote is supposed to end today, it would be great if you could take a look and also vote.

@felixfontein felixfontein changed the title [Associated vote ends on 2023-08-30] Update Python requirements for collections included in Ansible [Associated vote ended on 2023-08-30] Update Python requirements for collections included in Ansible Aug 30, 2023
@felixfontein felixfontein removed the active-vote-discussion These are proposals that are currently voted on label Aug 30, 2023
@felixfontein
Copy link
Contributor Author

I counted votes:

  • 8 x +1 from SC members (felixfontein gotmax23 markuman russoz Andersson007 mariolenz acozine cidrblock)
  • 4 x +1 from community members (aretrosen spyrexd sean-m-sullivan leogallego)

Could another SC member please also count the votes? Thanks :)

@mariolenz
Copy link
Contributor

I confirm your count @felixfontein: 8 x +1 from SC members, 4 x +1 from community members, no other votes.

Thanks everyone!

@felixfontein
Copy link
Contributor Author

@mariolenz thanks for counting, and everyone else thanks for participating!

@felixfontein felixfontein added the being_implemented This is currently being implemented label Aug 31, 2023
@felixfontein
Copy link
Contributor Author

I'm keeping this topic open for a bit so we can discuss improvents to the wording, that came up during the review of ansible/ansible-documentation#281 when the vote was already running.

@felixfontein
Copy link
Contributor Author

Closing since nobody started working on this. If this happens, we can re-open (or create a new issue).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
being_implemented This is currently being implemented discussion_topic
Projects
No open projects
Status: Resolved
Development

No branches or pull requests

4 participants