From c3910e7331c5d92f1c59a019734d362921f6cce0 Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Tue, 27 Jul 2021 13:08:26 -0700 Subject: [PATCH] libct/logs: parse log level implicitly There's no need to call logrus.ParseLevel as logrus.Level already implements encoding.TextUnmarshaler interface. Signed-off-by: Kir Kolyshkin --- libcontainer/logs/logs.go | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/libcontainer/logs/logs.go b/libcontainer/logs/logs.go index e625abc74..d9a6991be 100644 --- a/libcontainer/logs/logs.go +++ b/libcontainer/logs/logs.go @@ -34,18 +34,13 @@ func processEntry(text []byte) { } var jl struct { - Level string `json:"level"` - Msg string `json:"msg"` + Level logrus.Level `json:"level"` + Msg string `json:"msg"` } if err := json.Unmarshal(text, &jl); err != nil { logrus.Errorf("failed to decode %q to json: %v", text, err) return } - lvl, err := logrus.ParseLevel(jl.Level) - if err != nil { - logrus.Errorf("failed to parse log level %q: %v", jl.Level, err) - return - } - logrus.StandardLogger().Logf(lvl, jl.Msg) + logrus.StandardLogger().Logf(jl.Level, jl.Msg) }