Skip to content

Conversation

@eguzki
Copy link
Member

@eguzki eguzki commented Jun 6, 2022

what

  • apicast-source target not needed for cleaning up clean-containers
    • no need to have a container (running or stopped) to use docker-compose.yaml configuration
    • makes easier to support podman. But currently I am not sure podman can replace docker, I haven't tested.
  • prove service from docker-compose.yaml removed and created a dedicated docker-compose.prove.yml. That configuration is being used in make prove-docker command. Used to run integration tests without running the development image (make development)
  • clean-containers makefile target cleans all docker compose environments there could be up and running.

Verification steps

Run integration tests in docker

make prove-docker

Clean running environment(s)

make clean-containers

check apicast_build_0_prove and apicast_build_0_redis have been cleaned.

docker ps -a | grep apicast_build_0
(empty)

@eguzki eguzki requested a review from a team as a code owner June 6, 2022 19:22
@samugi
Copy link
Contributor

samugi commented Jun 6, 2022

the steps described work as expected. Anything specific to podman that can be tested here?

@eguzki
Copy link
Member Author

eguzki commented Jun 7, 2022

I would go over the makefile targets

make development then run busted and prove -> hits docker-compose-devel.yml
make prove-docker -> hits docker-compose.prove.yml
make runtime-image
make test-runtime-image -> hits docker-compose.yaml
make dev-build

If they work with podman, I would say that podman is supported.

@eguzki eguzki force-pushed the fix-prove-docker branch 2 times, most recently from 659bbee to 7f49da0 Compare June 7, 2022 10:57
@eguzki eguzki force-pushed the fix-prove-docker branch from 7f49da0 to 34d5021 Compare June 8, 2022 10:18
@samugi
Copy link
Contributor

samugi commented Jun 9, 2022

I would go over the makefile targets

make development then run busted and prove -> hits docker-compose-devel.yml make prove-docker -> hits docker-compose.prove.yml make runtime-image make test-runtime-image -> hits docker-compose.yaml make dev-build

If they work with podman, I would say that podman is supported.

so I reverted this and then ran all the commands which executed just fine, however when I do podman ps I see no running containers, they populate the docker ps output, is that expected? I can see in the output of commands that it is actually using podman, e.g.:

$ make runtime-image                                                                                                                             
/usr/bin/podman  build -t apicast-runtime-image:latest

@eguzki
Copy link
Member Author

eguzki commented Jun 10, 2022

I would expect podman ps to return running containers 🤷

I would say, let's investigate podman support in other PR before enabling podman with

DOCKER ?= $(shell which podman 2> /dev/null || which docker 2> /dev/null || echo "docker")

Copy link
Contributor

@samugi samugi left a comment

Choose a reason for hiding this comment

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

lgtm, we can investigate podman support separately

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants