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

Remove requirement for DNS to exist before machine is created #307

Merged
merged 3 commits into from
Apr 8, 2025

Conversation

SchoolGuy
Copy link
Collaborator

Fixes #253

This PR removes the need for Orthos 2 to have machines added to the DNS before entering their data. This will use Cobbler under the hood to generate the zonefile that is needed so the machines are resolvable via DNS.

@SchoolGuy SchoolGuy added this to the v1.4 milestone Feb 14, 2025
@SchoolGuy SchoolGuy modified the milestones: v1.4, v1.5 Mar 10, 2025
@SchoolGuy SchoolGuy removed this from the v1.5 milestone Mar 21, 2025
@marv7000 marv7000 self-assigned this Apr 1, 2025
@SchoolGuy SchoolGuy force-pushed the feature/remove-dns-check branch from d46af16 to ed164f0 Compare April 3, 2025 12:56
@marv7000 marv7000 force-pushed the feature/remove-dns-check branch from ed164f0 to c54832a Compare April 4, 2025 13:41
@SchoolGuy SchoolGuy requested a review from marv7000 April 8, 2025 12:04
@SchoolGuy SchoolGuy force-pushed the feature/remove-dns-check branch from 15e4022 to 1451ba7 Compare April 8, 2025 12:22
Cobbler generated the DNS entries that match the machine. As such there is no need that the DNS entry is existing before the machine itself is entered into Cobbler.

Co-authored-by: Marvin Friedrich <[email protected]>
@SchoolGuy SchoolGuy force-pushed the feature/remove-dns-check branch from 1451ba7 to fe4f275 Compare April 8, 2025 12:29
@SchoolGuy SchoolGuy marked this pull request as ready for review April 8, 2025 12:29
@SchoolGuy SchoolGuy added this to the v1.6 milestone Apr 8, 2025
@SchoolGuy
Copy link
Collaborator Author

The remaining TODO for this PR is to refactor add_primary_network_interface() that it allows adding any interface that either has an IPv4 or an IPv6 address assigned.

@marv7000 marv7000 force-pushed the feature/remove-dns-check branch from d47adc8 to f499698 Compare April 8, 2025 13:24
@marv7000 marv7000 force-pushed the feature/remove-dns-check branch from f499698 to 3714504 Compare April 8, 2025 13:29
@marv7000 marv7000 merged commit 7305a8a into master Apr 8, 2025
11 checks passed
@marv7000 marv7000 deleted the feature/remove-dns-check branch April 8, 2025 13:42
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.

Preserve IPs of the managed hosts in the Orthos database
2 participants