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

RSS could provision external DNS zone #2837

Merged
merged 7 commits into from
Apr 17, 2023
Merged

Conversation

davepacheco
Copy link
Collaborator

Add an External DNS zone to the list of zones provisioned by RSS during rack setup. Like many other zones, this is only until we do #732.

While I was here, I carried forward more of the changes from #2570. In that PR we changed the DNS server config properties from server_address and dns_address to http_address and dns_address. I updated that in a few more places here (since I was adding new ones for external DNS that are parallel to the ones for internal DNS).

@davepacheco davepacheco marked this pull request as ready for review April 15, 2023 00:54
@davepacheco davepacheco requested a review from smklein April 15, 2023 00:56
@davepacheco davepacheco enabled auto-merge (squash) April 17, 2023 16:58
@davepacheco davepacheco merged commit 687eaf8 into main Apr 17, 2023
@davepacheco davepacheco deleted the dap/provision-external-dns branch April 17, 2023 18:05
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.

2 participants