From 29a8608bfa4ed92e5bec61a6747f96b370b0fdc6 Mon Sep 17 00:00:00 2001 From: "zhongjun.li" Date: Wed, 3 Nov 2021 10:16:26 +0800 Subject: [PATCH] fix/skipped-propagating-apis_not_take_effect Signed-off-by: zhongjun.li --- cmd/controller-manager/app/controllermanager.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/cmd/controller-manager/app/controllermanager.go b/cmd/controller-manager/app/controllermanager.go index eeb38537c..1015969bf 100644 --- a/cmd/controller-manager/app/controllermanager.go +++ b/cmd/controller-manager/app/controllermanager.go @@ -114,6 +114,10 @@ func setupControllers(mgr controllerruntime.Manager, opts *options.Options, stop objectWatcher := objectwatcher.NewObjectWatcher(mgr.GetClient(), mgr.GetRESTMapper(), util.NewClusterDynamicClientSet) overrideManager := overridemanager.New(mgr.GetClient()) skippedResourceConfig := util.NewSkippedResourceConfig() + if err := skippedResourceConfig.Parse(opts.SkippedPropagatingAPIs); err != nil { + // The program will never go here because the parameters have been checked + return + } skippedPropagatingNamespaces := map[string]struct{}{} for _, ns := range opts.SkippedPropagatingNamespaces {