From 2afc78bac870286aa4f39069fb5ab3fa13a134ed Mon Sep 17 00:00:00 2001 From: wujunwei Date: Thu, 17 Nov 2022 09:39:30 +0800 Subject: [PATCH] kubectl: remove unnecessary TODO annotations --- staging/src/k8s.io/kubectl/pkg/cmd/cp/cp.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/staging/src/k8s.io/kubectl/pkg/cmd/cp/cp.go b/staging/src/k8s.io/kubectl/pkg/cmd/cp/cp.go index 9a90d133ade..bc50f770f98 100644 --- a/staging/src/k8s.io/kubectl/pkg/cmd/cp/cp.go +++ b/staging/src/k8s.io/kubectl/pkg/cmd/cp/cp.go @@ -304,7 +304,6 @@ func (o *CopyOptions) copyToPod(src, dest fileSpec, options *exec.ExecOptions) e }(srcFile, destFile, writer) var cmdArr []string - // TODO: Improve error messages by first testing if 'tar' is present in the container? if o.NoPreserve { cmdArr = []string{"tar", "--no-same-permissions", "--no-same-owner", "-xmf", "-"} } else { @@ -373,7 +372,6 @@ func (t *TarPipe) initReadFrom(n uint64) { PodName: t.src.PodName, }, - // TODO: Improve error messages by first testing if 'tar' is present in the container? Command: []string{"tar", "cf", "-", t.src.File.String()}, Executor: &exec.DefaultRemoteExecutor{}, }