Skip to content
This repository was archived by the owner on Mar 9, 2022. It is now read-only.

Cherrypick #1156 release 1.2#1159

Merged
mikebrow merged 1 commit intocontainerd:release/1.2from
Random-Liu:cherrypick-#1156-release-1.2
Jun 10, 2019
Merged

Cherrypick #1156 release 1.2#1159
mikebrow merged 1 commit intocontainerd:release/1.2from
Random-Liu:cherrypick-#1156-release-1.2

Conversation

@Random-Liu
Copy link
Member

@Random-Liu Random-Liu commented Jun 7, 2019

When context is cancelled, the process.Kill won't work. We need to kill the process in the cleanup defer.

Cherrypick #1156 release 1.2

Signed-off-by: Lantao Liu lantaol@google.com

Signed-off-by: Lantao Liu <lantaol@google.com>
@Random-Liu Random-Liu added this to the v1.2 milestone Jun 7, 2019
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

/LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants