summaryrefslogtreecommitdiff
path: root/libpod/healthcheck.go
diff options
context:
space:
mode:
authorPeter Hunt <pehunt@redhat.com>2019-10-31 10:41:26 -0400
committerPeter Hunt <pehunt@redhat.com>2019-10-31 11:20:12 -0400
commit1df4dba0a0c335ad68b1c2202b8099cd87aa3d94 (patch)
treeb19dfe0cda86f5248bd6627725a7e86e5381c61d /libpod/healthcheck.go
parent9ba8dae0bfa5a5e894cf80e2ed114f6ca4bceb60 (diff)
downloadpodman-1df4dba0a0c335ad68b1c2202b8099cd87aa3d94.tar.gz
podman-1df4dba0a0c335ad68b1c2202b8099cd87aa3d94.tar.bz2
podman-1df4dba0a0c335ad68b1c2202b8099cd87aa3d94.zip
Switch to bufio Reader for exec streams
There were many situations that made exec act funky with input. pipes didn't work as expected, as well as sending input before the shell opened. Thinking about it, it seemed as though the issues were because of how os.Stdin buffers (it doesn't). Dropping this input had some weird consequences. Instead, read from os.Stdin as bufio.Reader, allowing the input to buffer before passing it to the container. Signed-off-by: Peter Hunt <pehunt@redhat.com>
Diffstat (limited to 'libpod/healthcheck.go')
-rw-r--r--libpod/healthcheck.go4
1 files changed, 3 insertions, 1 deletions
diff --git a/libpod/healthcheck.go b/libpod/healthcheck.go
index 68ffc2349..e9c950713 100644
--- a/libpod/healthcheck.go
+++ b/libpod/healthcheck.go
@@ -133,7 +133,9 @@ func (c *Container) runHealthCheck() (HealthCheckStatus, error) {
streams := new(AttachStreams)
streams.OutputStream = hcw
streams.ErrorStream = hcw
- streams.InputStream = os.Stdin
+
+ streams.InputStream = bufio.NewReader(os.Stdin)
+
streams.AttachOutput = true
streams.AttachError = true
streams.AttachInput = true