Consolidate env vars + fix minion under compose

Since we're not deploying this, we can try to set environment variables
in fewer places.

Author: oalders

.env

@@ -1,4 +1,15 @@
-COMPOSE_PROJECT_NAME=metacpan
+API_SERVER="morbo -l http://*:5000 -w app.psgi -w bin -w lib -w templates --verbose"
+COLUMNS=80
+ES=elasticsearch:9200
+ES_TEST=elasticsearch_test:9200
+MINICPAN=/CPAN
+MOJO_MODE=development
+NET_ASYNC_HTTP_MAXCONNS=1
+PERL_CARTON_PATH=/carton
+PERL_MM_USE_DEFAULT=1
+PG_DATABASE=metacpan
+PG_HOST=pghost
+PG_PASSWORD=t00lchain
+PG_PORT=5432
+PG_USER=metacpan
 PLACK_ENV=development
-PGDB=pgdb:5432
-API_SERVER=morbo -l http://*:5000 -w app.psgi -w bin -w lib -w templates --verbose

docker-compose.yml

@@ -1,5 +1,6 @@
 ---
-
+x-project:
+  name: metacpan
 #  ____  _____ ______     _____ ____ _____ ____
 # / ___|| ____|  _ \ \   / /_ _/ ___| ____/ ___|
 # \___ \|  _| | |_) \ \ / / | | |   |  _| \___ \
@@ -100,16 +101,18 @@ services:
   api:
     depends_on:
       - elasticsearch
-      - pgdb
+      - pghost
       - traefik
     image: metacpan/metacpan-api:latest
     build:
       context: ./src/metacpan-api
+    # put variables for compose inside a .env file
+    # use env_file for variables to be set inside the container
     env_file:
-      - localapi.env
+      - .env
     command: >
       /metacpan-api/wait-for-es.sh http://elasticsearch:9200 "" --
-      /metacpan-api/wait-for-it.sh -t 15 -s ${PGDB} --
+      /metacpan-api/wait-for-it.sh -t 15 -s -h ${PG_HOST} -p ${PG_PORT} --
       ${API_SERVER} ./bin/api.pl
     volumes:
       - type: volume
@@ -154,15 +157,15 @@ services:
   api_test:
     depends_on:
       - elasticsearch_test
-      - pgdb
+      - pghost
     image: metacpan/metacpan-api:latest
     build:
       context: ./src/metacpan-api
     env_file:
       - localapi_test.env
     command: >
       /metacpan-api/wait-for-es.sh http://elasticsearch_test:9200 "" --
-      /metacpan-api/wait-for-it.sh -t 15 -s ${PGDB} --
+      /metacpan-api/wait-for-it.sh -t 15 -s -h ${PG_HOST} -p ${PG_PORT} --
       ${API_SERVER} ./bin/api.pl
     volumes:
       - type: volume
@@ -315,16 +318,16 @@ services:
   # |_|                 |___/                  |_|
   #
 
-  pgdb:
-    hostname: pgdb
+  pghost:
+    hostname: pghost
     image: "postgres:${PG_VERSION_TAG:-9.6-alpine}"
     build:
       context: "./pg"
       args:
         PG_TAG: "${PG_VERSION_TAG:-9.6-alpine}"
     environment:
       POSTGRES_PASSWORD: metacpan
-      POSTGRES_USERNAME: metacpan123
+      POSTGRES_USER: metacpan123
       POSTGRES_DB: metacpan
     networks:
       - database
@@ -336,7 +339,7 @@ services:
       test: ["CMD", "/healthcheck.sh"]
     volumes:
       - type: volume
-        source: pgdb-data
+        source: pghost-data
         target: /var/lib/postgresql/data
       - type: bind
         source: ./pg/docker-entrypoint-initdb.d
@@ -373,6 +376,6 @@ volumes:
   cpan:
   elasticsearch:
   elasticsearch_test:
-  pgdb-data:
+  pghost-data:
   metacpan_git_shared:
     external: true

localapi.env

@@ -1,4 +1,4 @@
-CREATE ROLE metacpan WITH LOGIN PASSWORD 'metacpan';
+CREATE ROLE metacpan WITH LOGIN PASSWORD 't00lchain';
 CREATE ROLE "metacpan-api" WITH LOGIN;
 
 -- make things easier for when we're poking around from inside the container