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

added ddns_protected support #8

Closed
wants to merge 0 commits into from
Closed

Conversation

xdkreij
Copy link

@xdkreij xdkreij commented Jul 21, 2020

Herewith the suggestion to add the ddns protected feature towards the infoblox host record module.
We've seen a couple of cases where ddns starts to overwrite the created host record after a while, this feature provides the capability to protect the record from being overwritten.

Kind regards,

Xdk

@anagha-infoblox anagha-infoblox self-requested a review October 7, 2021 06:32
@anagha-infoblox
Copy link
Contributor

Hi @xdkreij,
Thank you for your contribution to the Infoblox NIOS modules. We had to restructure the directory as per the requirements. As a result, there are some merge conflicts that have to be resolved. Can you please consider the same? Sorry for the inconvenience caused.

Copy link
Collaborator

@hemanthKa677 hemanthKa677 left a comment

Choose a reason for hiding this comment

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

Can you please provide us the working example to test this changes?

@xdkreij
Copy link
Author

xdkreij commented Nov 16, 2022

Can you please provide us the working example to test this changes?

lol - only 2 years later..

regardless; Visit link for the actual DDNS change.

    ddns_protected=dict(type='bool', default=False, required=False),

note: everything in the fork/merge request == working example (or so it did 2 years ago)

@edeka-spatt
Copy link
Contributor

Please merge this request, i need the same feature.

Copy link
Collaborator

@hemanthKa677 hemanthKa677 left a comment

Choose a reason for hiding this comment

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

Hi @xdkreij , Can you please check with conflict raising with file? So that we can merge this asap.

@xdkreij
Copy link
Author

xdkreij commented Jan 24, 2023

Hi @xdkreij , Can you please check with conflict raising with file? So that we can merge this asap.

Merged my fork with the infoblox-ansible repository; It should be up 2 date, kindly review.

@xdkreij
Copy link
Author

xdkreij commented Jan 24, 2023

visit #169 for new up2date request

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