Skip to content

Commit

Permalink
Remove GA feature-gates (k3s-io#8970)
Browse files Browse the repository at this point in the history
Remove KubeletCredentialProviders and JobTrackingWithFinalizers feature-gates, both of which are GA and cannot be disabled.

Signed-off-by: Brad Davidson <brad.davidson@rancher.com>
  • Loading branch information
brandond authored Dec 14, 2023
1 parent 71a3c35 commit 231cb6e
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
1 change: 0 additions & 1 deletion pkg/daemons/agent/agent_linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,6 @@ func kubeletArgs(cfg *config.Agent) map[string]string {

if ImageCredProvAvailable(cfg) {
logrus.Infof("Kubelet image credential provider bin dir and configuration file found.")
argsMap["feature-gates"] = util.AddFeatureGate(argsMap["feature-gates"], "KubeletCredentialProviders=true")
argsMap["image-credential-provider-bin-dir"] = cfg.ImageCredProvBinDir
argsMap["image-credential-provider-config"] = cfg.ImageCredProvConfig
}
Expand Down
1 change: 0 additions & 1 deletion pkg/daemons/agent/agent_windows.go
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@ func kubeletArgs(cfg *config.Agent) map[string]string {

if ImageCredProvAvailable(cfg) {
logrus.Infof("Kubelet image credential provider bin dir and configuration file found.")
argsMap["feature-gates"] = util.AddFeatureGate(argsMap["feature-gates"], "KubeletCredentialProviders=true")
argsMap["image-credential-provider-bin-dir"] = cfg.ImageCredProvBinDir
argsMap["image-credential-provider-config"] = cfg.ImageCredProvConfig
}
Expand Down
5 changes: 1 addition & 4 deletions pkg/daemons/control/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ func controllerManager(ctx context.Context, cfg *config.Control) error {
runtime := cfg.Runtime
argsMap := map[string]string{
"controllers": "*,tokencleaner",
"feature-gates": "JobTrackingWithFinalizers=true",
"kubeconfig": runtime.KubeConfigController,
"authorization-kubeconfig": runtime.KubeConfigController,
"authentication-kubeconfig": runtime.KubeConfigController,
Expand Down Expand Up @@ -153,9 +152,7 @@ func scheduler(ctx context.Context, cfg *config.Control) error {

func apiServer(ctx context.Context, cfg *config.Control) error {
runtime := cfg.Runtime
argsMap := map[string]string{
"feature-gates": "JobTrackingWithFinalizers=true",
}
argsMap := map[string]string{}

setupStorageBackend(argsMap, cfg)

Expand Down

0 comments on commit 231cb6e

Please sign in to comment.