Merge pull request #58030 from ahmetb/fix-fmt

Automatic merge from submit-queue (batch tested with PRs 57581, 58030). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. cmd/kubectl: fix broken error formatting for run This patch adds missing value to a format string (%s) in --restart flag validation for "kubectl run". "kubectl run --restart=foo" was giving error: error: invalid restart policy: %!s(MISSING) Now it says: error: invalid restart policy: foo /release-note-none

Merge pull request #58030 from ahmetb/fix-fmt
4989087f · Kubernetes Submit Queue · GitHub · 39ea0d51 · ee035132 · 4989087f
Unverified Commit 4989087f authored Jan 09, 2018 by Kubernetes Submit Queue Committed by GitHub Jan 09, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

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

No files found.
--- a/pkg/kubectl/cmd/run.go
+++ b/pkg/kubectl/cmd/run.go
@@ -501,7 +501,7 @@ func getRestartPolicy(cmd *cobra.Command, interactive bool) (api.RestartPolicy, 
 	case api.RestartPolicyNever:
 		return api.RestartPolicyNever, nil
 	}
-	return "", cmdutil.UsageErrorf(cmd, "invalid restart policy: %s")
+	return "", cmdutil.UsageErrorf(cmd, "invalid restart policy: %s", restart)
 }

 func verifyImagePullPolicy(cmd *cobra.Command) error {