Skip to content

Commit

Permalink
CV2-3435 update ports and expose so that it actually works across the…
Browse files Browse the repository at this point in the history
… container and move env file to allow for check tests
  • Loading branch information
DGaffney committed Sep 13, 2023
1 parent 6809ca0 commit 1a95048
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 3 deletions.
4 changes: 3 additions & 1 deletion local.env → .env_file.test
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
PRESTO_PORT=8000
DEPLOY_ENV=local
MODEL_NAME=mean_tokens.Model
# MODEL_NAME=mean_tokens.Model
MODEL_NAME=audio.Model
AWS_ACCESS_KEY_ID=SOMETHING
AWS_SECRET_ACCESS_KEY=OTHERTHING
7 changes: 6 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
ARG PRESTO_PORT

FROM python:3.9
EXPOSE 8000

ENV PRESTO_PORT=${PRESTO_PORT}
EXPOSE ${PRESTO_PORT}

WORKDIR /app
ENV DEBIAN_FRONTEND=noninteractive

Expand Down
6 changes: 5 additions & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,17 @@ services:
platform: linux/amd64
build: .
env_file:
- ./local.env
- ./.env_file
depends_on:
- elasticmq
links:
- elasticmq
volumes:
- ./:/app
ports:
- "${PRESTO_PORT}:${PRESTO_PORT}"
args:
- PRESTO_PORT=${PRESTO_PORT}
elasticmq:
image: softwaremill/elasticmq
hostname: presto-elasticmq
Expand Down

0 comments on commit 1a95048

Please sign in to comment.