From 0e8a7a23cb88396d615c8f2e951e6d83ee770e88 Mon Sep 17 00:00:00 2001 From: Benoit Gagnon Date: Thu, 6 Aug 2020 13:10:59 -0400 Subject: [PATCH] always use groupResource.String() when logging (fixes #2795) (#2796) Signed-off-by: Benoit Gagnon --- changelogs/unreleased/2796-bgagnon | 1 + pkg/backup/item_backupper.go | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) create mode 100644 changelogs/unreleased/2796-bgagnon diff --git a/changelogs/unreleased/2796-bgagnon b/changelogs/unreleased/2796-bgagnon new file mode 100644 index 000000000..1824b5fa2 --- /dev/null +++ b/changelogs/unreleased/2796-bgagnon @@ -0,0 +1 @@ +Fix inconsistent type for the "resource" structured logging field diff --git a/pkg/backup/item_backupper.go b/pkg/backup/item_backupper.go index 74b5eaf70..6c73b3d23 100644 --- a/pkg/backup/item_backupper.go +++ b/pkg/backup/item_backupper.go @@ -73,7 +73,7 @@ func (ib *itemBackupper) backupItem(logger logrus.FieldLogger, obj runtime.Unstr name := metadata.GetName() log := logger.WithField("name", name) - log = log.WithField("resource", groupResource) + log = log.WithField("resource", groupResource.String()) log = log.WithField("namespace", namespace) if metadata.GetLabels()["velero.io/exclude-from-backup"] == "true" {