cli/command: remove unused EndpointDefaultResolver interface

this was added as part of 520be05, but
is not used anywhere, so we may as well simplify things a bit.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Author: thaJeztah

cli/command/cli.go

@@ -214,7 +214,7 @@ func (cli *DockerCli) Initialize(opts *cliflags.ClientOptions, ops ...Initialize
 	cli.contextStore = &ContextStoreWithDefault{
 		Store: baseContextStore,
 		Resolver: func() (*DefaultContext, error) {
-			return ResolveDefaultContext(opts.Common, cli.contextStoreConfig)
+			return ResolveDefaultContext(opts.Common)
 		},
 	}
 	cli.currentContext, err = resolveContextName(opts.Common, cli.configFile, cli.contextStore)
@@ -238,11 +238,10 @@ func (cli *DockerCli) Initialize(opts *cliflags.ClientOptions, ops ...Initialize
 
 // NewAPIClientFromFlags creates a new APIClient from command line flags
 func NewAPIClientFromFlags(opts *cliflags.CommonOptions, configFile *configfile.ConfigFile) (client.APIClient, error) {
-	storeConfig := DefaultContextStoreConfig()
 	contextStore := &ContextStoreWithDefault{
-		Store: store.New(config.ContextStoreDir(), storeConfig),
+		Store: store.New(config.ContextStoreDir(), DefaultContextStoreConfig()),
 		Resolver: func() (*DefaultContext, error) {
-			return ResolveDefaultContext(opts, storeConfig)
+			return ResolveDefaultContext(opts)
 		},
 	}
 	contextName, err := resolveContextName(opts, configFile, contextStore)

cli/command/defaultcontextstore.go

@@ -29,66 +29,32 @@ type ContextStoreWithDefault struct {
 	Resolver DefaultContextResolver
 }
 
-// EndpointDefaultResolver is implemented by any EndpointMeta object
-// which wants to be able to populate the store with whatever their default is.
-type EndpointDefaultResolver interface {
-	// ResolveDefault returns values suitable for storing in store.Metadata.Endpoints
-	// and store.ContextTLSData.Endpoints.
-	//
-	// An error is only returned for something fatal, not simply
-	// the lack of a default (e.g. because the config file which
-	// would contain it is missing). If there is no default then
-	// returns nil, nil, nil.
-	ResolveDefault() (interface{}, *store.EndpointTLSData, error)
-}
-
 // ResolveDefaultContext creates a Metadata for the current CLI invocation parameters
-func ResolveDefaultContext(opts *cliflags.CommonOptions, config store.Config) (*DefaultContext, error) {
-	contextTLSData := store.ContextTLSData{
-		Endpoints: make(map[string]store.EndpointTLSData),
-	}
-	contextMetadata := store.Metadata{
-		Endpoints: make(map[string]interface{}),
-		Metadata: DockerContext{
-			Description: "",
-		},
-		Name: DefaultContextName,
-	}
-
+func ResolveDefaultContext(opts *cliflags.CommonOptions) (*DefaultContext, error) {
 	dockerEP, err := resolveDefaultDockerEndpoint(opts)
 	if err != nil {
 		return nil, err
 	}
-	contextMetadata.Endpoints[docker.DockerEndpoint] = dockerEP.EndpointMeta
-	if dockerEP.TLSData != nil {
-		contextTLSData.Endpoints[docker.DockerEndpoint] = *dockerEP.TLSData.ToStoreTLSData()
-	}
 
-	if err := config.ForeachEndpointType(func(n string, get store.TypeGetter) error {
-		if n == docker.DockerEndpoint { // handled above
-			return nil
-		}
-		ep := get()
-		if i, ok := ep.(EndpointDefaultResolver); ok {
-			meta, tls, err := i.ResolveDefault()
-			if err != nil {
-				return err
-			}
-			if meta == nil {
-				return nil
-			}
-			contextMetadata.Endpoints[n] = meta
-			if tls != nil {
-				contextTLSData.Endpoints[n] = *tls
-			}
+	contextTLSData := store.ContextTLSData{}
+	if dockerEP.TLSData != nil {
+		contextTLSData.Endpoints = map[string]store.EndpointTLSData{
+			docker.DockerEndpoint: *dockerEP.TLSData.ToStoreTLSData(),
 		}
-		// Nothing to be done
-		return nil
-	}); err != nil {
-		return nil, err
 	}
 
-	return &DefaultContext{Meta: contextMetadata, TLS: contextTLSData}, nil
+	return &DefaultContext{
+		Meta: store.Metadata{
+			Endpoints: map[string]interface{}{
+				docker.DockerEndpoint: dockerEP.EndpointMeta,
+			},
+			Metadata: DockerContext{
+				Description: "",
+			},
+			Name: DefaultContextName,
+		},
+		TLS: contextTLSData,
+	}, nil
 }
 
 // List implements store.Store's List

cli/command/defaultcontextstore_test.go

@@ -60,7 +60,7 @@ func TestDefaultContextInitializer(t *testing.T) {
 		TLSOptions: &tlsconfig.Options{
 			CAFile: "./testdata/ca.pem",
 		},
-	}, DefaultContextStoreConfig())
+	})
 	assert.NilError(t, err)
 	assert.Equal(t, "default", ctx.Meta.Name)
 	assert.DeepEqual(t, "ssh://someswarmserver", ctx.Meta.Endpoints[docker.DockerEndpoint].(docker.EndpointMeta).Host)