From b1565dafda4b28c243be4029be38c9c3dba83f75 Mon Sep 17 00:00:00 2001 From: jannfis Date: Sun, 16 Jan 2022 14:32:51 +0100 Subject: [PATCH 1/2] chore: Log out the resource triggering reconciliation Signed-off-by: jannfis --- controller/appcontroller.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/controller/appcontroller.go b/controller/appcontroller.go index 06391b8230ae6..55894c02427d0 100644 --- a/controller/appcontroller.go +++ b/controller/appcontroller.go @@ -319,6 +319,13 @@ func (ctrl *ApplicationController) handleObjectUpdated(managedByApp map[string]b if isManagedResource { level = CompareWithRecent } + + // Additional check for debug level so we don't need to evaluate the + // format string in case of non-debug scenarios + if log.GetLevel() >= log.DebugLevel { + log.Debugf("Refreshing app %s for change in cluster of object %s/%s of type %s/%s", appName, ref.Namespace, ref.Name, ref.APIVersion, ref.Kind) + } + ctrl.requestAppRefresh(appName, &level, nil) } } From d5a0efeb853143158f82644afc9b0768ff4709ce Mon Sep 17 00:00:00 2001 From: jannfis Date: Sun, 16 Jan 2022 15:02:19 +0100 Subject: [PATCH 2/2] chore: Log out the resource triggering reconciliation Signed-off-by: jannfis --- controller/appcontroller.go | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/controller/appcontroller.go b/controller/appcontroller.go index 55894c02427d0..3677195fef772 100644 --- a/controller/appcontroller.go +++ b/controller/appcontroller.go @@ -323,7 +323,13 @@ func (ctrl *ApplicationController) handleObjectUpdated(managedByApp map[string]b // Additional check for debug level so we don't need to evaluate the // format string in case of non-debug scenarios if log.GetLevel() >= log.DebugLevel { - log.Debugf("Refreshing app %s for change in cluster of object %s/%s of type %s/%s", appName, ref.Namespace, ref.Name, ref.APIVersion, ref.Kind) + var resKey string + if ref.Namespace != "" { + resKey = ref.Namespace + "/" + ref.Name + } else { + resKey = "(cluster-scoped)/" + ref.Name + } + log.Debugf("Refreshing app %s for change in cluster of object %s of type %s/%s", appName, resKey, ref.APIVersion, ref.Kind) } ctrl.requestAppRefresh(appName, &level, nil)