From b68fd5e5336311cb520d7b558fd6c38c4298544e Mon Sep 17 00:00:00 2001 From: lukasmetzner Date: Wed, 22 Oct 2025 13:30:15 +0200 Subject: [PATCH] docs: cleanup docstrings --- .../controllers/route/route_controller.go | 6 ++++-- .../controller-manager/pkg/features/kube_features.go | 11 +---------- 2 files changed, 5 insertions(+), 12 deletions(-) diff --git a/staging/src/k8s.io/cloud-provider/controllers/route/route_controller.go b/staging/src/k8s.io/cloud-provider/controllers/route/route_controller.go index e0ef4162cae..caf44dc5795 100644 --- a/staging/src/k8s.io/cloud-provider/controllers/route/route_controller.go +++ b/staging/src/k8s.io/cloud-provider/controllers/route/route_controller.go @@ -55,8 +55,10 @@ const ( // Maximal number of concurrent route operation API calls. // TODO: This should be per-provider. maxConcurrentRouteOperations int = 200 - // In the scenarios with a burst of node events, - // we don't want to be worse than a 10s interval (current default reconcile interval). + // In the scenarios with a burst of node events, we don't want to be worse + // than a 10s interval. + // The 10s interval was defined in KEP 5237, it is equivalent to the default + // sync period of the route controller. minRouteResyncInterval time.Duration = 10 * time.Second ) diff --git a/staging/src/k8s.io/controller-manager/pkg/features/kube_features.go b/staging/src/k8s.io/controller-manager/pkg/features/kube_features.go index 6a4bfacf5ba..dac61e68eb2 100644 --- a/staging/src/k8s.io/controller-manager/pkg/features/kube_features.go +++ b/staging/src/k8s.io/controller-manager/pkg/features/kube_features.go @@ -31,17 +31,8 @@ import ( // of code conflicts because changes are more likely to be scattered // across the file. const ( - // Every feature gate should add method here following this template: - // - // // owner: @username - // MyFeature featuregate.Feature = "MyFeature" - // - // Feature gates should be listed in alphabetical, case-sensitive - // (upper before any lower case character) order. This reduces the risk - // of code conflicts because changes are more likely to be scattered - // across the file. - // owner: @lukasmetzner + // kep: http://kep.k8s.io/5237 // Use watch based route controller reconciliation instead of frequent periodic reconciliation. CloudControllerManagerWatchBasedRoutesReconciliation featuregate.Feature = "CloudControllerManagerWatchBasedRoutesReconciliation"