github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-21 02:41:25 +00:00
Code Issues Projects Releases Wiki Activity

kubeadm: only taint CP nodes when the legacy "master" taint is present

Browse Source 
During upgrade when a CP node is missing the old / legacy "master"
taint, assume the user has manually removed it to allow
workloads to schedule.

In such cases do not re-taint the node with the new "control-plane"
taint.
This commit is contained in:
Lubomir I. Ivanov 2022-05-05 23:39:23 +03:00
parent 71df3e819b
commit b1f3034051
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
cmd/kubeadm/app/phases/upgrade/postupgrade.go
View File

@ -253,16 +253,20 @@ func AddNewControlPlaneTaint(client clientset.Interface) error {
}
for _, n := range nodes.Items {
// Check if the node has the taint already and skip it if so
hasTaint := false
// Check if the node has the old / new taints
hasOldTaint := false
hasNewTaint := false
for _, t := range n.Spec.Taints {
if t.String() == kubeadmconstants.ControlPlaneTaint.String() {
hasTaint = true
break
switch t.String() {
case kubeadmconstants.OldControlPlaneTaint.String():
hasOldTaint = true
case kubeadmconstants.ControlPlaneTaint.String():
hasNewTaint = true
}
}
// If the node does not have the taint, patch it
if !hasTaint {
// If the old taint is present and the new taint is missing, patch the node with the new taint.
// When the old taint is missing, assume the user has manually untainted the node and take no action.
if !hasNewTaint && hasOldTaint {
err = apiclient.PatchNode(client, n.Name, func(n *v1.Node) {
n.Spec.Taints = append(n.Spec.Taints, kubeadmconstants.ControlPlaneTaint)
})