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

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

Open
kodedylf opened this issue Oct 20, 2024 · 3 comments
Open

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

kodedylf opened this issue Oct 20, 2024 · 3 comments

Comments

@kodedylf
Copy link

kodedylf commented Oct 20, 2024

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
Copy link
Member

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
Copy link

arpitgupta-it commented Nov 1, 2024

@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"]
  1. Now create a file named docker-compose.yml
version: '3.8'

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

  curl-client:
    image: curlimages/curl
    depends_on:
      - python-server
    entrypoint: ["curl", "-s", "host.docker.internal:8000/hello"]
    extra_hosts:
      - "host.docker.internal:host-gateway"
  1. Build and Run the Containers
docker-compose up --build

@thaJeztah
Copy link
Member

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

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

No branches or pull requests

3 participants