Merge pull request #3515 from kralicky/disable-helm-controller

Add option to disable the built-in Helm Controller

pkg/cli/cmds/server.go

@@ -53,6 +53,7 @@ type Server struct {
 	DefaultLocalStoragePath  string
 	DisableCCM               bool
 	DisableNPC               bool
+	DisableHelmController    bool
 	DisableKubeProxy         bool
 	DisableAPIServer         bool
 	DisableControllerManager bool
@@ -339,6 +340,11 @@ func NewServerCommand(action func(*cli.Context) error) cli.Command {
 				Usage:       "(components) Disable " + version.Program + " default network policy controller",
 				Destination: &ServerConfig.DisableNPC,
 			},
+			cli.BoolFlag{
+				Name:        "disable-helm-controller",
+				Usage:       "(components) Disable Helm controller",
+				Destination: &ServerConfig.DisableHelmController,
+			},
 			cli.BoolFlag{
 				Name:        "disable-apiserver",
 				Hidden:      true,

pkg/cli/server/server.go

@@ -119,6 +119,7 @@ func run(app *cli.Context, cfg *cmds.Server, leaderControllers server.CustomCont
 	serverConfig.ControlConfig.ExtraCloudControllerArgs = cfg.ExtraCloudControllerArgs
 	serverConfig.ControlConfig.DisableCCM = cfg.DisableCCM
 	serverConfig.ControlConfig.DisableNPC = cfg.DisableNPC
+	serverConfig.ControlConfig.DisableHelmController = cfg.DisableHelmController
 	serverConfig.ControlConfig.DisableKubeProxy = cfg.DisableKubeProxy
 	serverConfig.ControlConfig.DisableETCD = cfg.DisableETCD
 	serverConfig.ControlConfig.DisableAPIServer = cfg.DisableAPIServer

pkg/daemons/config/types.go

@@ -140,6 +140,7 @@ type Control struct {
 	SystemDefaultRegistry    string
 	DisableCCM               bool
 	DisableNPC               bool
+	DisableHelmController    bool
 	DisableKubeProxy         bool
 	DisableAPIServer         bool
 	DisableControllerManager bool

pkg/server/server.go

@@ -186,14 +186,17 @@ func coreControllers(ctx context.Context, sc *Context, config *Config) error {
 		servicelb.DefaultLBImage = config.ControlConfig.SystemDefaultRegistry + "/" + servicelb.DefaultLBImage
 	}
 
-	helm.Register(ctx,
+	if !config.ControlConfig.DisableHelmController {
-		sc.Apply,
+		helm.Register(ctx,
-		sc.Helm.Helm().V1().HelmChart(),
+			sc.Apply,
-		sc.Helm.Helm().V1().HelmChartConfig(),
+			sc.Helm.Helm().V1().HelmChart(),
-		sc.Batch.Batch().V1().Job(),
+			sc.Helm.Helm().V1().HelmChartConfig(),
-		sc.Auth.Rbac().V1().ClusterRoleBinding(),
+			sc.Batch.Batch().V1().Job(),
-		sc.Core.Core().V1().ServiceAccount(),
+			sc.Auth.Rbac().V1().ClusterRoleBinding(),
-		sc.Core.Core().V1().ConfigMap())
+			sc.Core.Core().V1().ServiceAccount(),
+			sc.Core.Core().V1().ConfigMap())
+	}
+
 	if err := servicelb.Register(ctx,
 		sc.K8s,
 		sc.Apply,