remove unused go-etcd helper

Author: mfojtik

pkg/cmd/server/etcd/etcd.go

@@ -4,12 +4,9 @@ import (
 	"fmt"
 	"net"
 	"net/http"
-	"net/http/httputil"
 	"time"
 
 	newetcdclient "github.com/coreos/etcd/client"
-	etcdclient "github.com/coreos/go-etcd/etcd"
-	"github.com/golang/glog"
 	"golang.org/x/net/context"
 
 	"k8s.io/kubernetes/pkg/client/restclient"
@@ -23,48 +20,17 @@ import (
 // connect to the etcd server and block until the server responds at least once, or return an
 // error if the server never responded.
 // TODO: switch this function to use EtcdHelper.
-func GetAndTestEtcdClient(etcdClientInfo configapi.EtcdConnectionInfo) (*etcdclient.Client, error) {
-	etcdClient, err := EtcdClient(etcdClientInfo)
+func GetAndTestEtcdClient(etcdClientInfo configapi.EtcdConnectionInfo) (newetcdclient.Client, error) {
+	etcdClient, err := MakeNewEtcdClient(etcdClientInfo)
 	if err != nil {
 		return nil, err
 	}
-	if err := TestEtcdClient(etcdClient); err != nil {
+	if err := TestNewEtcdClient(etcdClient); err != nil {
 		return nil, err
 	}
 	return etcdClient, nil
 }
 
-// EtcdClient creates an etcd client based on the provided config.
-func EtcdClient(etcdClientInfo configapi.EtcdConnectionInfo) (*etcdclient.Client, error) {
-	tlsConfig, err := restclient.TLSConfigFor(&restclient.Config{
-		TLSClientConfig: restclient.TLSClientConfig{
-			CertFile: etcdClientInfo.ClientCert.CertFile,
-			KeyFile:  etcdClientInfo.ClientCert.KeyFile,
-			CAFile:   etcdClientInfo.CA,
-		},
-	})
-	if err != nil {
-		return nil, err
-	}
-
-	transport := knet.SetTransportDefaults(&http.Transport{
-		TLSClientConfig: tlsConfig,
-		Dial: (&net.Dialer{
-			// default from http.DefaultTransport
-			Timeout: 30 * time.Second,
-			// Lower the keep alive for connections.
-			KeepAlive: 1 * time.Second,
-		}).Dial,
-		// Because watches are very bursty, defends against long delays in watch reconnections.
-		MaxIdleConnsPerHost: 500,
-	})
-
-	etcdClient := etcdclient.NewClient(etcdClientInfo.URLs)
-	etcdClient.SetTransport(transport)
-	etcdClient.CheckRetry = NeverRetryOnFailure
-	return etcdClient, nil
-}
-
 // MakeNewEtcdClient creates an etcd client based on the provided config.
 func MakeNewEtcdClient(etcdClientInfo configapi.EtcdConnectionInfo) (newetcdclient.Client, error) {
 	tlsConfig, err := restclient.TLSConfigFor(&restclient.Config{
@@ -98,23 +64,6 @@ func MakeNewEtcdClient(etcdClientInfo configapi.EtcdConnectionInfo) (newetcdclie
 	return newetcdclient.New(cfg)
 }
 
-// TestEtcdClient verifies a client is functional.  It will attempt to
-// connect to the etcd server and block until the server responds at least once, or return an
-// error if the server never responded.
-func TestEtcdClient(etcdClient *etcdclient.Client) error {
-	for i := 0; ; i++ {
-		_, err := etcdClient.Get("/", false, false)
-		if err == nil || etcdutil.IsEtcdNotFound(err) {
-			break
-		}
-		if i > 100 {
-			return fmt.Errorf("could not reach etcd: %v", err)
-		}
-		time.Sleep(50 * time.Millisecond)
-	}
-	return nil
-}
-
 // TestEtcdClient verifies a client is functional.  It will attempt to
 // connect to the etcd server and block until the server responds at least once, or return an
 // error if the server never responded.
@@ -131,19 +80,3 @@ func TestNewEtcdClient(etcdClient newetcdclient.Client) error {
 	}
 	return nil
 }
-
-// NeverRetryOnFailure is a retry function for the etcdClient.  If there's only one machine, master election doesn't make much sense,
-// so we don't bother to retry, we simply dump the failure and return the error directly.
-func NeverRetryOnFailure(cluster *etcdclient.Cluster, numReqs int, lastResp http.Response, err error) error {
-	if len(cluster.Machines) > 1 {
-		return etcdclient.DefaultCheckRetry(cluster, numReqs, lastResp, err)
-	}
-
-	content, err := httputil.DumpResponse(&lastResp, true)
-	if err != nil {
-		glog.Errorf("failure dumping response: %v", err)
-	} else {
-		glog.Errorf("etcd failure response: %s", string(content))
-	}
-	return err
-}

pkg/cmd/server/start/start_master.go

@@ -449,10 +449,8 @@ func StartAPI(oc *origin.MasterConfig, kc *kubernetes.MasterConfig) error {
 	}
 
 	// verify we can connect to etcd with the provided config
-	if etcdClient, err := etcd.GetAndTestEtcdClient(oc.Options.EtcdClientInfo); err != nil {
+	if _, err := etcd.GetAndTestEtcdClient(oc.Options.EtcdClientInfo); err != nil {
 		return err
-	} else {
-		etcdClient.Close()
 	}
 
 	// Must start policy caching immediately