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

feat: replace ansible.netcommon utils with python3 ipaddress module #416

Conversation

jooola
Copy link
Collaborator

@jooola jooola commented Dec 11, 2023

SUMMARY

Replace ansible.netcommon deprecated ipaddr utils with python ipaddress module. The ansible.netcommon collection is no longer required by the collections. We still use the ansible.utils collections for testing

@jooola jooola requested a review from apricote December 11, 2023 13:43
@jooola jooola force-pushed the replace-ansible.netcommon-with-ipaddress branch from 649aa22 to 7c71b26 Compare December 11, 2023 14:18
Copy link
Collaborator

@apricote apricote left a comment

Choose a reason for hiding this comment

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

Do you have a link for me for "ansible.netcommons is deprecated"? I did not find any indication of this on their repo.

@jooola
Copy link
Collaborator Author

jooola commented Dec 11, 2023

The ansible.netcommon collection is not deprecated, but the ipaddr utils present in it have been moved to the ansible.utils collection: ansible-collections/ansible.netcommon@db4920e

https://github.com/ansible-collections/ansible.netcommon/blob/95d2f6bd66ce5841d951b26790298cf97e3d591b/meta/runtime.yml#L15-L19

@jooola jooola merged commit 4cfdf50 into ansible-collections:main Dec 12, 2023
30 checks passed
@jooola jooola deleted the replace-ansible.netcommon-with-ipaddress branch December 12, 2023 10:50
jooola pushed a commit that referenced this pull request Feb 2, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.5.0](2.4.1...2.5.0)
(2024-02-02)


### Features

* add `hostvars_prefix` and `hostvars_suffix` options to inventory
hostvars
([#423](#423))
([4e3f89a](4e3f89a))
* allow forcing the deletion of firewalls that are still in use
([#447](#447))
([559d315](559d315))
* improve firewall resources management
([#324](#324))
([2757fe7](2757fe7))
* replace `ansible.netcommon` utils with python3 `ipaddress` module
([#416](#416))
([4cfdf50](4cfdf50))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants