Skip to content
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

CNI: Weave tolerations do not support "NoExecute" effects #6158

Closed
fydrah opened this issue May 18, 2020 · 0 comments · Fixed by #6159
Closed

CNI: Weave tolerations do not support "NoExecute" effects #6158

fydrah opened this issue May 18, 2020 · 0 comments · Fixed by #6159
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@fydrah
Copy link
Contributor

fydrah commented May 18, 2020

Hi,

Environment:

  • Cloud provider or hardware configuration: N/A

  • OS (printf "$(uname -srm)\n$(cat /etc/os-release)\n"): N/A

  • Version of Ansible (ansible --version): N/A

  • Version of Python (python --version): N/A

Kubespray version (commit) (git rev-parse --short HEAD): 437189c2

Network plugin used: weave

Full inventory with variables (ansible -i inventory/sample/inventory.ini all -m debug -a "var=hostvars[inventory_hostname]"): N/A

kube_network_plugin: weave

Command used to invoke ansible:

ansible-playbook -i $INVENTORY cluster.yml

Output of ansible run: N/A

Anything else do we need to know:

If nodes have taints with NoExecute effect, weave DaemonSet is not configured to be deployed on them.

  • The following PR changed all CNI manifests to support any toleration effects
  • But this PR overwrites toleration changes

Weave network plugin manifest should be configured to support any effects.

# roles/network_plugin/weave/templates/weave-net.yml.j2#226
[...]
          tolerations:
            - operator: Exists
            # Mark pod as critical for rescheduling (Will have no effect starting with kubernetes 1.12)
            - key: CriticalAddonsOnly
              operator: "Exists"
[...]

I will push the associated patch soon.

@fydrah fydrah added the kind/bug Categorizes issue or PR as related to a bug. label May 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant