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.

Sign up for GitHub

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

Jump to bottom

Prevent eviction of pods weave addon with priorityClassName #7873

Closed
jochen42 opened this issue Nov 4, 2019 · 0 comments · Fixed by #7874
Closed

Prevent eviction of pods weave addon with priorityClassName #7873

jochen42 opened this issue Nov 4, 2019 · 0 comments · Fixed by #7874

Comments

@jochen42
Copy link
Contributor

jochen42 commented Nov 4, 2019

1. Problem description

Weave Pods should never be evicted because it would break the node networking and impacts the whole cluster.

2. Solution from weaveworks

daemonSet:
  spec:
    template:
      spec:
        priorityClassName: system-node-critical
@jochen42 jochen42 mentioned this issue Nov 4, 2019
Merged
k8s-ci-robot added a commit that referenced this issue Nov 5, 2019
@k8s-ci-robot
Merge pull request #7874 from jochen42/feature/#7873-weave-pods-node-…
b1fba5c 
…critical

mark weavenet-pod as system-critical
ReillyTevera pushed a commit to ReillyTevera/kops that referenced this issue Dec 11, 2019
@k8s-ci-robot
Merge pull request kubernetes#7874 from jochen42/feature/kubernetes#7873
87a3989 
-weave-pods-node-critical

mark weavenet-pod as system-critical
ReillyTevera pushed a commit to ReillyTevera/kops that referenced this issue Dec 26, 2019
@k8s-ci-robot
Merge pull request kubernetes#7874 from jochen42/feature/kubernetes#7873
4d8f982 
-weave-pods-node-critical

mark weavenet-pod as system-critical
ReillyTevera pushed a commit to ReillyTevera/kops that referenced this issue Dec 30, 2019
@k8s-ci-robot
Merge pull request kubernetes#7874 from jochen42/feature/kubernetes#7873
114323b 
-weave-pods-node-critical

mark weavenet-pod as system-critical
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

mark weavenet-pod as system-critical
1 participant
@jochen42