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

Allow to enable host loopback and use 10.0.2.2 to connect to the host (OPTIONALLY) #47352

Merged
merged 1 commit into from Feb 29, 2024

Conversation

serhii-nakon
Copy link
Contributor

@serhii-nakon serhii-nakon commented Feb 7, 2024

- 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)

@serhii-nakon
Copy link
Contributor Author

@AkihiroSuda Hello, can you look at it one more time?

@AkihiroSuda
Copy link
Member

Looks good but please squash the commits

@serhii-nakon
Copy link
Contributor Author

@AkihiroSuda OK, now it squashed.

Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@serhii-nakon
Copy link
Contributor Author

Thank you too

@serhii-nakon
Copy link
Contributor Author

Hello @AkihiroSuda does it will be merged or I need something change?

@thaJeztah
Copy link
Member

@akerouanton @robmry PTAL

Copy link
Contributor

@robmry robmry left a 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.

contrib/dockerd-rootless.sh Outdated Show resolved Hide resolved
… (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>
@serhii-nakon
Copy link
Contributor Author

@robmry I have updated "Description for the changelog"

Copy link
Member

@akerouanton akerouanton left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah merged commit b8aa857 into moby:master Feb 29, 2024
128 checks passed
@thaJeztah thaJeztah added this to the 26.0.0 milestone Feb 29, 2024
@serhii-nakon
Copy link
Contributor Author

Thanks to all of you!

@serhii-nakon serhii-nakon deleted the allow_host_loopback branch March 5, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants