Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uid is required to switch netns #70

Merged
merged 3 commits into from Jan 27, 2017
Merged

Conversation

pyama86
Copy link
Contributor

@pyama86 pyama86 commented Jan 27, 2017

hi!
Fixed as netns did not work when attach command haconiwa
Always next to you!!

@pyama86 pyama86 changed the base branch from master to 0.7.x-dev January 27, 2017 03:48
flag = base.namespace.to_flag & (~(::Namespace::CLONE_NEWPID))
::Namespace.setns(flag, pid: base.pid)
flag = base.namespace.to_flag_without_pid_and_user
::Namespace.setns(flag, pid: base.pi:)
Copy link
Member

Choose a reason for hiding this comment

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

typo...

@udzura
Copy link
Member

udzura commented Jan 27, 2017

I wander what is the difference...

This would unshare new USER namespace at https://github.com/pyama86/haconiwa/blob/46bf4f98c4a277e31c9c4c0bc63196cbcf5d4796/mrblib/haconiwa/runner.rb#L272 and then create yet another namespace, I guess.

@udzura
Copy link
Member

udzura commented Jan 27, 2017

This PR is including cgroupv2 diff (#69). Can you rebase to current master and force-push it?

@pyama86 pyama86 force-pushed the fix-net-ns branch 2 times, most recently from 2580f01 to 1dab28a Compare January 27, 2017 08:43
@udzura udzura merged commit 4606581 into haconiwa:0.7.x-dev Jan 27, 2017
udzura added a commit that referenced this pull request Jan 27, 2017
udzura added a commit that referenced this pull request Jan 27, 2017
udzura added a commit that referenced this pull request Apr 13, 2017
Uid is required to switch netns
udzura added a commit that referenced this pull request Apr 13, 2017
udzura added a commit that referenced this pull request Apr 13, 2017
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.

None yet

2 participants