-
Notifications
You must be signed in to change notification settings - Fork 65
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
Consider draining and rebooting the node when deploying the Operator #64
Comments
@fidencio agree that using draining and node reboot is safer. We will need to improve the pre-install and post-uninstall flow to incorporate this. |
We might use something like this - https://github.com/weaveworks/kured |
Not only pre-install / post-uninstall flow, but also the normal flow where we add / remove a new runtimeclass to the containerd configuration. |
@fidencio is this issue still relevant or can be closed? |
This is still relevant and should be addressed in the short-term future. I'm labelling it to be part of the 0.3.0, but we'll re-asses when that time comes. |
Cool, thanks |
Sasha mentioned that restarting basic services in the node may lead to Kubernetes failures that would be avoided by draining and rebooting the node.
We should consider doing that as part of the deployment.
The text was updated successfully, but these errors were encountered: