metacpan / metacpan-docker

DEVELOPMENT Docker configs for MetaCPAN
Artistic License 2.0
26 stars 23 forks source link

Fix container definitions #74

Closed ssoriche closed 3 years ago

ssoriche commented 3 years ago

Removing the api_test container, as tests should be executable from the api container. In order to do so however, there needs to be 2 ES containers, one for running and one for testing. To facilitate this removing the loccalhost port binding definition for the ElasticSearch containers. Communication with the containers is done over the elasticsearch network, and the host communication is still available over the dynamically assigned port.

There was also an issue starting the postgres container because the image now requires a password to be set for the postgres user. This issue is now fixed.

oalders commented 3 years ago

So, how do we run the tests now?