-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
Update preemption document with the new improvements added in 1.9 #6505
Conversation
cc/ @davidopp |
Deploy preview ready! Built with commit 9f01370 https://deploy-preview-6505--kubernetes-io-vnext-staging.netlify.com |
pending Pod possible. In a future Kubernetes release, priority will also affect | ||
out-of-resource eviction ordering on the Node. | ||
pending Pod possible. In Kubernetes 1.9 and later, Priority also affects scheduling | ||
order of pods and out-of-resource eviction ordering on the Node. | ||
|
||
**Note:** Preemption does not respect PodDisruptionBudget; see | ||
[the limitations section](#poddisruptionbudget-is-not-supported) for more details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this limitation still valid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay, there is a new subsection on this topic. We can remove the note here then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right. I will delete this note.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Deploy preview ready! Built with commit 8bbbcca https://deploy-preview-6505--kubernetes-io-master-staging.netlify.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/assign @steveperry-53 |
ping! Can we have this doc merged sooner? Some of our users are waiting for the doc to be merged to try the feature. |
/cc @zacharysarah |
@bsalamat 👋 The docs are planned to go live this afternoon, in a few hours. |
c4ea942
to
989acbf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Changing base to |
…henopis-user-journeys * 'master' of https://github.com/kubernetes/website: (92 commits) Update partner-script.js (#6570) Update redirects for kubectl commands. (#6706) Improve redirection of 1.8 ref docs. (#6705) Modify Service Catalog snippet long description add prepend option to glossary definition tag Mark CPU manager feature as beta for v1.10. Balaji, Seth, Connor as CPU manager doc approvers. Fix broken link (#6691) Clarify that StatefulSet is stable in 1.9 (#6697) Update coredns.md (#6698) Update preemption document with the new improvements added in 1.9 (#6505) v1.9 patch (#6690) Remove 1.9 info from the PR template Fix release 1.9 (#6686) Release 1.9 (#5978) [maintenance] December link fixes (#6680) Update audit.md (#6381) Fixed typo concepts/policy/pod-security-policy.md: fix broken links. Fixed a typo `illgal` ...
Update preemption document with the new improvements added in 1.9.
This change is