Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete fails on notfound errors. #8628

Merged
merged 1 commit into from
May 27, 2015
Merged

Conversation

j3ffml
Copy link
Contributor

@j3ffml j3ffml commented May 21, 2015

If deleting multiple resources, command will continue on error and
report not found resources at the end. You can ignore not found errors (previous behavior) by passing --ignore-not-found.

Closes #8218

@k8s-bot
Copy link

k8s-bot commented May 21, 2015

Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist")

If this message is too spammy, please complain @ixdy.

@j3ffml
Copy link
Contributor Author

j3ffml commented May 21, 2015

ok to test

@dchen1107 dchen1107 self-assigned this May 22, 2015
@dchen1107
Copy link
Member

LGTM

@dchen1107 dchen1107 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 22, 2015
@thockin
Copy link
Member

thockin commented May 27, 2015

rebase

@j3ffml j3ffml force-pushed the delete-not-found branch 2 times, most recently from bfacc92 to 9e4fb14 Compare May 27, 2015 17:12
@thockin
Copy link
Member

thockin commented May 27, 2015

Fails CI

If deleting multiple resources, command will continue on error and
report not found resources at the end.
@j3ffml
Copy link
Contributor Author

j3ffml commented May 27, 2015

Whoops, I accidentally rebased/pushed from an older repo. Rebased the correct one now. PTAL.

thockin added a commit that referenced this pull request May 27, 2015
@thockin thockin merged commit eab1cec into kubernetes:master May 27, 2015
@j3ffml j3ffml deleted the delete-not-found branch May 29, 2015 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kubectl delete doesn't return error or print resource not found
5 participants