Skip to content

Commit

Permalink
Switch the compose file to use images built by CI
Browse files Browse the repository at this point in the history
I've left the test image to build the local application so that it can
specify the "dev" multistage build target. Otherwise, it won't be able
to run tests.
  • Loading branch information
ian-noaa committed Jan 4, 2024
1 parent c9653ff commit 8f7030d
Showing 1 changed file with 5 additions and 13 deletions.
18 changes: 5 additions & 13 deletions compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,17 @@ secrets:

services:
shell:
build:
context: .
dockerfile: docker/ingest/Dockerfile
target: prod
image: ghcr.io/noaa-gsl/vxingest/ingest:main
volumes:
- ${data}:/opt/data
- ${public}:/public
secrets:
- CREDENTIALS_FILE
environment:
- CREDENTIALS=/run/secrets/CREDENTIALS_FILE
command: /bin/bash
entrypoint: /bin/bash
test:
build:
build: # build the image so we can specify the target
context: .
dockerfile: docker/ingest/Dockerfile
target: dev
Expand All @@ -31,10 +28,7 @@ services:
command: poetry run pytest tests
# FIXME - add a way to specify unit & integration tests
ingest:
build:
context: .
dockerfile: docker/ingest/Dockerfile
target: prod
image: ghcr.io/noaa-gsl/vxingest/ingest:main
volumes:
- ${data}:/opt/data
- ${public}:/public
Expand All @@ -47,9 +41,7 @@ services:
- "--metrics_dir=/opt/data/test/metrics"
- "--transfer_dir=/opt/data/test/xfer"
import:
build:
context: .
dockerfile: docker/import/Dockerfile
image: ghcr.io/noaa-gsl/vxingest/import:main
volumes:
- ${data}:/opt/data
secrets:
Expand Down

0 comments on commit 8f7030d

Please sign in to comment.