{"push_id":1185386238,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"a67c0ff8a1e00d2c7fbcb37478ed75f0b3f55056","before":"8cabbcbdcfa2bdf17c9ddc1db6754c862d6e08a2","commits":[{"sha":"b5cef55af5176c91c8b59b67999de959dfa405c0","author":{"name":"Random-Liu","email":"76dec8bcf19c97957369f3139bf49570363c8165@gmail.com"},"message":"fix pull image test flake","distinct":true,"url":"https://api.github.com/repos/kubernetes/kubernetes/commits/b5cef55af5176c91c8b59b67999de959dfa405c0"},{"sha":"a67c0ff8a1e00d2c7fbcb37478ed75f0b3f55056","author":{"name":"k8s-merge-robot","email":"90609b3a0723d5b4f9968c7035bcd6629ebfd177@gmail.com"},"message":"Merge pull request #28323 from Random-Liu/fix-node-confomance-test\n\nAutomatic merge from submit-queue\n\nFix node confomance test\n\nFixes https://github.com/kubernetes/kubernetes/issues/28255, https://github.com/kubernetes/kubernetes/issues/28250, https://github.com/kubernetes/kubernetes/issues/28341.\r\n\r\nThe main reason of the flake is that in the failed test expects the `PodPhase` to keep `Pending`. It did `Eventually` check and `Consistently` check for 5 seconds. However, the default `PodPhase` is `Pending`, when the check passes, the `PodStatus` could still be in default state.\r\n\r\nAfter that, the test expects the container status to be `Waiting`, which may not be the case, because the default `ContainerStatuses` is empty, and the pod could still be in the default state.\r\n\r\nThis PR changes the test to ensure `ContainerStatuses` first and then check the `PodPhase` after that.\r\n\r\nMark P1 because the test fails relatively frequently and does block some PRs.\r\n\r\n@pwittrock \r\n\r\n/cc @liangchenye @ncdc \r\n[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/.github/PULL_REQUEST_TEMPLATE.md?pixel)]()","distinct":true,"url":"https://api.github.com/repos/kubernetes/kubernetes/commits/a67c0ff8a1e00d2c7fbcb37478ed75f0b3f55056"}]}
{"actor":{"display_login":"k8s-merge-robot"}}