From 64fca6bda7d36b9cc1667aa317896b540d4dcf8c Mon Sep 17 00:00:00 2001 From: Huy Pham Date: Tue, 30 Mar 2021 09:56:23 -0700 Subject: [PATCH] doc: remove the description `socat` is required This is no longer true as of `containerd` v1.4.0 because [0] removes `socat` dependency for port-forwarding. For lower versions of `containerd`, the error message is descriptive i.e. `"socat": executable file not found in $PATH`. [0] https://github.com/containerd/cri/pull/1470 --- staging/src/k8s.io/kubectl/pkg/cmd/portforward/portforward.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/staging/src/k8s.io/kubectl/pkg/cmd/portforward/portforward.go b/staging/src/k8s.io/kubectl/pkg/cmd/portforward/portforward.go index a6c1bac8c01..71922bb9d23 100644 --- a/staging/src/k8s.io/kubectl/pkg/cmd/portforward/portforward.go +++ b/staging/src/k8s.io/kubectl/pkg/cmd/portforward/portforward.go @@ -61,7 +61,7 @@ type PortForwardOptions struct { var ( portforwardLong = templates.LongDesc(i18n.T(` - Forward one or more local ports to a pod. This command requires the node to have 'socat' installed. + Forward one or more local ports to a pod. Use resource type/name such as deployment/mydeployment to select a pod. Resource type defaults to 'pod' if omitted.