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

DNS records are re-created even if heath check is failing #319

Closed
trepel opened this issue Nov 27, 2024 · 4 comments · Fixed by Kuadrant/kuadrant-operator#1051
Closed

DNS records are re-created even if heath check is failing #319

trepel opened this issue Nov 27, 2024 · 4 comments · Fixed by Kuadrant/kuadrant-operator#1051

Comments

@trepel
Copy link

trepel commented Nov 27, 2024

I scaled down my backend so that my DNS heath_check started failing. I manually removed the DNS records from cloud provider (AWS in my case). The records were re-created in AWS.

Unclear if this is the desired behavior or not, discussion needed.

@trepel trepel added this to Kuadrant Nov 27, 2024
@maleck13
Copy link
Collaborator

I don't consider this a blocker for v1 but I think we can document it. Really it is unadvisable to manually change DNS records managed by the controller.

There are other ways to achieve the same result:

  • Delete the listener from the gateway
  • If you have a DNSPolicy per listener, delete the dnspolicy for that listener
  • (possibly) remove the DNSRecord directly (the health check should stop this from being re-published)

cc @maksymvavilov @mikenairn any other options for un-publishing a record set?

@trepel
Copy link
Author

trepel commented Nov 27, 2024

The third options works in rc10. The records are removed along with DNSRecord removal. The DNSRecord itself gets recreated but ends up with Not publishing unhealthy records so nothing is created in cloud provider. Everything gets sorted once backend is scaled up.

@maleck13
Copy link
Collaborator

nice this is what we will document so!

@maleck13
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants