-
Notifications
You must be signed in to change notification settings - Fork 18.9k
Allow to enable host loopback and use 10.0.2.2 to connect to the host (OPTIONALLY) #47352
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
Conversation
3c1b8bd to
16ab43d
Compare
|
@AkihiroSuda Hello, can you look at it one more time? |
|
Looks good but please squash the commits |
69be0c0 to
86a0829
Compare
|
@AkihiroSuda OK, now it squashed. |
AkihiroSuda
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
|
Thank you too |
|
Hello @AkihiroSuda does it will be merged or I need something change? |
|
@akerouanton @robmry PTAL |
robmry
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In "Description for the changelog", defaults false needs to be changed to defaults to true.
Otherwise, LGTM.
… (OPTIONALLY) This should allow to enable host loopback by setting DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK to false, defaults true. Signed-off-by: serhii.n <serhii.n@thescimus.com>
86a0829 to
b649e27
Compare
|
@robmry I have updated "Description for the changelog" |
akerouanton
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
thaJeztah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Thanks to all of you! |
c.f. `DOCKERD_ROOTLESS_ROOTLESSKIT_NET=slirp4netns` and `DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK=false` > rootless: Add support for enabling host loopback by setting the `DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK` environment variable to `false` (defaults to `true`). This lets containers connect to the host by using IP address 10.0.2.2. [moby/moby#47352](moby/moby#47352) > > _-- https://docs.docker.com/engine/release-notes/26.0/_ > ``` > # * DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK=(true|false): prohibit connections to 127.0.0.1 on the host (including via 10.0.2.2, in the case of slirp4netns). Defaults to "true". > ``` > > *-- https://github.com/moby/moby/blob/b649e272bb0253fe766069966416b9330a2e38c0/contrib/dockerd-rootless.sh#L18*


- What I did
I allow to enable host loopback by setting DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK to false,
- How I did it
...
- How to verify it
Set DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK to false and it should allow connections to host by using 10.0.2.2 IP
- Description for the changelog
Allow to enable host loopback by setting DOCKERD_ROOTLESS_ROOTLESSKIT_DISABLE_HOST_LOOPBACK to false, defaults true. It allow to connect to host by using 10.0.2.2 IP
- A picture of a cute animal (not mandatory but encouraged)