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

API to generate DNSRecord endpoints for a given strategy and Gateway #187

Closed
mikenairn opened this issue Jul 11, 2024 · 0 comments · Fixed by #207 or Kuadrant/kuadrant-operator#799
Assignees

Comments

@mikenairn
Copy link
Member

mikenairn commented Jul 11, 2024

What

Add a public API to the dns operator that generates endpoints for a given strategy (simple/loadbalanced) and Gateway. This should replace the logic currently in the kuadrant operator.

Why

  • Allows the logic generating the endpoints to exist next to the code that is expected to reconcile the results.
  • Allows us to more easily create test cases based on strategies

Notes

  • It is not the intention that the dnsrecord controller itself will become aware of strategies. It should continue to function based on the contents of the DNSRecord.Endpoints
  • Strategies will now be defined by the dns operator and not the kuadrant operator
  • Do we remove ACM support which complicates things? Yes
@philbrookes philbrookes moved this to Todo in Kuadrant Jul 11, 2024
@maksymvavilov maksymvavilov self-assigned this Jul 31, 2024
@maksymvavilov maksymvavilov moved this from Todo to In Progress in Kuadrant Jul 31, 2024
github-merge-queue bot pushed a commit that referenced this issue Sep 26, 2024
Add api to generate endpoints
@github-project-automation github-project-automation bot moved this from Ready For Review to Done in Kuadrant Sep 26, 2024
@maksymvavilov maksymvavilov reopened this Sep 26, 2024
@maksymvavilov maksymvavilov moved this from Done to Ready For Review in Kuadrant Sep 26, 2024
@maksymvavilov maksymvavilov moved this from Ready For Review to In Progress in Kuadrant Sep 26, 2024
@maksymvavilov maksymvavilov moved this from In Progress to Ready For Review in Kuadrant Sep 27, 2024
@github-project-automation github-project-automation bot moved this from Ready For Review to Done in Kuadrant Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
3 participants