feat(): adding knot-based ns_lord implementation#1187
Open
nils-wisiol wants to merge 14 commits intomainfrom
Open
feat(): adding knot-based ns_lord implementation#1187nils-wisiol wants to merge 14 commits intomainfrom
nils-wisiol wants to merge 14 commits intomainfrom
Conversation
nils-wisiol
force-pushed
the
20260203_nslord_knot
branch
2 times, most recently
from
b45ad6e to
9cf8325
Compare
- add guidance to down -v and rerun if e2e2 hits a startup 504
- add .cache and webapp Vite cache to .gitignore
nils-wisiol
force-pushed
the
20260203_nslord_knot
branch
from
c685597 to
49eab1d
Compare
- split change tracking into base + PDNS/Knot implementations with NSLord routing - add domain nslord field, serializer support, and migration - implement knot backend (DNSKEY/AXFR/updates) and nslord_knot service/config - update e2e2 and unit tests for knot domains, cert handling, and fail-fast - tune PDNS API timeout and nsmaster transfer intervals
- knot does not advance SOA serial on faketime shift yet
- knot lowercases AFSDB target, so skip the uppercase-only case
Skip faketime shifts for debugging and mark RRset update scenarios as known-knot failures to avoid flaky CI.\n\nAdjust RR canonicalization expectations for knot by skipping case-normalization-sensitive records and lowercasing expected CNAME/DNAME/NS results under the knot backend.
nils-wisiol
force-pushed
the
20260203_nslord_knot
branch
from
76725f0 to
214df0e
Compare
Contributor
Author
|
This is generally ready for review. There are still a couple of skipped e2e2 tests (marked as TODO commits) - let's discuss. Also, there are timeout failure in GitHub Actions that I cannot reproduce locally. I'm in the process of looking into these. |
nils-wisiol
force-pushed
the
20260203_nslord_knot
branch
from
8ed2e8f to
fc52c68
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.