flannel-io / flannel

flannel is a network fabric for containers, designed for Kubernetes
Apache License 2.0
8.74k stars 2.87k forks source link

taint node-role.kubernetes.io/master:NoSchedule not tolerated. #1921

Closed fungaren closed 6 months ago

fungaren commented 6 months ago

Expected Behavior

The default taint for master nodes node-role.kubernetes.io/master:NoSchedule is not tolerated, and it makes flannel unable to be scheduled.

Current Behavior

This commit introduced this bug: https://github.com/flannel-io/flannel/pull/1907/files

Possible Solution

Add this to values.yaml:

  - key: "node-role.kubernetes.io/master"
    effect: "NoSchedule"

Or, just like the previous behavior, tolerate all taints.

Steps to Reproduce (for bugs)

1. 2. 3. 4.

Context

Your Environment

manuelbuil commented 6 months ago

This is a deprecated taint, it was replaced by "node-role.kubernetes.io/control-plane" several releases ago: https://kubernetes.io/docs/reference/labels-annotations-taints/#node-role-kubernetes-io-master-taint

fungaren commented 6 months ago

It is deprecated though, tolerate it will not produce any overhead. Keep it compatible is better, isn't it?

rbrtbnfgl commented 6 months ago

I think you can have the same behaviour if you change the toleration value through Helm.