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

IPAM Address and Next Available IP Support for Bloxone Ansible v2 #55

Merged
merged 22 commits into from
Jan 22, 2025

Conversation

ybhalchim
Copy link

@ybhalchim ybhalchim commented Dec 11, 2024

This PR contains :

  • Support for Address and Address Info module
  • Support for Next Available IP
  • Unit tests corresponding to added attributes
  • Fix for Address Block UT by changing required_together to required_by in address_block.py

@unasra unasra changed the title ipam_address Support for Bloxone Ansible v2 IPAM Address Support for Bloxone Ansible v2 Dec 17, 2024
@unasra unasra self-requested a review December 17, 2024 05:42
@unasra
Copy link
Collaborator

unasra commented Dec 17, 2024

Missing change logs and deprecation notices for this object .

Earlier this module was termed as ipv4_reservation !

@unasra unasra changed the title IPAM Address Support for Bloxone Ansible v2 IPAM Address and Next Available IP Support for Bloxone Ansible v2 Jan 20, 2025
@unasra
Copy link
Collaborator

unasra commented Jan 21, 2025

LGTM
@mathewab Please take a look !

@unasra unasra requested a review from mathewab January 21, 2025 11:52
@mathewab mathewab merged commit f40219b into infobloxopen:v2 Jan 22, 2025
16 checks passed
unasra pushed a commit to unasra/bloxone-ansible that referenced this pull request Feb 4, 2025
…fobloxopen#55)

* Ipam_address_Integration test cases

* Added address info

* modified Ipam_Address according Sanity test

* modified ipam_address tests

* added ChangeLog and Deprecation Notice

* added extra spaces

* modifies the examples from .py files and .yml files

* added spaces in .yml files

* refactor the address

* refactor code as per comment

* changed Asset statement

* added cleanup task

* added next available ip functionality

* added changelog

* added extra space

* fixed antsibull-docs and next_available_ip_info

* added required_by instead of required_together in address_block

* modified the documentation
unasra pushed a commit to unasra/bloxone-ansible that referenced this pull request Feb 4, 2025
…fobloxopen#55)

* Ipam_address_Integration test cases

* Added address info

* modified Ipam_Address according Sanity test

* modified ipam_address tests

* added ChangeLog and Deprecation Notice

* added extra spaces

* modifies the examples from .py files and .yml files

* added spaces in .yml files

* refactor the address

* refactor code as per comment

* changed Asset statement

* added cleanup task

* added next available ip functionality

* added changelog

* added extra space

* fixed antsibull-docs and next_available_ip_info

* added required_by instead of required_together in address_block

* modified the documentation
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.

4 participants