Make default context behaves like a real context:

- when using "--context default" parameter
- when printing the list of contexts
- when exporting the default context to a tarball

Signed-off-by: Jean-Christophe Sirot <jean-christophe.sirot@docker.com>

Author: Jean-Christophe Sirot

cli/command/cli.go

@@ -210,7 +210,13 @@ func (cli *DockerCli) Initialize(opts *cliflags.ClientOptions, ops ...Initialize
 	cli.configFile = cliconfig.LoadDefaultConfigFile(cli.err)
 
 	if cli.client == nil {
-		cli.contextStore = store.New(cliconfig.ContextStoreDir(), cli.contextStoreConfig)
+		baseContextSore := store.New(cliconfig.ContextStoreDir(), cli.contextStoreConfig)
+		cli.contextStore = &ContextStoreWithDefault{
+			Store: baseContextSore,
+			Resolver: func() (*DefaultContext, error) {
+				return resolveDefaultContext(opts.Common, cli.ConfigFile(), cli.Err())
+			},
+		}
 		cli.currentContext, err = resolveContextName(opts.Common, cli.configFile, cli.contextStore)
 		if err != nil {
 			return err
@@ -280,7 +286,7 @@ func newAPIClientFromEndpoint(ep docker.Endpoint, configFile *configfile.ConfigF
 }
 
 func resolveDockerEndpoint(s store.Store, contextName string, opts *cliflags.CommonOptions) (docker.Endpoint, error) {
-	if contextName != "" {
+	if contextName != "" || contextName == DefaultContextName {
 		ctxMeta, err := s.GetContextMetadata(contextName)
 		if err != nil {
 			return docker.Endpoint{}, err
@@ -291,6 +297,10 @@ func resolveDockerEndpoint(s store.Store, contextName string, opts *cliflags.Com
 		}
 		return docker.WithTLSData(s, contextName, epMeta)
 	}
+	return resolveDefaultDockerEndpoint(opts)
+}
+
+func resolveDefaultDockerEndpoint(opts *cliflags.CommonOptions) (docker.Endpoint, error) {
 	host, err := getServerHost(opts.Hosts, opts.TLSOptions)
 	if err != nil {
 		return docker.Endpoint{}, err
@@ -396,7 +406,7 @@ func (cli *DockerCli) StackOrchestrator(flagValue string) (Orchestrator, error)
 	if currentContext == "" {
 		currentContext = configFile.CurrentContext
 	}
-	if currentContext != "" {
+	if currentContext != "" && currentContext != DefaultContextName {
 		contextstore := cli.contextStore
 		if contextstore == nil {
 			contextstore = store.New(cliconfig.ContextStoreDir(), cli.contextStoreConfig)
@@ -508,7 +518,7 @@ func resolveContextName(opts *cliflags.CommonOptions, config *configfile.ConfigF
 	if opts.Context != "" && len(opts.Hosts) > 0 {
 		return "", errors.New("Conflicting options: either specify --host or --context, not both")
 	}
-	if opts.Context != "" {
+	if opts.Context != "" && opts.Context != DefaultContextName {
 		return opts.Context, nil
 	}
 	if len(opts.Hosts) > 0 {

cli/command/context/create_test.go

@@ -23,7 +23,26 @@ func makeFakeCli(t *testing.T, opts ...func(*test.FakeCli)) (*test.FakeCli, func
 		store.EndpointTypeGetter(docker.DockerEndpoint, func() interface{} { return &docker.EndpointMeta{} }),
 		store.EndpointTypeGetter(kubernetes.KubernetesEndpoint, func() interface{} { return &kubernetes.EndpointMeta{} }),
 	)
-	store := store.New(dir, storeConfig)
+	store := &command.ContextStoreWithDefault{
+		Store: store.New(dir, storeConfig),
+		Resolver: func() (*command.DefaultContext, error) {
+			return &command.DefaultContext{
+				Meta: store.ContextMetadata{
+					Endpoints: map[string]interface{}{
+						docker.DockerEndpoint: docker.EndpointMeta{
+							Host: "unix:///var/run/docker.sock",
+						},
+					},
+					Metadata: command.DockerContext{
+						Description:       "",
+						StackOrchestrator: command.OrchestratorSwarm,
+					},
+					Name: command.DefaultContextName,
+				},
+				TLS: store.ContextTLSData{},
+			}, nil
+		},
+	}
 	cleanup := func() {
 		os.RemoveAll(dir)
 	}
@@ -52,6 +71,12 @@ func TestCreateInvalids(t *testing.T) {
 		{
 			expecterErr: `context name cannot be empty`,
 		},
+		{
+			options: CreateOptions{
+				Name: "default",
+			},
+			expecterErr: `"default" is a reserved context name`,
+		},
 		{
 			options: CreateOptions{
 				Name: " ",

cli/command/context/export.go

@@ -77,7 +77,7 @@ func writeTo(dockerCli command.Cli, reader io.Reader, dest string) error {
 
 // RunExport exports a Docker context
 func RunExport(dockerCli command.Cli, opts *ExportOptions) error {
-	if err := validateContextName(opts.ContextName); err != nil {
+	if err := validateContextName(opts.ContextName); err != nil && opts.ContextName != "default" {
 		return err
 	}
 	ctxMeta, err := dockerCli.ContextStore().GetContextMetadata(opts.ContextName)

cli/command/context/inspect.go

@@ -40,9 +40,6 @@ func newInspectCommand(dockerCli command.Cli) *cobra.Command {
 
 func runInspect(dockerCli command.Cli, opts inspectOptions) error {
 	getRefFunc := func(ref string) (interface{}, []byte, error) {
-		if ref == "default" {
-			return nil, nil, errors.New(`context "default" cannot be inspected`)
-		}
 		c, err := dockerCli.ContextStore().GetContextMetadata(ref)
 		if err != nil {
 			return nil, nil, err

cli/command/context/list.go

@@ -9,7 +9,6 @@ import (
 	"github.com/docker/cli/cli/command/formatter"
 	"github.com/docker/cli/cli/context/docker"
 	kubecontext "github.com/docker/cli/cli/context/kubernetes"
-	"github.com/docker/cli/kubernetes"
 	"github.com/spf13/cobra"
 	"vbom.ml/util/sortorder"
 )
@@ -61,39 +60,19 @@ func runList(dockerCli command.Cli, opts *listOptions) error {
 		if kubernetesEndpoint != nil {
 			kubEndpointText = fmt.Sprintf("%s (%s)", kubernetesEndpoint.Host, kubernetesEndpoint.DefaultNamespace)
 		}
+		if rawMeta.Name == command.DefaultContextName {
+			meta.Description = "Current DOCKER_HOST based configuration"
+		}
 		desc := formatter.ClientContext{
 			Name:               rawMeta.Name,
-			Current:            rawMeta.Name == curContext,
+			Current:            rawMeta.Name == curContext || (curContext == "" && rawMeta.Name == command.DefaultContextName),
 			Description:        meta.Description,
 			StackOrchestrator:  string(meta.StackOrchestrator),
 			DockerEndpoint:     dockerEndpoint.Host,
 			KubernetesEndpoint: kubEndpointText,
 		}
 		contexts = append(contexts, &desc)
 	}
-	if !opts.quiet {
-		desc := &formatter.ClientContext{
-			Name:        "default",
-			Description: "Current DOCKER_HOST based configuration",
-		}
-		if dockerCli.CurrentContext() == "" {
-			orchestrator, _ := dockerCli.StackOrchestrator("")
-			kubEndpointText := ""
-			kubeconfig := kubernetes.NewKubernetesConfig("")
-			if cfg, err := kubeconfig.ClientConfig(); err == nil {
-				ns, _, _ := kubeconfig.Namespace()
-				if ns == "" {
-					ns = "default"
-				}
-				kubEndpointText = fmt.Sprintf("%s (%s)", cfg.Host, ns)
-			}
-			desc.Current = true
-			desc.StackOrchestrator = string(orchestrator)
-			desc.DockerEndpoint = dockerCli.DockerEndpoint().Host
-			desc.KubernetesEndpoint = kubEndpointText
-		}
-		contexts = append(contexts, desc)
-	}
 	sort.Slice(contexts, func(i, j int) bool {
 		return sortorder.NaturalLess(contexts[i].Name, contexts[j].Name)
 	})

cli/command/context/list_test.go

@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/docker/cli/cli/command"
-	"github.com/docker/cli/cli/context/docker"
 	"gotest.tools/assert"
 	"gotest.tools/env"
 	"gotest.tools/golden"
@@ -36,20 +35,6 @@ func TestList(t *testing.T) {
 	golden.Assert(t, cli.OutBuffer().String(), "list.golden")
 }
 
-func TestListNoContext(t *testing.T) {
-	cli, cleanup := makeFakeCli(t)
-	defer cleanup()
-	defer env.Patch(t, "KUBECONFIG", "./testdata/test-kubeconfig")()
-	cli.SetDockerEndpoint(docker.Endpoint{
-		EndpointMeta: docker.EndpointMeta{
-			Host: "https://someswarmserver",
-		},
-	})
-	cli.OutBuffer().Reset()
-	assert.NilError(t, runList(cli, &listOptions{}))
-	golden.Assert(t, cli.OutBuffer().String(), "list.no-context.golden")
-}
-
 func TestListQuiet(t *testing.T) {
 	cli, cleanup := makeFakeCli(t)
 	defer cleanup()

cli/command/context/remove_test.go

@@ -62,3 +62,12 @@ func TestRemoveCurrentForce(t *testing.T) {
 	assert.NilError(t, err)
 	assert.Equal(t, "", reloadedConfig.CurrentContext)
 }
+
+func TestRemoveDefault(t *testing.T) {
+	cli, cleanup := makeFakeCli(t)
+	defer cleanup()
+	createTestContextWithKubeAndSwarm(t, cli, "other", "all")
+	cli.SetCurrentContext("current")
+	err := RunRemove(cli, RemoveOptions{}, []string{"default"})
+	assert.ErrorContains(t, err, "default: context \"default\" cannot be removed")
+}

cli/command/context/testdata/list.golden

@@ -1,5 +1,5 @@
-NAME                DESCRIPTION                               DOCKER ENDPOINT           KUBERNETES ENDPOINT            ORCHESTRATOR
-current *           description of current                    https://someswarmserver   https://someserver (default)   all
-default             Current DOCKER_HOST based configuration                                                            
-other               description of other                      https://someswarmserver   https://someserver (default)   all
-unset               description of unset                      https://someswarmserver   https://someserver (default)   
+NAME                DESCRIPTION                               DOCKER ENDPOINT               KUBERNETES ENDPOINT            ORCHESTRATOR
+current *           description of current                    https://someswarmserver       https://someserver (default)   all
+default             Current DOCKER_HOST based configuration   unix:///var/run/docker.sock                                  swarm
+other               description of other                      https://someswarmserver       https://someserver (default)   all
+unset               description of unset                      https://someswarmserver       https://someserver (default)   

cli/command/context/testdata/list.no-context.golden

@@ -1,2 +1,3 @@
 current
+default
 other