From c201a78dff911abe210be9ebb92e97274ef5efe8 Mon Sep 17 00:00:00 2001 From: Matthew Wong Date: Thu, 13 May 2021 15:26:51 -0700 Subject: [PATCH] Remove etcd connection apiserver preflight check --- cmd/kube-apiserver/app/server.go | 13 ------------- vendor/modules.txt | 1 - 2 files changed, 14 deletions(-) diff --git a/cmd/kube-apiserver/app/server.go b/cmd/kube-apiserver/app/server.go index 03205f07626..0aed0179b19 100644 --- a/cmd/kube-apiserver/app/server.go +++ b/cmd/kube-apiserver/app/server.go @@ -37,7 +37,6 @@ import ( utilerrors "k8s.io/apimachinery/pkg/util/errors" utilnet "k8s.io/apimachinery/pkg/util/net" "k8s.io/apimachinery/pkg/util/sets" - utilwait "k8s.io/apimachinery/pkg/util/wait" "k8s.io/apiserver/pkg/admission" "k8s.io/apiserver/pkg/authorization/authorizer" openapinamer "k8s.io/apiserver/pkg/endpoints/openapi" @@ -47,7 +46,6 @@ import ( "k8s.io/apiserver/pkg/server/filters" serveroptions "k8s.io/apiserver/pkg/server/options" serverstorage "k8s.io/apiserver/pkg/server/storage" - "k8s.io/apiserver/pkg/storage/etcd3/preflight" utilfeature "k8s.io/apiserver/pkg/util/feature" utilflowcontrol "k8s.io/apiserver/pkg/util/flowcontrol" "k8s.io/apiserver/pkg/util/webhook" @@ -81,11 +79,6 @@ import ( "k8s.io/kubernetes/pkg/serviceaccount" ) -const ( - etcdRetryLimit = 60 - etcdRetryInterval = 1 * time.Second -) - // TODO: delete this check after insecure flags removed in v1.24 func checkNonZeroInsecurePort(fs *pflag.FlagSet) error { for _, name := range options.InsecurePortFlags { @@ -298,12 +291,6 @@ func CreateKubeAPIServerConfig( return nil, nil, nil, err } - if _, port, err := net.SplitHostPort(s.Etcd.StorageConfig.Transport.ServerList[0]); err == nil && port != "0" && len(port) != 0 { - if err := utilwait.PollImmediate(etcdRetryInterval, etcdRetryLimit*etcdRetryInterval, preflight.EtcdConnection{ServerList: s.Etcd.StorageConfig.Transport.ServerList}.CheckEtcdServers); err != nil { - return nil, nil, nil, fmt.Errorf("error waiting for etcd connection: %v", err) - } - } - capabilities.Initialize(capabilities.Capabilities{ AllowPrivileged: s.AllowPrivileged, // TODO(vmarmol): Implement support for HostNetworkSources. diff --git a/vendor/modules.txt b/vendor/modules.txt index fbb99d5217c..cb338280ffd 100644 --- a/vendor/modules.txt +++ b/vendor/modules.txt @@ -1498,7 +1498,6 @@ k8s.io/apiserver/pkg/storage/cacher k8s.io/apiserver/pkg/storage/errors k8s.io/apiserver/pkg/storage/etcd3 k8s.io/apiserver/pkg/storage/etcd3/metrics -k8s.io/apiserver/pkg/storage/etcd3/preflight k8s.io/apiserver/pkg/storage/etcd3/testing k8s.io/apiserver/pkg/storage/etcd3/testing/testingcert k8s.io/apiserver/pkg/storage/names