Skip to content

v2: Close ttrpc connection when Delete()#3496

Merged
fuweid merged 1 commit intocontainerd:masterfrom
darfux:v2_close_ttrpc_when_delete
Aug 6, 2019
Merged

v2: Close ttrpc connection when Delete()#3496
fuweid merged 1 commit intocontainerd:masterfrom
darfux:v2_close_ttrpc_when_delete

Conversation

@darfux
Copy link
Copy Markdown
Contributor

@darfux darfux commented Aug 6, 2019

This avoids potential socket leak when the connected v2 shim of runtime
serving multiple containers.

Signed-off-by: Li Yuxuan [email protected]

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 6, 2019

Build succeeded.

@darfux darfux force-pushed the v2_close_ttrpc_when_delete branch from 2c59006 to 282e1a1 Compare August 6, 2019 08:27
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 6, 2019

Build succeeded.

@darfux darfux closed this Aug 6, 2019
@darfux darfux reopened this Aug 6, 2019
@darfux darfux force-pushed the v2_close_ttrpc_when_delete branch from 282e1a1 to 0edfe48 Compare August 6, 2019 09:18
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 6, 2019

Build succeeded.

@darfux darfux force-pushed the v2_close_ttrpc_when_delete branch from 0edfe48 to 0c7950c Compare August 6, 2019 10:47
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 6, 2019

Build succeeded.

This avoids potential socket leak when the connected v2 shim of runtime
serving multiple containers.

Signed-off-by: Li Yuxuan <[email protected]>
@darfux darfux force-pushed the v2_close_ttrpc_when_delete branch from 0c7950c to 08483d1 Compare August 6, 2019 12:36
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Aug 6, 2019

Build succeeded.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

@fuweid fuweid merged commit 7fddefc into containerd:master Aug 6, 2019
@darfux darfux deleted the v2_close_ttrpc_when_delete branch August 7, 2019 03:12
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