Merge pull request #84562 from yutedz/drain-filter

Only put un-filtered pod in podDeleteList
This commit is contained in:
Kubernetes Prow Robot 2019-11-08 18:43:52 -08:00 committed by GitHub
commit 1aece964da
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 6 deletions

View File

@ -867,7 +867,7 @@ func TestDrain(t *testing.T) {
switch { switch {
case recovered != nil && !sawFatal: case recovered != nil && !sawFatal:
t.Fatalf("got panic: %v", recovered) t.Fatalf("got panic: %v", recovered)
case test.expectFatal && !sawFatal: case test.expectDelete && test.expectFatal && !sawFatal:
t.Fatalf("%s: unexpected non-error when using %s", test.description, currMethod) t.Fatalf("%s: unexpected non-error when using %s", test.description, currMethod)
case !test.expectFatal && sawFatal: case !test.expectFatal && sawFatal:
t.Fatalf("%s: unexpected error when using %s: %s", test.description, currMethod, fatalMsg) t.Fatalf("%s: unexpected error when using %s: %s", test.description, currMethod, fatalMsg)
@ -903,7 +903,7 @@ func TestDrain(t *testing.T) {
t.Fatalf("%s: same pod deleted %d times and evicted %d times", test.description, deletions, evictions) t.Fatalf("%s: same pod deleted %d times and evicted %d times", test.description, deletions, evictions)
} }
if len(test.expectWarning) > 0 { if test.expectDelete && len(test.expectWarning) > 0 {
if len(errBuf.String()) == 0 { if len(errBuf.String()) == 0 {
t.Fatalf("%s: expected warning, but found no stderr output", test.description) t.Fatalf("%s: expected warning, but found no stderr output", test.description)
} }

View File

@ -155,10 +155,12 @@ func (d *Helper) GetPodsForDeletion(nodeName string) (*podDeleteList, []error) {
break break
} }
} }
pods = append(pods, podDelete{ if status.delete {
pod: pod, pods = append(pods, podDelete{
status: status, pod: pod,
}) status: status,
})
}
} }
list := &podDeleteList{items: pods} list := &podDeleteList{items: pods}