app-214 Enabled validate only for experimental mode

Signed-off-by: Anca Iordache <[email protected]>

Author: Anca Iordache

internal/commands/root.go

@@ -48,7 +48,7 @@ func NewRootCmd(use string, dockerCli command.Cli) *cobra.Command {
 }
 
 func addCommands(cmd *cobra.Command, dockerCli command.Cli) {
-	cmd.AddCommand(
+	listOfCommands := []*cobra.Command{
 		runCmd(dockerCli),
 		updateCmd(dockerCli),
 		removeCmd(dockerCli),
@@ -59,8 +59,19 @@ func addCommands(cmd *cobra.Command, dockerCli command.Cli) {
 		pullCmd(dockerCli),
 		image.Cmd(dockerCli),
 		build.Cmd(dockerCli),
-		inspectCmd(dockerCli),
-	)
+	}
+
+	isExperimentalMode := dockerCli.ClientInfo().HasExperimental
+	for _, ccmd := range listOfCommands {
+		switch ccmd.Annotations["experimental"] {
+		case "true":
+			if isExperimentalMode {
+				cmd.AddCommand(ccmd)
+			}
+		default:
+			cmd.AddCommand(ccmd)
+		}
+	}
 }
 
 func firstOrEmpty(list []string) string {

internal/commands/validate.go

@@ -22,10 +22,11 @@ type validateOptions struct {
 func validateCmd() *cobra.Command {
 	var opts validateOptions
 	cmd := &cobra.Command{
-		Use:     "validate [OPTIONS] APP_DEFINITION",
-		Short:   "Check that an App definition (.dockerapp) is syntactically correct",
-		Example: `$ docker app validate myapp.dockerapp --set key=value --parameters-file myparam.yml`,
-		Args:    cli.RequiresMaxArgs(1),
+		Use:         "validate [OPTIONS] APP_DEFINITION",
+		Short:       "Check that an App definition (.dockerapp) is syntactically correct",
+		Example:     `$ docker app validate myapp.dockerapp --set key=value --parameters-file myparam.yml`,
+		Args:        cli.RequiresMaxArgs(1),
+		Annotations: map[string]string{"experimental": "true"},
 		RunE: func(cmd *cobra.Command, args []string) error {
 			return runValidate(args, opts)
 		},