summaryrefslogtreecommitdiff
path: root/libpod/container_log.go
diff options
context:
space:
mode:
authorjgallucci32 <john.gallucci.iv@gmail.com>2020-06-20 09:45:18 -0700
committerjgallucci32 <john.gallucci.iv@gmail.com>2020-06-20 09:48:55 -0700
commit78dd4f2ecb9f85fda87e581b9b2800618223154f (patch)
tree4d37a070bb83f856b0dd14e9e5a5c6c4aede5700 /libpod/container_log.go
parentf403aa3baa23a64e7d6bf3fcbc815644d95b768d (diff)
downloadpodman-78dd4f2ecb9f85fda87e581b9b2800618223154f.tar.gz
podman-78dd4f2ecb9f85fda87e581b9b2800618223154f.tar.bz2
podman-78dd4f2ecb9f85fda87e581b9b2800618223154f.zip
Stop following logs using timers
This incorporates code from PR #6591 and #6614 but does not use event channels to detect container state and rather uses timers with a defined wait duration before calling t.StopAtEOF() to ensure the last log entry is output before a container exits. The polling interval is set to 250 milliseconds based on polling interval defined in hpcloud/tail here: https://github.com/hpcloud/tail/blob/v1.0.0/watch/polling.go#L117 Co-authored-by: Qi Wang <qiwan@redhat.com> Signed-off-by: jgallucci32 <john.gallucci.iv@gmail.com>
Diffstat (limited to 'libpod/container_log.go')
-rw-r--r--libpod/container_log.go30
1 files changed, 30 insertions, 0 deletions
diff --git a/libpod/container_log.go b/libpod/container_log.go
index 071882bc2..9a2f8aa2f 100644
--- a/libpod/container_log.go
+++ b/libpod/container_log.go
@@ -1,7 +1,9 @@
package libpod
import (
+ "fmt"
"os"
+ "time"
"github.com/containers/libpod/libpod/define"
"github.com/containers/libpod/libpod/logs"
@@ -81,5 +83,33 @@ func (c *Container) readFromLogFile(options *logs.LogOptions, logChannel chan *l
}
options.WaitGroup.Done()
}()
+ // Check if container is still running or paused
+ if options.Follow {
+ go func() {
+ for {
+ state, err := c.State()
+ if err != nil {
+ time.Sleep(250 * time.Millisecond)
+ tailError := t.StopAtEOF()
+ if tailError != nil && fmt.Sprintf("%v", tailError) != "tail: stop at eof" {
+ logrus.Error(tailError)
+ }
+ if errors.Cause(err) != define.ErrNoSuchCtr {
+ logrus.Error(err)
+ }
+ break
+ }
+ if state != define.ContainerStateRunning && state != define.ContainerStatePaused {
+ time.Sleep(250 * time.Millisecond)
+ tailError := t.StopAtEOF()
+ if tailError != nil && fmt.Sprintf("%v", tailError) != "tail: stop at eof" {
+ logrus.Error(tailError)
+ }
+ break
+ }
+ time.Sleep(250 * time.Millisecond)
+ }
+ }()
+ }
return nil
}