@k8s-bot e2e test this issue: #27335

Fixed rebase again > <

@resouer too bad you'll need to rebase again :-(

Let's hope this is the last time since the queue is now open to all PRs.

ping?

Ah, sorry for missing this notification, will update this soon

On Tue, Jul 12, 2016 at 12:32 AM, Yu-Ju Hong notifications@github.com
wrote:

ping?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#26332 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABn3lpO-sZEnFQ5VUixfyql9BCTxwjapks5qUnAGgaJpZM4InKVd
.

Failed unit test ..

@resouer

ok      k8s.io/kubernetes/pkg/kubectl/cmd/config    2.048s
ok      k8s.io/kubernetes/pkg/kubectl/cmd/util  2.667s
ok      k8s.io/kubernetes/pkg/kubectl/cmd/util/editor   1.024s
ok      k8s.io/kubernetes/pkg/kubectl/resource  1.391s
FAIL    k8s.io/kubernetes/pkg/kubelet [build failed]
ok      k8s.io/kubernetes/pkg/kubelet/client    1.173s
ok      k8s.io/kubernetes/pkg/kubelet/cm    2.021s
ok      k8s.io/kubernetes/pkg/kubelet/config    1.341s
ok      k8s.io/kubernetes/pkg/kubelet/container 1.123s

The change looks good to me. Once you fix the build failure, I'll apply the label. Thanks!

i can not re-produce this failure by doing make test in local. really don't know why ...

It's an easy fix :-)
From the build.log:

# k8s.io/kubernetes/pkg/kubelet
pkg/kubelet/kubelet_test.go:64: imported and not used: "k8s.io/kubernetes/pkg/util"

Aha, pkg/util is used in my commit, that's what I am struggling, but actually it has been deleted in a latest commit, that's why CI will complain after it do rebase but I can not reproduce it locally ...

GCE e2e build/test passed for commit cb14b35.

• Test Results
• Build Log
• Test Artifacts
• Internal Jenkins Results

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

GCE e2e build/test passed for commit cb14b35.

• Test Results
• Build Log
• Test Artifacts
• Internal Jenkins Results

Automatic merge from submit-queue