Merge pull request #32504 from kargakis/top-pod-fixes

Automatic merge from submit-queue

kubectl: run validation for 'top pod'

@mksalawa @kubernetes/kubectl
This commit is contained in:
Kubernetes Submit Queue 2016-09-14 05:35:42 -07:00 committed by GitHub
commit 33e5b0e0db
2 changed files with 6 additions and 0 deletions

View File

@ -122,6 +122,9 @@ func (o TopNodeOptions) RunTopNode() error {
if err != nil {
return err
}
if len(metrics) == 0 {
return errors.New("metrics not available yet")
}
var nodes []api.Node
if len(o.ResourceName) > 0 {

View File

@ -81,6 +81,9 @@ func NewCmdTopPod(f *cmdutil.Factory, out io.Writer) *cobra.Command {
if err := options.Complete(f, cmd, args, out); err != nil {
cmdutil.CheckErr(err)
}
if err := options.Validate(); err != nil {
cmdutil.CheckErr(cmdutil.UsageError(cmd, err.Error()))
}
if err := options.RunTopPod(); err != nil {
cmdutil.CheckErr(err)
}