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

Unable to launch x11docker when --network is specified #463

Closed
SimplicityGuy opened this issue Aug 6, 2022 · 6 comments
Closed

Unable to launch x11docker when --network is specified #463

SimplicityGuy opened this issue Aug 6, 2022 · 6 comments
Labels

Comments

@SimplicityGuy
Copy link

With this commit: e50feeb, it is no longer possible to run an image with x11dock when --network=... is specified by the user. Please revert or rework this change.

@SimplicityGuy
Copy link
Author

This was also missed in the release notes, requiring code inspection to find that --hostname was now being set.

mviereck added a commit that referenced this issue Aug 7, 2022
@mviereck mviereck added the bug label Aug 7, 2022
@mviereck
Copy link
Owner

mviereck commented Aug 7, 2022

Thank you for the report!
Sorry that this caused trouble for you. This change was just intended to be a minor convenience to make it easier seeing which container belongs to which open windows. As a quite minor feature I did not mention it in the release notes.
I have removed the --hostname code, it should work now again.
May I ask how exactly this caused trouble with --network?

@SimplicityGuy
Copy link
Author

Thanks for the fix! Using --hostname and --network is incompatible per Docker, when using --network=container:vpn. I get this error:

conflicting options: hostname and the network mode

See the Docker run reference:

Network: container
With the network set to container a container will share the network stack of another container. The other container’s name must be provided in the format of --network container:<name|id>. Note that --add-host --hostname --dns --dns search --dns-option and --mac-address are invalid in container netmode, and --publish --publish-all --expose are also invalid in container netmode.

@SimplicityGuy
Copy link
Author

So, given that, perhaps the fix here could be to use --hostname when --network is not a container?

@mviereck
Copy link
Owner

mviereck commented Aug 7, 2022

See the Docker run reference:

Oh, I should have looked there before. I did not imagine that this could have such a major impact.

So, given that, perhaps the fix here could be to use --hostname when --network is not a container?

That's a good idea! However, I think I won't mess with --hostname anymore, since the use case was quite minor.

@mviereck mviereck closed this as completed Aug 7, 2022
@SimplicityGuy
Copy link
Author

No worries, mate. Bugs happen. Appreciate the quick turnaround on this. The docker commands are all over the place. Mutually exclusive flags galore. Wait until you try using --filter on docker volumes... oh boy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants