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 Host Support for Bloxone Ansible v2 #52

Merged
merged 24 commits into from
Jan 9, 2025

Conversation

VishrutiBuddhadev
Copy link

  • Generated resources for ipam_host
  • Written integration tests
  • deprecate b1_ipam_host and b1_ipam_host_gather

@VishrutiBuddhadev VishrutiBuddhadev marked this pull request as ready for review December 10, 2024 16:30
@VishrutiBuddhadev VishrutiBuddhadev changed the title ipam_host implementation for Bloxone Ansible v2 ipam_host support for Bloxone Ansible v2 Dec 11, 2024
@VishrutiBuddhadev VishrutiBuddhadev changed the title ipam_host support for Bloxone Ansible v2 ipam_host Support for Bloxone Ansible v2 Dec 11, 2024
@unasra unasra changed the title ipam_host Support for Bloxone Ansible v2 IPAM Host Support for Bloxone Ansible v2 Dec 16, 2024
@unasra unasra self-requested a review December 17, 2024 05:28
@unasra unasra self-requested a review January 3, 2025 10:31
@unasra
Copy link
Collaborator

unasra commented Jan 6, 2025

LGTM
@mathewab please take a look.

@unasra unasra requested a review from mathewab January 6, 2025 15:19
@mathewab mathewab merged commit c7fccd1 into infobloxopen:v2 Jan 9, 2025
16 checks passed
unasra pushed a commit to unasra/bloxone-ansible that referenced this pull request Feb 4, 2025
* generated resources for ipam_host and added examples

* written integration tests for ipam_host

* deprecate b1_ipam_host and b1_ipam_host_gather

* fixed sanity issues and added empty line at the end

* removed instances of inherit

* fixed documentation and added empty line at the end of file

* refactored integration tests

* addressed review comments

* fixed sanity issues

* modified examples

* fixed sanity issues

* addressed review comments

* added new line at the end

* modified examples

* modified examples

* added code for cleanup

* addressed review comments

* added new line

* addressed review comments

* addressed review comments

---------

Co-authored-by: Ashish Mathew <54074687+mathewab@users.noreply.github.com>
unasra pushed a commit to unasra/bloxone-ansible that referenced this pull request Feb 4, 2025
* generated resources for ipam_host and added examples

* written integration tests for ipam_host

* deprecate b1_ipam_host and b1_ipam_host_gather

* fixed sanity issues and added empty line at the end

* removed instances of inherit

* fixed documentation and added empty line at the end of file

* refactored integration tests

* addressed review comments

* fixed sanity issues

* modified examples

* fixed sanity issues

* addressed review comments

* added new line at the end

* modified examples

* modified examples

* added code for cleanup

* addressed review comments

* added new line

* addressed review comments

* addressed review comments

---------

Co-authored-by: Ashish Mathew <54074687+mathewab@users.noreply.github.com>
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.

3 participants