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

[WIP] Replacing nsenter with netns #139

Closed
wants to merge 1 commit into from
Closed

Conversation

abhi
Copy link
Member

@abhi abhi commented Aug 17, 2017

This commit has changes to replace the usage of nsenter with
netns. This can relax the requirement of requiring nsenter
installation.

Signed-off-by: Abhinandan Prativadi abhi@docker.com

This commit has changes to replace the usage of nsenter with
netns. This can relax the requirement of requiring nsenter
installation.

Signed-off-by: Abhinandan Prativadi <abhi@docker.com>
@abhi
Copy link
Member Author

abhi commented Aug 17, 2017

Will update doc if we want to take this path

@Random-Liu
Copy link
Member

Random-Liu commented Aug 17, 2017

Need to figure out how to solve vishvananda/netns#17 and golang/go#20676.

This seems to be a WIP then. :)

@abhi abhi changed the title Replacing nsenter with netns [WIP] Replacing nsenter with netns Aug 18, 2017
@Random-Liu
Copy link
Member

Random-Liu commented Aug 18, 2017

The golang issue will be solved in golang 1.10 (hopefully). At that time, we could use NetNS.Do to run code inside the network namespace.

We could get rid of all the nsenters at that time.

@abhi
Copy link
Member Author

abhi commented Aug 18, 2017

Woot Woot ! Thanks for the update @Random-Liu

@abhi abhi closed this Mar 16, 2018
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