ref: use plugins part of config file

To avoid any conflict with the CLI, use the `plugins` section of the
config. This section is read and saved by the CLI without the risk to
remove the value in there. So it's a safe way to deal with this feature.

As it's a cross plugin configuration (now for scout but goal is wider
than that) then put it under a generic `cliHints` name so that other
plugins might use it if needed.

Signed-off-by: Yves Brissaud <[email protected]>

Author: eunomie

api/config/config.go

@@ -100,6 +100,6 @@ func configFilePath(dir string) string {
 
 // File contains the current context from the docker configuration file
 type File struct {
-	CurrentContext string `json:"currentContext,omitempty"`
-	CliHints       *bool  `json:"cliHints,omitempty"`
+	CurrentContext string                       `json:"currentContext,omitempty"`
+	Plugins        map[string]map[string]string `json:"plugins,omitempty"`
 }

cli/mobycli/cli_hints.go

@@ -26,6 +26,9 @@ import (
 const (
 	cliHintsEnvVarName       = "DOCKER_CLI_HINTS"
 	cliHintsDefaultBehaviour = true
+
+	cliHintsPluginName  = "cliHints"
+	cliHintsEnabledName = "enabled"
 )
 
 func CliHintsEnabled() bool {
@@ -40,8 +43,12 @@ func CliHintsEnabled() bool {
 		// can't read the config file, use the default behaviour
 		return cliHintsDefaultBehaviour
 	}
-	if conf.CliHints != nil {
-		return *conf.CliHints
+	if cliHintsPluginConfig, ok := conf.Plugins[cliHintsPluginName]; ok {
+		if cliHintsValue, ok := cliHintsPluginConfig[cliHintsEnabledName]; ok {
+			if cliHints, err := strconv.ParseBool(cliHintsValue); err == nil {
+				return cliHints
+			}
+		}
 	}
 
 	return cliHintsDefaultBehaviour

cli/mobycli/cli_hints_test.go

@@ -66,6 +66,14 @@ func TestCliHintsEnabled(t *testing.T) {
 			},
 			true,
 		},
+		{
+			"plugin defined in config file but no enabled entry",
+			func() {
+				d := testConfigDir(t)
+				writeSampleConfig(t, d, configPartial)
+			},
+			true,
+		},
 		{
 			"disabled in config file",
 			func() {
@@ -120,9 +128,24 @@ func writeSampleConfig(t *testing.T, d string, conf []byte) {
 }
 
 var configEnabled = []byte(`{
-	"cliHints": true
+	"plugins": {
+		"cliHints": {
+			"enabled": "true"
+		}
+	}
 }`)
 
 var configDisabled = []byte(`{
-	"cliHints": false
+	"plugins": {
+		"cliHints": {
+			"enabled": "false"
+		}
+	}
+}`)
+
+var configPartial = []byte(`{
+	"plugins": {
+		"cliHints": {
+		}
+	}
 }`)