-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Migrate metadata from TXT to DynamoDB registries #3726
Migrate metadata from TXT to DynamoDB registries #3726
Conversation
61b709a
to
96e7fba
Compare
Small nits other than that lgtm |
@szuecs I am awaiting answers to my questions on your review comments. |
/lgtm |
/test pull-external-dns-lint |
96e7fba
to
ef8b223
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johngmyers, szuecs The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
Description
Migrates the registry metadata from the TXT registry to the DynamoDB registry when the latter is enabled.
One annoyance is that the TXT ownership records don't get deleted unless TXT is added to the set of managed record types. It's not in the default set.
Checklist