Skip to content

[release/1.5] runc.v1/v2: return init pid when clean dead shim#6570

Merged
mxpv merged 1 commit intocontainerd:release/1.5from
fuweid:cp-6452
Feb 21, 2022
Merged

[release/1.5] runc.v1/v2: return init pid when clean dead shim#6570
mxpv merged 1 commit intocontainerd:release/1.5from
fuweid:cp-6452

Conversation

@fuweid
Copy link
Copy Markdown
Member

@fuweid fuweid commented Feb 20, 2022

If containerd-shim-runc-v1/v2 process dead abnormally, such as received
kill 9 signal, panic or other unkown reasons, the containerd-shim-runc-v1/v2
server can not reap runc container and forward init process exit event.
This will lead the container leaked in dockerd. When shim dead, containerd
will clean dead shim, here read init process pid and forward exit event
with pid at the same time.

Signed-off-by: Jeff Zvier zvier20@gmail.com
Signed-off-by: Wei Fu fuweid89@gmail.com


Cherry-pick from #6452 and fixes runc.v1 as well

If containerd-shim-runc-v1/v2 process dead abnormally, such as received
kill 9 signal, panic or other unkown reasons, the containerd-shim-runc-v1/v2
server can not reap runc container and forward init process exit event.
This will lead the container leaked in dockerd. When shim dead, containerd
will clean dead shim, here read init process pid and forward exit event
with pid at the same time.

Signed-off-by: Jeff Zvier <zvier20@gmail.com>
Signed-off-by: Wei Fu <fuweid89@gmail.com>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Feb 20, 2022

Build succeeded.

@fuweid
Copy link
Copy Markdown
Member Author

fuweid commented Feb 20, 2022

/retest

@AkihiroSuda
Copy link
Copy Markdown
Member

Marking as a draft, as #6571 is not merged yet

@AkihiroSuda AkihiroSuda marked this pull request as draft February 21, 2022 02:38
@fuweid fuweid marked this pull request as ready for review February 21, 2022 05:39
Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@mxpv mxpv merged commit 2cbf075 into containerd:release/1.5 Feb 21, 2022
@fuweid fuweid deleted the cp-6452 branch February 21, 2022 22:50
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.

4 participants