Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Commit

Permalink
Move the Dockerfile for Lambda to the API service
Browse files Browse the repository at this point in the history
Rename the Dockerfile for the Lambda function to Dockerfile-diag-etl in
the API service. Update it to run the handler function from the
unified_graphics package instead of ugdata.

I’m pretty sure this is all we need to do to get this working. It feels
a little bad installing the whole unified_graphics package in the
container for the Lambda function, since it doesn’t need Flask or
anything like that. Maybe we can look into groups in poetry to at least
reduce the size of the dependencies.
  • Loading branch information
esheehan-gsl committed May 5, 2023
1 parent 6a45ed2 commit ad04ccb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions services/data/Dockerfile → services/api/Dockerfile-diag-etl
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,6 @@ COPY poetry.lock .
RUN poetry export --format requirements.txt --without dev --without-hashes --output requirements.txt \
&& pip install --target "${LAMBDA_TASK_ROOT}" --requirement requirements.txt

COPY src/ugdata/ "${LAMBDA_TASK_ROOT}/ugdata"
COPY src/unified_graphics/ "${LAMBDA_TASK_ROOT}/unified_graphics"

CMD ["ugdata.aws.lambda_handler"]
CMD ["unified_graphics.etl.aws.lambda_handler"]

0 comments on commit ad04ccb

Please sign in to comment.