-
Notifications
You must be signed in to change notification settings - Fork 484
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
node-local-dns
should support graceful shutdown with the ready
plugin
#642
Comments
Hi, I’m currently exploring this option( cilium + nodelocal dns), but I see no reason to use image provided by this project instead of using coredns directly, did you try to use coredns directly? Is there any hidden disadvantages? I tried to use coredns instead and even drafted a PR for this. Hope that would help you coredns/helm#173 |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
Hi, I'm creating this PR to solve the readiness issue. |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
We are using
node-local-dns
with a CiliumLocal Redirect Policy as described here so we neednode-local-dns
to be able to become Not Ready once it receives aSIGTERM
so time is given for the Endpoint to stop receiving traffic and for requests to be directed tokube-dns
instead, before the pod is fully terminated. This will allow graceful rollouts of thenode-local-dns
daemonset.In #485 it was requested the
ready
plugin be added tonode-local-dns
but it was closed on the basis it wouldn't serve any purpose as it would behave the same way as thehealth
plugin. However I believe that's not the case and theready
plugin would allownode-local-dns
to gracefully shutdown, since once it receives aSIGTERM
the/ready
endpoint would start returning 503 which can be used in a Readiness Probe on the pod to determine the pod is shutting down and give time for the Endpoint to be removed when used with thehealth
pluginslameduck
setting.So I am again requesting the
ready
plugin be added to the plugins list here.The text was updated successfully, but these errors were encountered: