Skip to content

[release/1.2] when kill a container check if it has been deleted#4121

Merged
crosbymichael merged 1 commit intocontainerd:release/1.2from
payall4u:hotfix-delete-container-error
Apr 1, 2020
Merged

[release/1.2] when kill a container check if it has been deleted#4121
crosbymichael merged 1 commit intocontainerd:release/1.2from
payall4u:hotfix-delete-container-error

Conversation

@payall4u
Copy link
Copy Markdown
Contributor

From #4020 . When use cri-containerd and k8s, pod may be stuck in terminating. It happens when container is delete by runc, however containerd unawares of this or deletes container failed. When cri retry to delete container it will got a runc error container <cid> does not exist. This is because cri ignore IsNotFound error, but shim is not converting the runc error to a containerd NotFound error.

@mxpv
Copy link
Copy Markdown
Member

mxpv commented Mar 31, 2020

Could you please sign off your commits (try git commit -s) in order to pass CI checks?
https://github.com/containerd/project/blob/master/CONTRIBUTING.md#sign-your-work

@payall4u payall4u force-pushed the hotfix-delete-container-error branch from 672d499 to f8be3cf Compare March 31, 2020 02:56
@payall4u
Copy link
Copy Markdown
Contributor Author

payall4u commented Mar 31, 2020

Could you please sign off your commits (try git commit -s) in order to pass CI checks?
https://github.com/containerd/project/blob/master/CONTRIBUTING.md#sign-your-work

@mxpv
Sorry, and done.

Copy link
Copy Markdown
Member

@mxpv mxpv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit b3b1ef3 into containerd:release/1.2 Apr 1, 2020
@fuweid fuweid changed the title when kill a container check if it has been deleted [release/1.2] when kill a container check if it has been deleted Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants