Hi @louyihua. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with @k8s-bot ok to test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

Once you've signed, please reply here (e.g. "I signed it!") and we'll verify. Thanks.

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.

This change is 

[APPROVALNOTIFIER] This PR is NOT APPROVED

The following people have approved this PR: louyihua

Needs approval from an approver in each of these OWNERS Files:

• pkg/api/OWNERS
• pkg/kubelet/OWNERS

We suggest the following people:
cc @Random-Liu @smarterclayton
You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-bot tell me a joke

@louyihua: What do you call a fake noodle? An impasta.

@smarterclayton
This is a required patch to solve the openshift BUG 1405440 & 1430729.

@kubernetes/sig-network-pr-reviews ptal

We generally recommend keeping real code changes and autogenerated changes in separate commit for easier reviewing.

This is an API change so cc @kubernetes/api-reviewers.

Assigning to @thockin to triage since this a network related API change.

@k8s-bot ok to test

Always fail on the aws e2e with strange error :(

I would also add:

diff --git a/pkg/kubectl/describe.go b/pkg/kubectl/describe.go
index 0359b9c..c63654d 100644
--- a/pkg/kubectl/describe.go
+++ b/pkg/kubectl/describe.go
@@ -1123,7 +1123,7 @@ func DescribeProbe(probe *api.Probe) string {
                url.Path = probe.HTTPGet.Path
                return fmt.Sprintf("http-get %s %s", url.String(), attrs)
        case probe.TCPSocket != nil:
-               return fmt.Sprintf("tcp-socket :%s %s", probe.TCPSocket.Port.String(), attrs)
+               return fmt.Sprintf("tcp-socket %s:%s %s", probe.TCPSocket.Host, probe.TCPSocket.Port.String(), attrs)
        }
        return fmt.Sprintf("unknown %s", attrs)
 }

@jwendell
Thanks! Have added this to commit.

@louyihua no problem. I was also working on that feature, for the last 3 days at least, and just saw this PR. The only difference to my work was that chunk above.

@thockin
All tests are passed now.

Automatic merge from submit-queue (batch tested with PRs 42998, 42902, 42959, 43020, 42948)