Merge pull request #105599 from jonyhy96/fix-pod-workers-test

fix: pod workers test
This commit is contained in:
Kubernetes Prow Robot 2022-04-02 06:50:09 -07:00 committed by GitHub
commit e04a4e1c5b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -398,7 +398,7 @@ func TestUpdatePod(t *testing.T) {
func TestUpdatePodWithTerminatedPod(t *testing.T) { func TestUpdatePodWithTerminatedPod(t *testing.T) {
podWorkers, _ := createPodWorkers() podWorkers, _ := createPodWorkers()
terminatedPod := newPodWithPhase("0000-0000-0000", "done-pod", v1.PodSucceeded) terminatedPod := newPodWithPhase("0000-0000-0000", "done-pod", v1.PodSucceeded)
runningPod := &kubecontainer.Pod{ID: "0000-0000-0001", Name: "done-pod"} orphanedPod := &kubecontainer.Pod{ID: "0000-0000-0001", Name: "orphaned-pod"}
pod := newPod("0000-0000-0002", "running-pod") pod := newPod("0000-0000-0002", "running-pod")
podWorkers.UpdatePod(UpdatePodOptions{ podWorkers.UpdatePod(UpdatePodOptions{
@ -411,17 +411,17 @@ func TestUpdatePodWithTerminatedPod(t *testing.T) {
}) })
podWorkers.UpdatePod(UpdatePodOptions{ podWorkers.UpdatePod(UpdatePodOptions{
UpdateType: kubetypes.SyncPodKill, UpdateType: kubetypes.SyncPodKill,
RunningPod: runningPod, RunningPod: orphanedPod,
}) })
drainAllWorkers(podWorkers)
if podWorkers.IsPodKnownTerminated(pod.UID) == true { if podWorkers.IsPodKnownTerminated(pod.UID) == true {
t.Errorf("podWorker state should not be terminated") t.Errorf("podWorker state should not be terminated")
} }
if podWorkers.IsPodKnownTerminated(terminatedPod.UID) == false { if podWorkers.IsPodKnownTerminated(terminatedPod.UID) == false {
t.Errorf("podWorker state should be terminated") t.Errorf("podWorker state should be terminated")
} }
if podWorkers.IsPodKnownTerminated(runningPod.ID) == true { if podWorkers.IsPodKnownTerminated(orphanedPod.ID) == false {
t.Errorf("podWorker state should not be marked terminated for a running pod") t.Errorf("podWorker state should be terminated for orphaned pod")
} }
} }