feat(plugin): add flags shorthand like kubectl

cmd/plugin/commands/info/info.go

@@ -29,21 +29,16 @@ import (
 
 // CreateCommand creates and returns this cobra subcommand
 func CreateCommand(flags *genericclioptions.ConfigFlags) *cobra.Command {
+	var service *string
 	cmd := &cobra.Command{
 		Use:   "info",
 		Short: "Show information about the ingress-nginx service",
 		RunE: func(cmd *cobra.Command, args []string) error {
-			service, err := cmd.Flags().GetString("service")
+			util.PrintError(info(flags, *service))
-			if err != nil {
-				return err
-			}
-
-			util.PrintError(info(flags, service))
 			return nil
 		},
 	}
-
+	service = util.AddServiceFlag(cmd)
-	cmd.Flags().String("service", util.DefaultIngressServiceName, "The name of the ingress-nginx service")
 	return cmd
 }
 

cmd/plugin/commands/ingresses/ingresses.go

@@ -32,6 +32,7 @@ import (
 
 // CreateCommand creates and returns this cobra subcommand
 func CreateCommand(flags *genericclioptions.ConfigFlags) *cobra.Command {
+	var allNamespaces bool
 	cmd := &cobra.Command{
 		Use:     "ingresses",
 		Aliases: []string{"ingress", "ing"},
@@ -42,18 +43,12 @@ func CreateCommand(flags *genericclioptions.ConfigFlags) *cobra.Command {
 				return err
 			}
 
-			allNamespaces, err := cmd.Flags().GetBool("all-namespaces")
-			if err != nil {
-				return err
-			}
-
 			util.PrintError(ingresses(flags, host, allNamespaces))
 			return nil
 		},
 	}
 	cmd.Flags().String("host", "", "Show just the ingress definitions for this hostname")
-	cmd.Flags().Bool("all-namespaces", false, "Find ingress definitions from all namespaces")
+	cmd.Flags().BoolVarP(&allNamespaces, "all-namespaces", "A", false, "Find ingress definitions from all namespaces")
-
 	return cmd
 }
 

cmd/plugin/commands/lint/main.go

@@ -92,7 +92,7 @@ func addCommonOptions(flags *genericclioptions.ConfigFlags, cmd *cobra.Command)
 	out := lintOptions{
 		flags: flags,
 	}
-	cmd.Flags().BoolVar(&out.allNamespaces, "all-namespaces", false, "Check resources in all namespaces")
+	cmd.Flags().BoolVarP(&out.allNamespaces, "all-namespaces", "A", false, "Check resources in all namespaces")
 	cmd.Flags().BoolVar(&out.showAll, "show-all", false, "Show all resources, not just the ones with problems")
 	cmd.Flags().BoolVarP(&out.verbose, "verbose", "v", false, "Show extra information about the lints")
 	cmd.Flags().StringVarP(&out.versionFrom, "from-version", "f", "0.0.0", "Use lints added for versions starting with this one")

cmd/plugin/util/util.go

@@ -135,6 +135,13 @@ func AddContainerFlag(cmd *cobra.Command) *string {
 	return &v
 }
 
+// AddServiceFlag adds a --service flag to a cobra command
+func AddServiceFlag(cmd *cobra.Command) *string {
+	v := ""
+	cmd.Flags().StringVar(&v, "service", DefaultIngressServiceName, "The name of the ingress-nginx service")
+	return &v
+}
+
 // GetNamespace takes a set of kubectl flag values and returns the namespace we should be operating in
 func GetNamespace(flags *genericclioptions.ConfigFlags) string {
 	namespace, _, err := flags.ToRawKubeConfigLoader().Namespace()