diff --git a/cli/cobra.go b/cli/cobra.go index 7cf7e810d9..41622064c7 100644 --- a/cli/cobra.go +++ b/cli/cobra.go @@ -116,7 +116,13 @@ type TopLevelCommand struct { // NewTopLevelCommand returns a new TopLevelCommand object func NewTopLevelCommand(cmd *cobra.Command, dockerCli *command.DockerCli, opts *cliflags.ClientOptions, flags *pflag.FlagSet) *TopLevelCommand { - return &TopLevelCommand{cmd, dockerCli, opts, flags, os.Args[1:]} + return &TopLevelCommand{ + cmd: cmd, + dockerCli: dockerCli, + opts: opts, + flags: flags, + args: os.Args[1:], + } } // SetArgs sets the args (default os.Args[:1] used to invoke the command diff --git a/cli/command/cli.go b/cli/command/cli.go index b6fb6b5124..0cce3fa505 100644 --- a/cli/command/cli.go +++ b/cli/command/cli.go @@ -69,6 +69,7 @@ type Cli interface { // Instances of the client can be returned from NewDockerCli. type DockerCli struct { configFile *configfile.ConfigFile + options *cliflags.ClientOptions in *streams.In out *streams.Out err io.Writer @@ -132,16 +133,13 @@ func ShowHelp(err io.Writer) func(*cobra.Command, []string) error { // ConfigFile returns the ConfigFile func (cli *DockerCli) ConfigFile() *configfile.ConfigFile { + // TODO(thaJeztah): when would this happen? Is this only in tests (where cli.Initialize() is not called first?) if cli.configFile == nil { - cli.loadConfigFile() + cli.configFile = config.LoadDefaultConfigFile(cli.err) } return cli.configFile } -func (cli *DockerCli) loadConfigFile() { - cli.configFile = config.LoadDefaultConfigFile(cli.err) -} - // ServerInfo returns the server version details for the host this client is // connected to func (cli *DockerCli) ServerInfo() ServerInfo { @@ -225,17 +223,16 @@ func (cli *DockerCli) Initialize(opts *cliflags.ClientOptions, ops ...Initialize return errors.New("conflicting options: either specify --host or --context, not both") } - cli.loadConfigFile() - - baseContextStore := store.New(config.ContextStoreDir(), cli.contextStoreConfig) + cli.options = opts + cli.configFile = config.LoadDefaultConfigFile(cli.err) + cli.currentContext = resolveContextName(cli.options, cli.configFile) cli.contextStore = &ContextStoreWithDefault{ - Store: baseContextStore, + Store: store.New(config.ContextStoreDir(), cli.contextStoreConfig), Resolver: func() (*DefaultContext, error) { - return ResolveDefaultContext(opts, cli.contextStoreConfig) + return ResolveDefaultContext(cli.options, cli.contextStoreConfig) }, } - cli.currentContext = resolveContextName(opts, cli.configFile) - cli.dockerEndpoint, err = resolveDockerEndpoint(cli.contextStore, cli.currentContext) + cli.dockerEndpoint, err = resolveDockerEndpoint(cli.contextStore, resolveContextName(opts, cli.configFile)) if err != nil { return errors.Wrap(err, "unable to resolve docker endpoint") }