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

Bugfix/second record with same name option 1 #202

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

lenseB
Copy link

@lenseB lenseB commented Nov 29, 2023

I am developing an Ansible Playbook automating IPAM.
While using the infoblox.nios_modules I noticed a weird behaviour. After creating a hostrecord with an IP-address x and a hostname, I executed the playbook again now with a different IP-address y and the same hostname. The secon hostrecord was placed in the IP-Adress y but also the first hostrecord was placed there. Also the extra attributes of the first hostrecord changed.

This was already described in #108

I fixed the problem in the commit 58cb936. This only fixes the problem if the if_obj_type is "NIOS_HOST_RECORD".
A general fix is going to be given in the PR: Bugfix/second record with same name option 2. I can not forsee if the general fix influences any other use case though.

for the case of using  the "nios_host_record"-module:
- when there is one hostrecord with the same name already, it should behave like there are already >=2 hostrecords.
- therefore the if-statement needs to be adjusted thatthe else-statement in line 343 is not executed
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.

1 participant