Remove Python 3.10 support for the controller #83221
Conversation
ansibot
added
feature
| @@ -32,27 +32,8 @@ def import_module(name): # type: ignore[misc] | |||
| __import__(name) | |||
| return sys.modules[name] | |||
|
|
|||
| try: | |||
There was a problem hiding this comment.
This Python module needs to support target Python versions, not just controller Python versions. So the Python 2.7 support can be dropped, but not Python 3.8.
There was a problem hiding this comment.
On a related note, there is more Python 2.7 code that can be removed, such as the use of ansible.module_utils.six.
There was a problem hiding this comment.
ansible.module_utils.six is used in quite a few collections all over the place.
There was a problem hiding this comment.
We're not removing that feature -- we're just discontinuing use of it in core where it's no longer needed.
There was a problem hiding this comment.
That's totally sensible :) Thanks for clarifying!
ansibot
removed
the
ci_verified
webknjaz
added
the
ci_verified
|
This PR is blocked on completion of #83093 |
mkrizek
removed
the
needs_triage
ansibot
added
the
stale_ci
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
I've restarted the tests now that the AZP container has been updated to include Python 3.11. Before this PR is merged we should check on the status of the updates to the other projects using AZP: ansible-collections/news-for-maintainers#71 Any projects testing against |
ansibot
removed
ci_verified
|
The test |
| # Remove this once py3.11 is our controller minimum | ||
| # Workaround for https://bugs.python.org/issue47231 | ||
| # See _extract_tar_dir | ||
| collection_tar._ansible_normalized_cache = { |
There was a problem hiding this comment.
There are unit tests that still refer to this and will need to be updated.
webknjaz
added
the
ci_verified
SUMMARY
Fixes #83094
ISSUE TYPE