Backport: [node-group] Fix remove taints patch from master NG

[deckhouse-BOaTswain]

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:

    taints:
    - effect: NoSchedule
      key: node-role.kubernetes.io/control-plane

and if kubernetes version < 1.24 - kubeadm sets both taints for the master node during the bootstrap process:

  taints:
  - effect: NoSchedule
    key: node-role.kubernetes.io/control-plane
  - effect: NoSchedule
    key: node-role.kubernetes.io/master

After patching the master NG only control-plane taint will be removed explicitly. We also have to remove master taint if control-plane taint was removed, otherwise cluster will not be bootstrapped.

What is the expected result?

Checklist

• The code is covered by unit tests.
• e2e tests passed.
• Documentation updated according to the changes.
• Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: candi
type: fix
summary: Remove `master` taint if `control-plane` taint was removed in a single node installation.