From b30ca0c8a27cf0703c9789332d8d43300379b862 Mon Sep 17 00:00:00 2001 From: Minglong Song <1247768069@qq.com> Date: Mon, 10 Jan 2022 14:48:58 +0800 Subject: [PATCH] Fix: Do not fiter event triggered by resync for application controller (#3064) Signed-off-by: songminglong <1247768069@qq.com> --- .../v1alpha2/application/application_controller.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/pkg/controller/core.oam.dev/v1alpha2/application/application_controller.go b/pkg/controller/core.oam.dev/v1alpha2/application/application_controller.go index 9d86685e6..72ca87ff6 100644 --- a/pkg/controller/core.oam.dev/v1alpha2/application/application_controller.go +++ b/pkg/controller/core.oam.dev/v1alpha2/application/application_controller.go @@ -398,6 +398,12 @@ func (r *Reconciler) SetupWithManager(mgr ctrl.Manager) error { if !isNewApp || !isOldApp { return filterManagedFieldChangesUpdate(e) } + + // We think this event is triggered by resync + if reflect.DeepEqual(old, new) { + return true + } + // filter managedFields changes new.ManagedFields = old.ManagedFields