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-DNS coupling #67

Closed
jean1 opened this issue Sep 26, 2023 · 2 comments · Fixed by #102
Closed

IPAM-DNS coupling #67

jean1 opened this issue Sep 26, 2023 · 2 comments · Fixed by #102
Assignees
Labels
enhancement New feature or request

Comments

@jean1
Copy link
Contributor

jean1 commented Sep 26, 2023

Problem statement
We need consistency between IP addresses and names:

  • when adding an IP address, a (A or AAAA) DNS record must be created.
  • when deleting and IP address, the corresponding DNS record must be deleted.

Approach
The solution is based on three additional fields associated with the IP Address, "name", "zone" and "dns_record". When we submit the form or post data to the API, both the IP address and the DNS record are created, and the "dns_record" field points to the record. Ditto for deleting the IP Address with a linked DNS record.

Alternatives
As an alternative, I have considered using the "dns_name" field. Using this field entails several problems:

  • it is only an informative field,
  • as a character string, there is no possibility to use the Zone object,
  • there is no way to enforce permissions on DNS objects (Zone, Records),
  • finally, there is no guarantee of consistency between DNS records and IP addresses.
@peteeckel
Copy link
Owner

peteeckel commented Oct 17, 2023

Hi @jean1, I just released version 0.20.0, which mainly adds your new feature. Thank you very much for this!

@peteeckel
Copy link
Owner

Hi @jean1, after another round of commits (this time abandoning the middleware concept altogether) I think the experimental feature should be generally working, even with custom scripts (see issue #105).

I'm planning on some enhancements (making the automatism more general and more comfortable), but for the time being this looks pretty useful to me. Thank you very much again for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants