diff options
| author | Valentin Rothberg <rothberg@redhat.com> | 2021-06-21 14:11:27 +0200 |
|---|---|---|
| committer | Valentin Rothberg <rothberg@redhat.com> | 2021-06-22 13:18:04 +0200 |
| commit | 4d9a9149dd127cff417766512ab902e0a7e009a7 (patch) | |
| tree | c9f5bc856a9a90d3764d5e10064dc4881b0513a5 /libpod/container_log.go | |
| parent | 017116242dd4a12f3c97fa7449348b5a6f0d1622 (diff) | |
| download | podman-4d9a9149dd127cff417766512ab902e0a7e009a7.tar.gz podman-4d9a9149dd127cff417766512ab902e0a7e009a7.tar.bz2 podman-4d9a9149dd127cff417766512ab902e0a7e009a7.zip | |
logs: k8s-file: fix spurious error logs
Fix the suprious "Error: nil" messages. Also add some more context to
logged error messages which makes error sources more obvious.
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
Diffstat (limited to 'libpod/container_log.go')
| -rw-r--r-- | libpod/container_log.go | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/libpod/container_log.go b/libpod/container_log.go index a30e4f5cc..8bbb7ebe8 100644 --- a/libpod/container_log.go +++ b/libpod/container_log.go @@ -74,7 +74,7 @@ func (c *Container) readFromLogFile(ctx context.Context, options *logs.LogOption } nll, err := logs.NewLogLine(line.Text) if err != nil { - logrus.Error(err) + logrus.Errorf("Error getting new log line: %v", err) continue } if nll.Partial() { @@ -100,10 +100,10 @@ func (c *Container) readFromLogFile(ctx context.Context, options *logs.LogOption // read the file until EOF. tailError := t.StopAtEOF() if tailError != nil && fmt.Sprintf("%v", tailError) != "tail: stop at eof" { - logrus.Error(tailError) + logrus.Errorf("Error stopping logger: %v", tailError) } - if errors.Cause(err) != define.ErrNoSuchCtr { - logrus.Error(err) + if err != nil && errors.Cause(err) != define.ErrNoSuchCtr { + logrus.Errorf("Error getting container state: %v", err) } return nil } @@ -126,7 +126,7 @@ func (c *Container) readFromLogFile(ctx context.Context, options *logs.LogOption <-eventChannel tailError := t.StopAtEOF() if tailError != nil && fmt.Sprintf("%v", tailError) != "tail: stop at eof" { - logrus.Error(tailError) + logrus.Errorf("Error stopping logger: %v", tailError) } }() } |