diff --git a/pkg/kubectl/cmd/rollout/rollout_pause.go b/pkg/kubectl/cmd/rollout/rollout_pause.go index 9460e70397..51d8ea3faa 100644 --- a/pkg/kubectl/cmd/rollout/rollout_pause.go +++ b/pkg/kubectl/cmd/rollout/rollout_pause.go @@ -154,7 +154,9 @@ func (o PauseConfig) RunPause() error { allErrs = append(allErrs, err) continue } - printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out) + if err = printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out); err != nil { + allErrs = append(allErrs, err) + } continue } @@ -170,7 +172,9 @@ func (o PauseConfig) RunPause() error { allErrs = append(allErrs, err) continue } - printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out) + if err = printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out); err != nil { + allErrs = append(allErrs, err) + } } return utilerrors.NewAggregate(allErrs) diff --git a/pkg/kubectl/cmd/rollout/rollout_resume.go b/pkg/kubectl/cmd/rollout/rollout_resume.go index 3208bc3c2d..8ca44d8a76 100644 --- a/pkg/kubectl/cmd/rollout/rollout_resume.go +++ b/pkg/kubectl/cmd/rollout/rollout_resume.go @@ -159,7 +159,9 @@ func (o ResumeConfig) RunResume() error { allErrs = append(allErrs, err) continue } - printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out) + if err = printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out); err != nil { + allErrs = append(allErrs, err) + } } obj, err := resource.NewHelper(info.Client, info.Mapping).Patch(info.Namespace, info.Name, types.StrategicMergePatchType, patch.Patch) @@ -174,7 +176,9 @@ func (o ResumeConfig) RunResume() error { allErrs = append(allErrs, err) continue } - printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out) + if err = printer.PrintObj(cmdutil.AsDefaultVersionedOrOriginal(info.Object, info.Mapping), o.Out); err != nil { + allErrs = append(allErrs, err) + } } return utilerrors.NewAggregate(allErrs) diff --git a/pkg/kubectl/cmd/set/set_env.go b/pkg/kubectl/cmd/set/set_env.go index 7f91e186b0..d2ec28e1a7 100644 --- a/pkg/kubectl/cmd/set/set_env.go +++ b/pkg/kubectl/cmd/set/set_env.go @@ -482,7 +482,7 @@ func (o *EnvOptions) RunEnv() error { if o.Local || o.dryRun { if err := o.PrintObj(info.Object, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } continue } @@ -500,7 +500,7 @@ func (o *EnvOptions) RunEnv() error { } if err := o.PrintObj(actual, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } } return utilerrors.NewAggregate(allErrs) diff --git a/pkg/kubectl/cmd/set/set_image.go b/pkg/kubectl/cmd/set/set_image.go index 0f3269bb52..b1de0b7b71 100644 --- a/pkg/kubectl/cmd/set/set_image.go +++ b/pkg/kubectl/cmd/set/set_image.go @@ -276,7 +276,7 @@ func (o *SetImageOptions) Run() error { if o.Local || o.DryRun { if err := o.PrintObj(info.Object, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } continue } @@ -289,7 +289,7 @@ func (o *SetImageOptions) Run() error { } if err := o.PrintObj(actual, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } } return utilerrors.NewAggregate(allErrs) diff --git a/pkg/kubectl/cmd/set/set_resources.go b/pkg/kubectl/cmd/set/set_resources.go index 8a70ec56b4..a52dafcdfe 100644 --- a/pkg/kubectl/cmd/set/set_resources.go +++ b/pkg/kubectl/cmd/set/set_resources.go @@ -277,7 +277,7 @@ func (o *SetResourcesOptions) Run() error { if o.Local || o.DryRun { if err := o.PrintObj(info.Object, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } continue } @@ -289,7 +289,7 @@ func (o *SetResourcesOptions) Run() error { } if err := o.PrintObj(actual, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } } return utilerrors.NewAggregate(allErrs) diff --git a/pkg/kubectl/cmd/set/set_serviceaccount.go b/pkg/kubectl/cmd/set/set_serviceaccount.go index 9b40cf96a9..ab4e843a61 100644 --- a/pkg/kubectl/cmd/set/set_serviceaccount.go +++ b/pkg/kubectl/cmd/set/set_serviceaccount.go @@ -198,7 +198,7 @@ func (o *SetServiceAccountOptions) Run() error { } if o.local || o.dryRun { if err := o.PrintObj(info.Object, o.Out); err != nil { - return err + patchErrs = append(patchErrs, err) } continue } @@ -209,7 +209,7 @@ func (o *SetServiceAccountOptions) Run() error { } if err := o.PrintObj(actual, o.Out); err != nil { - return err + patchErrs = append(patchErrs, err) } } return utilerrors.NewAggregate(patchErrs) diff --git a/pkg/kubectl/cmd/set/set_subject.go b/pkg/kubectl/cmd/set/set_subject.go index 9a0cc736d5..3e01e00a36 100644 --- a/pkg/kubectl/cmd/set/set_subject.go +++ b/pkg/kubectl/cmd/set/set_subject.go @@ -251,7 +251,7 @@ func (o *SubjectOptions) Run(fn updateSubjects) error { if o.Local || o.DryRun { if err := o.PrintObj(info.Object, o.Out); err != nil { - return err + allErrs = append(allErrs, err) } continue }