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.

Sign up for GitHub

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

Jump to bottom

Need to replace distutils.* in ansible.posix to address PEP 632 #303

Closed
2 tasks
saito-hideki opened this issue Dec 22, 2021 · 0 comments · Fixed by #304
Closed
2 tasks

Need to replace distutils.* in ansible.posix to address PEP 632 #303

saito-hideki opened this issue Dec 22, 2021 · 0 comments · Fixed by #304
Assignees
@saito-hideki
Labels
bug This issue/PR relates to a bug. has_pr verified This issue has been verified/reproduced by maintainer

Comments

@saito-hideki
Copy link
Collaborator

saito-hideki commented Dec 22, 2021 •
edited
Loading

SUMMARY

distutils module is depricated in Python3.10 and it will be removed in Python3.12:

In Python 3.12, distutils will no longer be installed by make install or any of the first-party distribution. Third-party redistributors should no longer include distutils in their bundles or repositories.

We need to address the above changes on ansible.posix side.

ISSUE TYPE
  • Bug report
COMPONENT NAME
  • plugins/module_utils/firewalld.py
  • plugins/modules/firewalld_info.py
ADDITIONAL INFORMATION
@saito-hideki saito-hideki self-assigned this Dec 22, 2021
@saito-hideki saito-hideki added feature This issue/PR relates to a feature request. verified This issue has been verified/reproduced by maintainer bug This issue/PR relates to a bug. and removed feature This issue/PR relates to a feature request. labels Dec 22, 2021
@saito-hideki saito-hideki mentioned this issue Dec 23, 2021
Merged
Akasurde pushed a commit to Akasurde/ansible.posix that referenced this issue Jan 6, 2022
@saito-hideki @Akasurde
Replace distutils with included module in ansible-core 2.12
a45448b 
- Fixes ansible-collections#303
- Addresses PEP 632(https://www.python.org/dev/peps/pep-0632/)

Signed-off-by: Hideki Saito <[email protected]>
ansible-zuul bot added a commit that referenced this issue Jan 7, 2022
@ansible-zuul
Merge pull request #304 from saito-hideki/issue/303
3acced9 
Replace distutils with included module in ansible-core 2.12 to address PEP 632

SUMMARY
Replace distutils.* with included module in ansible-core 2.12 to addresss PEP 632 if available.
It does not change the behavior of ansible-core 2.11 or earlier.

Fixes #303
Addresses PEP 632(https://www.python.org/dev/peps/pep-0632/)

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

plugins/module_utils/firewalld.py


plugins/modules/firewalld_info.py



ADDITIONAL INFORMATION
None

Reviewed-by: Andrew Klychkov <[email protected]>
Reviewed-by: Hideki Saito <[email protected]>
Reviewed-by: Felix Fontein <[email protected]>
Reviewed-by: None <None>
@bot-ross bot-ross bot mentioned this issue May 23, 2022
Merged
1 task
@midnight-bot-6658 midnight-bot-6658 bot mentioned this issue May 23, 2022
Merged
1 task
@negative-7-bot negative-7-bot bot mentioned this issue May 23, 2022
Merged
1 task
@renovate renovate bot mentioned this issue May 23, 2022
Merged
1 task
@lumiere-bot lumiere-bot bot mentioned this issue May 23, 2022
Merged
1 task
@stimpy-bot stimpy-bot bot mentioned this issue May 23, 2022
Merged
1 task
@wyoa-bot wyoa-bot bot mentioned this issue May 23, 2022
Merged
1 task
This was referenced May 23, 2022
Merged
Merged
Merged
@ghost ghost mentioned this issue May 23, 2022
Merged
1 task
@renovate renovate bot mentioned this issue May 23, 2022
Merged
1 task
@iudicael-bot iudicael-bot bot mentioned this issue May 23, 2022
Merged
1 task
@bloopy-boi bloopy-boi bot mentioned this issue May 23, 2022
Merged
1 task
This was referenced May 23, 2022
Open
Closed
Closed
Merged
Merged
Merged
@renovate renovate bot mentioned this issue May 27, 2022
Merged
1 task
@aki-bot aki-bot bot mentioned this issue May 28, 2022
Merged
1 task
This was referenced May 28, 2022
Closed
Merged
Open
Merged
Open
Merged
Merged
Closed
Merged
Closed
Merged
Merged
This was referenced Jun 5, 2022
Merged
Closed
Merged
This was referenced Jun 18, 2022
Closed
Closed
This was referenced Jul 31, 2022
Closed
Merged
This was referenced Aug 17, 2022
Merged
Closed
Closed
This was referenced Sep 14, 2022
Closed
Closed
@trux-bot trux-bot bot mentioned this issue Sep 16, 2022
Merged
1 task
@snoopy82481-bot snoopy82481-bot bot mentioned this issue Sep 21, 2022
Closed
1 task
@renovate renovate bot mentioned this issue Oct 2, 2022
Merged
1 task
@renovate-tests renovate-tests mentioned this issue Nov 24, 2022
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@saito-hideki saito-hideki

Labels
bug This issue/PR relates to a bug. has_pr verified This issue has been verified/reproduced by maintainer
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace distutils with included module in ansible-core 2.12 to address PEP 632
1 participant
@saito-hideki