Merge pull request #65370 from deads2k/cli-78-delete-verb

Automatic merge from submit-queue (batch tested with PRs 65377, 63837, 65370, 65294, 65376). 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>. delete should tolerate a failed wait because of missing verbs The power and ability to delete does not imply the power and ability to watch. We correctly handled missing power (authz), but failed to account for ability (method not supported) @kubernetes/sig-cli-maintainers @soltysh ```release-note Tolerate missing watch permission when deleting a resource ```

Merge pull request #65370 from deads2k/cli-78-delete-verb
a3fd4528 · Kubernetes Submit Queue · GitHub · 32c3ffa1 · 2c514b5b · a3fd4528
Unverified Commit a3fd4528 authored Jun 22, 2018 by Kubernetes Submit Queue Committed by GitHub Jun 22, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

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

No files found.
--- a/pkg/kubectl/cmd/delete.go
+++ b/pkg/kubectl/cmd/delete.go
@@ -285,7 +285,7 @@ func (o *DeleteOptions) DeleteResult(r *resource.Result) error {
 		IOStreams:   o.IOStreams,
 	}
 	err = waitOptions.RunWait()
-	if errors.IsForbidden(err) {
+	if errors.IsForbidden(err) || errors.IsMethodNotSupported(err) {
 		// if we're forbidden from waiting, we shouldn't fail.
 		glog.V(1).Info(err)
 		return nil