-
Notifications
You must be signed in to change notification settings - Fork 38.9k
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 logic of adding default DS pod tolerations #67036
update logic of adding default DS pod tolerations #67036
Conversation
/assign @mikedanese @k82cn |
/ok-to-test |
/retest |
Friendly ping @k82cn @mikedanese :) |
ee52c61
to
e06b2f5
Compare
@janetkuo @mikedanese Could you kindly review this PR? It's a (small) behavior fix on default daemonset pods:
|
/lgtm |
ping @mikedanese , PTAL :) |
/retest |
1 similar comment
/retest |
Friendly ping @mikedanese @janetkuo PTAL |
@janetkuo , I think we'd better to get this merged in 1.12 :) |
e06b2f5
to
7abfb32
Compare
Code rebased. @k82cn PTAL. |
@@ -120,11 +108,7 @@ func AddOrUpdateDaemonPodTolerations(spec *v1.PodSpec, isCritical bool) { | |||
func CreatePodTemplate(ns string, template v1.PodTemplateSpec, generation *int64, hash string) v1.PodTemplateSpec { |
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.
also remove ns string
here :)
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.
I did see a reminder in one of the comments, but forgot...
Done.
7abfb32
to
5f38a82
Compare
/retest |
2 similar comments
/retest |
/retest |
LGTM, for 1.13, please help to rebase :) |
5f38a82
to
1d1c8bb
Compare
@mikedanese @janetkuo could you kindly take a look at the Daemonset part? |
Friendly ping @mikedanese @janetkuo :) |
@Huang-Wei , would you help to rebase this PR? I'd like to get this merged in 1.14 :) |
- update DS pod default tolerations: add PIDPressure, remove OutOfDisk - remove useless tolerations testcases
1d1c8bb
to
ad6e361
Compare
@k82cn rebased. PTAL. |
/retest |
/lgtm We're going to remove |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Huang-Wei, k82cn The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
Which issue(s) this PR fixes:
ScheduleDaemonSetPods
is enabled #66348 (comment).Special notes for your reviewer:
Correct the behavior of daemonset controller as "out-of-disk" toleration doesn't fit any more.
Release note:
/sig scheduling
/sig apps