docker / cli

The Docker CLI
Apache License 2.0
4.97k stars 1.94k forks source link

--add-host example doesn't work as shown #5558

Open kodedylf opened 1 month ago

kodedylf commented 1 month ago

In the documentation at https://github.com/docker/cli/blob/master/docs/reference/commandline/container_run.md#add-host, a python3 web server is run to server the hello file.

However, as shown, the command will run in the foreground and the user will be unable to run the rest of the commands in the same terminal session. If the user presses ctrl-c to terminate python to be able to run the container, the example won't work since the web server isn't running.

My own solution was to run python3 in the background by adding & to the command. However then I needed to kill the web server after running the container. Getting the pid and killing python isn't very elegant. Hopefully you can come up with a better solution.

thaJeztah commented 1 month ago

Thanks for reporting!

Hm... yes, that's a good one; I know we tried to write up a minimal-as-possible example just to illustrate the concept, but indeed running things in the background can be slightly tricky, or at least to keep things "simple" as well as (where possible) working across platforms (mac, linux, windows).

I guess a "scrappy" workaround here could be to have the user open two separate shells; one to run the Python server and another one to run the docker commands to connect to it.

@neersighted @dvdksn any thoughts / ideas ?

arpitgupta-it commented 4 weeks ago

@thaJeztah How about using Docker Compose?

  1. Create a Dockerfile for the Python Server
    
    # Use the official Python image
    FROM python:3.9-slim

WORKDIR /usr/src/app

Copy the hello file into the image

COPY hello.txt .

Start the HTTP server

CMD ["python3", "-m", "http.server", "8000"]

2. Now create a file named docker-compose.yml

version: '3.8'

services: python-server: build: context: . ports:

thaJeztah commented 4 weeks ago

Right, but the host.docker.internal feature was added for users to have an option to connect to a services that's not running inside a container, so some service on the host. For the docker-compose case, the recommended approach would for the curl-client service to connect with the python-server service through the internal network (are least; "don't map ports that shouldn't be accessible publicly").