-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
kubelet: clean up tests #9346
kubelet: clean up tests #9346
Conversation
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
bd5946f
to
b757aa1
Compare
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
Sorry for the delay. I was out last week and most of this week too. @yifan-gu, thanks for fixing the tests! The PR looks good to me, but needs to be rebased. |
4a2f515
to
5d027b5
Compare
@yujuhong Rebased. No worries, take your time :) |
Uh....you need to rebase again :-( |
5d027b5
to
9d2830f
Compare
Done @yujuhong |
LGTM. |
@k8s-bot ok to test |
GCE e2e build/test failed for commit 9d2830fb53adf0852db9370d034ae41ea6142fda. |
The e2e failures:
|
@yujuhong No idea what causes this :( This PR barely touches anything other than the tests... |
I doubt that it was caused by your PR, but e2e seemed to be passing for other PRs. I will take a look later to see what's going on... |
@k8s-bot ok to test |
GCE e2e build/test failed for commit 9d2830fb53adf0852db9370d034ae41ea6142fda. |
Also refactor TestPortForward() to be neutral to container runtime.
Update the fakeOptionGenerator to create the pod container directory, so that fake docker client can see and record the termination log path.
… functions. This cleans up the kubelet tests. Almost no docker specified tests now.
9d2830f
to
40e46bb
Compare
GCE e2e build/test passed for commit 40e46bb. |
@yifan-gu, did you change anything in your commits?...e2e passed. |
@yujuhong No, just rebased. Think there is some flakiness in the tests. |
So they are just flaky :( |
Move all fake docker related tests to dockertools. Remov unused functions.
/cc @yujuhong @vmarmol @dchen1107