Skip to content

Document dns_nsupdate provider in managing DNS section#3200

Merged
ekohl merged 1 commit intotheforeman:masterfrom
ekohl:add-managing-dns_nsupdate
Sep 6, 2024
Merged

Document dns_nsupdate provider in managing DNS section#3200
ekohl merged 1 commit intotheforeman:masterfrom
ekohl:add-managing-dns_nsupdate

Conversation

@ekohl
Copy link
Copy Markdown
Member

@ekohl ekohl commented Aug 8, 2024

This takes the idea that there should be a central place with documentation on all DNS providers. Not spread over multiple guides. It starts with the dns_nsupdate provider, but this should be completed with all of the providers.

This is split off from #2933 to keep the scope limited. Infoblox turned out to be much harder than expected.

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.12/Katello 4.14
  • Foreman 3.11/Katello 4.13
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6/6.7)
  • We do not accept PRs for Foreman older than 3.5.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 8, 2024

Comment thread guides/common/modules/proc_configuring-dns-nsupdate.adoc Outdated
@ekohl ekohl force-pushed the add-managing-dns_nsupdate branch from 2bffcbf to aca0670 Compare August 17, 2024 10:21
Copy link
Copy Markdown
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

Checked foreman-installer on Foreman 3.11: option is present ✔️

I believe you wanted to add additional information; feel free to extend your commit message or get additional technical ACK if you deem necessary.

LGTM from my side.

Comment thread guides/common/modules/proc_configuring-dns-nsupdate.adoc Outdated
Comment thread guides/common/modules/proc_configuring-dns-nsupdate.adoc Outdated
Comment thread guides/common/modules/proc_configuring-dns-nsupdate.adoc
Comment thread guides/common/modules/proc_configuring-dns-nsupdate.adoc Outdated
@ekohl
Copy link
Copy Markdown
Member Author

ekohl commented Aug 26, 2024

I should have noted that we already sort of have this documented in https://docs.theforeman.org/nightly/Installing_Server/index-katello.html#reverting-to-internal-dns-service_foreman (where I just noticed we have a dead anchor) and https://docs.theforeman.org/nightly/Installing_Server/index-katello.html#configuring-dns-dhcp-and-tftp_foreman. My primary aim was to provide readers with an easy way to set up only DNS instead of configuring DHCP & TFTP as well since they don't need to run on the same server. My general recommendation is to aim for a single DNS Smart Proxy unless you have a specific need for multiple.

This takes the idea that there should be a central place with
documentation on all DNS providers. Not spread over multiple guides. It
starts with the dns_nsupdate provider, but this should be completed with
all of the providers.
@ekohl ekohl force-pushed the add-managing-dns_nsupdate branch from 79f88d2 to acc5aeb Compare September 6, 2024 14:22
@ekohl
Copy link
Copy Markdown
Member Author

ekohl commented Sep 6, 2024

Rebased to resolve conflicts and I included @asteflova's suggestions, except the header one.

@ekohl ekohl merged commit fcb13f4 into theforeman:master Sep 6, 2024
@ekohl ekohl deleted the add-managing-dns_nsupdate branch September 6, 2024 16:25
@ekohl
Copy link
Copy Markdown
Member Author

ekohl commented Sep 6, 2024

cd7b2a1..691fc57 3.12 -> 3.12

And 3.11 had a conflict so I'll keep this to 3.12+.

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