implement --publish in docker-compose run

[beledouxdenis]

Documentation

docker-compose run options:
https://docs.docker.com/reference/cli/docker/compose/run/#options

-p, --publish Publish a container's port(s) to the host

Explanations

Using the below docker-compose.yml:

version: '3.8'

services:
  netcat-server:
    image: alpine:latest
    command: sh -c "apk add --no-cache netcat-openbsd && nc -lk -p 12345"

The below docker-compose run command:

docker-compose run -p 12345:12345 netcat-server

publishes the port 12345 from the container to the host:

$ docker ps -a
CONTAINER ID   IMAGE           COMMAND                  CREATED          STATUS          PORTS                                           NAMES
0adac614c107   alpine:latest   "sh -c 'apk add --no…"   29 seconds ago   Up 29 seconds   0.0.0.0:12345->12345/tcp, :::12345->12345/tcp   src-netcat-server-run-79dad3a4579c

while the same using podman-compose run:

podman-compose run -p 12345:12345 netcat-server

does not publish the port:

$ podman ps -a
CONTAINER ID  IMAGE                            COMMAND               CREATED        STATUS        PORTS       NAMES
acc4cd67b10a  docker.io/library/alpine:latest  sh -c apk add --n...  7 seconds ago  Up 8 seconds              src_netcat-server_tmp59130

The changes in this pull request aims to add the implementation of the --publish options in podman-compose run, which was completely forgotten. --publish was added to the parser arguments:

podman-compose/podman_compose.py

Lines 3002 to 3007 in 2681566

	parser.add_argument(
	"-p",
	"--publish",
	action="append",
	help="Publish a container's port(s) to the host (can be used multiple times)",
	)

but then never used later (no occurences of args.publish).

These changes allow the expected behavior:

$ podman ps -a
CONTAINER ID  IMAGE                            COMMAND               CREATED       STATUS       PORTS                     NAMES
60e58363e0c0  docker.io/library/alpine:latest  sh -c apk add --n...  1 second ago  Up 1 second  0.0.0.0:12345->12345/tcp  src_netcat-server_tmp20781

The new block if args.publish: is stategically put after the block if not args.service_ports:, which is emptying cnt['ports'] when --service-ports is not passed.
Otherwise the ports added by --publish would be immediately emptied when --service-ports is not passed, which is not what is expected :D.

[p12tic]

Thanks!

[beledouxdenis]

Thanks to you !