Merge pull request #5941 from XiShanYongYe-Chang/cherry-pick-#5899-for-release-1.11

Disable cluster failover by default which should be explicitly enabled by administrators after a fully evaluation

Author: karmada-bot

Diff for: artifacts/deploy/karmada-controller-manager.yaml

@@ -31,7 +31,7 @@ spec:
             - --cluster-status-update-frequency=10s
             - --failover-eviction-timeout=30s
             - --controllers=*,hpaScaleTargetMarker,deploymentReplicasSyncer
-            - --feature-gates=PropagationPolicyPreemption=true,MultiClusterService=true
+            - --feature-gates=Failover=true,PropagationPolicyPreemption=true,MultiClusterService=true
             - --health-probe-bind-address=0.0.0.0:10357
             - --v=4
           livenessProbe:

Diff for: cmd/controller-manager/app/controllermanager.go

@@ -592,9 +592,6 @@ func startGracefulEvictionController(ctx controllerscontext.Context) (enabled bo
 }
 
 func startApplicationFailoverController(ctx controllerscontext.Context) (enabled bool, err error) {
-	if !features.FeatureGate.Enabled(features.Failover) {
-		return false, nil
-	}
 	rbApplicationFailoverController := applicationfailover.RBApplicationFailoverController{
 		Client:              ctx.Mgr.GetClient(),
 		EventRecorder:       ctx.Mgr.GetEventRecorderFor(applicationfailover.RBApplicationFailoverControllerName),