Merge pull request #650 from ykulazhenkov/pr-remove-stdin-from-pod-exec

Experiment: Remove stdin from the ExecCommand call in tests
k8snetworkplumbingwg · Mar 11, 2024 · 5bc82da · 5bc82da
2 parents 07b5fd3 + 4b5087a
commit 5bc82da
Showing 1 changed file with 0 additions and 3 deletions.
diff --git a/test/util/pod/pod.go b/test/util/pod/pod.go
@@ -4,7 +4,6 @@ import (
 	"bytes"
 	"context"
 	"io"
-	"os"
 	"strings"
 	"time"
 
@@ -116,7 +115,6 @@ func ExecCommand(cs *testclient.ClientSet, pod *corev1.Pod, command ...string) (
 		VersionedParams(&corev1.PodExecOptions{
 			Container: pod.Spec.Containers[0].Name,
 			Command:   command,
-			Stdin:     true,
 			Stdout:    true,
 			Stderr:    true,
 			TTY:       true,
@@ -128,7 +126,6 @@ func ExecCommand(cs *testclient.ClientSet, pod *corev1.Pod, command ...string) (
 	}
 
 	err = exec.Stream(remotecommand.StreamOptions{
-		Stdin:  os.Stdin,
 		Stdout: &buf,
 		Stderr: &errbuf,
 		Tty:    true,