-
Notifications
You must be signed in to change notification settings - Fork 108
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-group] Fix remove taints patch from master NG #2809
Conversation
Signed-off-by: Yuriy Losev <yuriy.losev@flant.com>
🟢 Workflow details🟢 |
Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com> Signed-off-by: Yuriy Losev <yuriy.losev@flant.com>
better
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix e2e (run (better) or skip)
E2E was passed. We had just comments changes after the test |
Signed-off-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com>
Cherry pick PR 2814 to the branch release-1.37 successful! |
Signed-off-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com> Signed-off-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com>
/backport 1.38 |
Signed-off-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com>
Cherry pick PR 2816 to the branch release-1.38 successful! |
Signed-off-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com> Signed-off-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Yuriy Losev <yuriy.losev@flant.com> Co-authored-by: Ivan Mikheykin <ivan.mikheykin@flant.com>
Signed-off-by: Yuriy Losev yuriy.losev@flant.com
Description
Remove "master" taint if "control-plane" taint is removed
Why do we need it, and what problem does it solve?
When we are bootstraping a new single master cluster, we use command
kubectl patch nodegroup master --type json -p '[{"op": "remove", "path": "/spec/nodeTemplate/taints"}]'
for removing node taints.Meanwhile our NG master only has:
and if kubernetes version < 1.24 - kubeadm sets both taints for the master node during the bootstrap process:
After patching the master NG only
control-plane
taint will be removed explicitly. We also have to removemaster
taint ifcontrol-plane
taint was removed, otherwise cluster will not be bootstrapped.What is the expected result?
Checklist
Changelog entries