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.

Sign up for GitHub

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

Jump to bottom

report relevant zone records in DNS Record status #205

Closed
philbrookes opened this issue Aug 1, 2024 · 0 comments · Fixed by #241
Closed

report relevant zone records in DNS Record status #205

philbrookes opened this issue Aug 1, 2024 · 0 comments · Fixed by #241
Assignees
@maksymvavilov

Comments

@philbrookes
Copy link
Collaborator

philbrookes commented Aug 1, 2024 •
edited
Loading

Get all records from the root domain of this DNS Record down to the leaf records (for all owner IDs), and report them (along with owner IDs) in the status of the DNS Record.

In the same format as proposed in #204

zone overview:
  array of yaml output for each endpoint pulled from the zone

Build this from the data pulled from the provider during reconcile if possible, and avoid making a new request to the DNS Provider API.

Add tests to validate the status is updated correctly.
@maksymvavilov maksymvavilov self-assigned this Sep 6, 2024
maksymvavilov added a commit that referenced this issue Sep 19, 2024
@maksymvavilov
gh-205 add related endpoints to DNSRecord status
967cd55 
Signed-off-by: Maskym Vavilov <[email protected]>
@maksymvavilov maksymvavilov linked a pull request Sep 19, 2024 that will close this issue
Add related endpoints to DNSRecord status #241
Merged
maksymvavilov added a commit that referenced this issue Sep 19, 2024
@maksymvavilov
gh-205 add related endpoints to DNSRecord status
2f541bb 
Signed-off-by: Maskym Vavilov <[email protected]>
maksymvavilov added a commit that referenced this issue Sep 26, 2024
@maksymvavilov
gh-205 add related endpoints to DNSRecord status
dddc636 
Signed-off-by: Maskym Vavilov <[email protected]>
maksymvavilov added a commit that referenced this issue Sep 26, 2024
@maksymvavilov
gh-205 add related endpoints to DNSRecord status
e5e39ae 
Signed-off-by: Maskym Vavilov <[email protected]>
maksymvavilov added a commit that referenced this issue Oct 7, 2024
@maksymvavilov
gh-205 add related endpoints to DNSRecord status
46255e0 
Signed-off-by: Maskym Vavilov <[email protected]>
github-merge-queue bot pushed a commit that referenced this issue Oct 18, 2024
@maleck13
Merge pull request #241 from Kuadrant/gh-205
3821308 
Add related endpoints to DNSRecord status
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maksymvavilov maksymvavilov

Labels
None yet
Projects
Kuadrant
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add related endpoints to DNSRecord status Kuadrant/dns-operator
2 participants
@philbrookes @maksymvavilov