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
Add WithProcessKill DeleteOpt #1301
Conversation
t.Error(err) | ||
return | ||
} | ||
if _, err := task.Delete(ctx); err == nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One delete too many
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm testing before and after with the option
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, read the condition wrong, my bad 👍
d6abb27
to
bbbad93
Compare
LGTM. Thanks! |
LGTM |
bbbad93
to
33eaab9
Compare
Add an option that allows users for force kill and delete a process/task when calling `Delete` Fixes containerd#1274 Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
33eaab9
to
d8c075a
Compare
Fixed the checkpoint tests that were broken after this change. |
Codecov Report
@@ Coverage Diff @@
## master #1301 +/- ##
=======================================
Coverage 33.96% 33.96%
=======================================
Files 24 24
Lines 2723 2723
=======================================
Hits 925 925
Misses 1598 1598
Partials 200 200 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add an option that allows users for force kill and delete a process/task
when calling
Delete
Fixes #1274
Signed-off-by: Michael Crosby crosbymichael@gmail.com