From 21bdaaf84f20b3f98d5328731b14a2b51da8dcb7 Mon Sep 17 00:00:00 2001 From: Wainer dos Santos Moschetta Date: Mon, 1 Mar 2021 15:26:11 -0500 Subject: [PATCH] runtime: Fix missing 'name' field on containerd-shim-v2 logs Each Kata Containers application should generate log records with a specified structure. Currently on containerd-shim-v2's logs, the required 'name' field is missing. This changed its logger to append the application name on each and every emitted entries. Fixes #1479 Related-to: github.com/kata-containers/tests/issues/3260 Suggested-by: James O. D. Hunt Signed-off-by: Wainer dos Santos Moschetta --- src/runtime/containerd-shim-v2/service.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/runtime/containerd-shim-v2/service.go b/src/runtime/containerd-shim-v2/service.go index 26d83c3ee..80117baa1 100644 --- a/src/runtime/containerd-shim-v2/service.go +++ b/src/runtime/containerd-shim-v2/service.go @@ -62,7 +62,10 @@ var ( var vci vc.VC = &vc.VCImpl{} // shimLog is logger for shim package -var shimLog = logrus.WithField("source", "containerd-kata-shim-v2") +var shimLog = logrus.WithFields(logrus.Fields{ + "source": "containerd-kata-shim-v2", + "name": "containerd-shim-v2", +}) // New returns a new shim service that can be used via GRPC func New(ctx context.Context, id string, publisher events.Publisher) (cdshim.Shim, error) {