-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
rootless: add new netmode "slirp4netns" #1849
rootless: add new netmode "slirp4netns" #1849
Conversation
I've marked it as WIP as I want to work more on this PR and add tests |
So far LGTM |
2b27867
to
76c246f
Compare
c445205
to
c857072
Compare
c857072
to
d3e99eb
Compare
I wonder if we should open the possibility of using |
d3e99eb
to
01f8d1f
Compare
ab9fc8d
to
0f18bf2
Compare
Is slirp4netns useful for rootful mode? |
I think it is not. I've only wanted to make it possible to use, if the user wishes so, but I cannot find any reason why do it. |
Well I don't think we want it default, but would it be useful for running inside of a container which had not net-admin capabiltiies? |
probably yes. It should enable the use case of running podman as root without having |
0f18bf2
to
05117f6
Compare
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
so that inspect reports the correct network configuration. Closes: containers#1453 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
b01a578
to
12957d5
Compare
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
12957d5
to
95f22a2
Compare
all tests are green! |
LGTM |
LGTM |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vrothberg The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@rhatdan @vrothberg there are two LGTM, can someone do the |
LGTM and all happy green test buttons. |
/lgtm |
Ah. Didn't know one person can approve and do the lgtm. |
so that inspect reports the correct network configuration.
Closes: #1453
Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com