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

registry: Use txt registry for owned records #56

Merged
merged 3 commits into from
Mar 20, 2024

Conversation

mikenairn
Copy link
Member

@mikenairn mikenairn commented Mar 11, 2024

Use the TXT Registry when an ownerID is specified for a record being processed. Adds an optional field ownerID to the DNSRecord spec. This is immutable since changing it after initial creation will likely not result in desirable behaviour.

Requires:

closes #48

@mikenairn mikenairn force-pushed the 48_use_txt_registry branch 2 times, most recently from f2a112e to 204b52b Compare March 15, 2024 16:30
@mikenairn mikenairn changed the title WIP registry: Use txt registry for owned records registry: Use txt registry for owned records Mar 15, 2024
@mikenairn mikenairn marked this pull request as ready for review March 15, 2024 16:34
We will be using a forked version of external-dns for the time being and
will track the latest release version (v0.14.0) + kuadrant related
changes in
https://github.com/Kuadrant/external-dns/tree/v0.14.0_kuadrant
@mikenairn mikenairn force-pushed the 48_use_txt_registry branch from 204b52b to 97e32be Compare March 20, 2024 12:29
Use the TXT Registry when an `ownerID` is specified for a record being
processed.  Adds an optional field `ownerID` to the DNSRecord spec. This
is immutable since changing it after initial creation will likely not
result in desirable behaviour.

Update e2e tests to verify contents of zone after record creation.
Add `local-cleanup` make target which deletes the development kind
cluster created by `make local-setup`.
@mikenairn mikenairn force-pushed the 48_use_txt_registry branch from 97e32be to 00b6621 Compare March 20, 2024 12:34
Copy link
Contributor

@philbrookes philbrookes left a comment

Choose a reason for hiding this comment

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

lgtm

@mikenairn mikenairn merged commit 0f0feb3 into Kuadrant:main Mar 20, 2024
9 checks passed
@mikenairn mikenairn deleted the 48_use_txt_registry branch March 20, 2024 17:19
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.

Update default registry to the TXT Registry
2 participants