From 457c62bd0cadb8afd411d69f73fc4f292802f4ab Mon Sep 17 00:00:00 2001 From: Derek Su Date: Mon, 17 Feb 2025 07:56:58 +0800 Subject: [PATCH] fix: fix the false alarm while stopping syncing backup status Use warning-level messages for context canceled. Longhorn 10301 Signed-off-by: Derek Su --- engineapi/backup_monitor.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/engineapi/backup_monitor.go b/engineapi/backup_monitor.go index 8a4a04d0d4..63a7ae66bf 100644 --- a/engineapi/backup_monitor.go +++ b/engineapi/backup_monitor.go @@ -9,6 +9,7 @@ import ( "sync" "time" + "github.com/pkg/errors" "github.com/sirupsen/logrus" "k8s.io/apimachinery/pkg/util/wait" @@ -209,7 +210,11 @@ func (m *BackupMonitor) linerTimer() { m.syncCallBack(currentBackupStatus) return false, nil }); err != nil { - m.logger.WithError(err).Error("Failed to sync backup status") + if errors.Is(err, context.Canceled) { + m.logger.WithError(err).Warn("Sync backup status canceled") + } else { + m.logger.WithError(err).Error("Failed to sync backup status") + } } }