Merge pull request #39218 from luksa/kubectl_top_short_forms

Automatic merge from submit-queue Kubectl top now also accepts short forms for "node" and "pod" ("no", "po") **What this PR does / why we need it**: People are used to typing `kubectl get po` and will expect `kubectl top po` to also work (without having to type the full resource name).

Merge pull request #39218 from luksa/kubectl_top_short_forms
3a2492f4 · Kubernetes Submit Queue · GitHub · 80f07b47 · 764e8bd6 · 3a2492f4
Commit 3a2492f4 authored Dec 28, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 28, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

top_node.go pkg/kubectl/cmd/top_node.go +1 -1

top_pod.go pkg/kubectl/cmd/top_pod.go +1 -1

No files found.
--- a/pkg/kubectl/cmd/top_node.go
+++ b/pkg/kubectl/cmd/top_node.go
@@ -88,7 +88,7 @@ func NewCmdTopNode(f cmdutil.Factory, out io.Writer) *cobra.Command {
 				cmdutil.CheckErr(err)
 			}
 		},
-		Aliases: []string{"nodes"},
+		Aliases: []string{"nodes", "no"},
 	}
 	cmd.Flags().StringVarP(&options.Selector, "selector", "l", "", "Selector (label query) to filter on, supports '=', '==', and '!='.")
 	options.HeapsterOptions.Bind(cmd.Flags())

--- a/pkg/kubectl/cmd/top_pod.go
+++ b/pkg/kubectl/cmd/top_pod.go
@@ -90,7 +90,7 @@ func NewCmdTopPod(f cmdutil.Factory, out io.Writer) *cobra.Command {
 				cmdutil.CheckErr(err)
 			}
 		},
-		Aliases: []string{"pods"},
+		Aliases: []string{"pods", "po"},
 	}
 	cmd.Flags().StringVarP(&options.Selector, "selector", "l", "", "Selector (label query) to filter on, supports '=', '==', and '!='.")
 	cmd.Flags().BoolVar(&options.PrintContainers, "containers", false, "If present, print usage of containers within a pod.")