From 2f485ec84b0d02faac1ca3c4e93cd0e5a2b1a4aa Mon Sep 17 00:00:00 2001 From: Shijiang Wei Date: Wed, 28 Mar 2018 11:25:55 +0800 Subject: [PATCH] kubectl: fix a panic when createGeneratedObject failed Reproduce: kubectl run --image=alpine hello -- sleep 1d kubectl run --image=alpine --stdin --tty --attach hello -- /bin/sh Signed-off-by: Shijiang Wei --- pkg/kubectl/cmd/run.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/kubectl/cmd/run.go b/pkg/kubectl/cmd/run.go index 59eca16bac9..f0ffeae233d 100644 --- a/pkg/kubectl/cmd/run.go +++ b/pkg/kubectl/cmd/run.go @@ -276,14 +276,14 @@ func RunRun(f cmdutil.Factory, cmdIn io.Reader, cmdOut, cmdErr io.Writer, cmd *c params["env"] = cmdutil.GetFlagStringArray(cmd, "env") - allErrs := []error{} var runObjectMap = map[string]*RunObject{} runObject, err := createGeneratedObject(f, cmd, generator, names, params, cmdutil.GetFlagString(cmd, "overrides"), namespace) if err != nil { - allErrs = append(allErrs, err) + return err } else { runObjectMap[generatorName] = runObject } + allErrs := []error{} if cmdutil.GetFlagBool(cmd, "expose") { serviceGenerator := cmdutil.GetFlagString(cmd, "service-generator") if len(serviceGenerator) == 0 {