Skip to content

Commit

Permalink
[TM-1627] Fix the sentry connection.
Browse files Browse the repository at this point in the history
  • Loading branch information
roguenet committed Jan 14, 2025
1 parent cf38939 commit 73a0c7f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/deploy-service.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ jobs:
: # Don't build the base image with NODE_ENV because it'll limit the packages that are installed
docker build -t terramatch-microservices-base:nx-base .
SERVICE_IMAGE=$ECR_REGISTRY/$ECR_REPOSITORY:$IMAGE_TAG
docker build --build-arg NODE_ENV=production --build-arg BUILD_FLAG='--prod --verbose' -f apps/${{ inputs.service }}/Dockerfile -t $SERVICE_IMAGE .
docker build --build-arg NODE_ENV=production --build-arg SENTRY_DSN="${{ vars.SENTRY_DSN }}" --build-arg BUILD_FLAG='--prod --verbose' -f apps/${{ inputs.service }}/Dockerfile -t $SERVICE_IMAGE .
docker push $SERVICE_IMAGE
echo "image=$SERVICE_IMAGE"
Expand Down
2 changes: 2 additions & 0 deletions apps/unified-database-service/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,10 @@ RUN npx nx build unified-database-service ${BUILD_FLAG}
FROM terramatch-microservices-base:nx-base

ARG NODE_ENV
ARG SENTRY_DSN
WORKDIR /app
COPY --from=builder /app/builder ./
ENV NODE_ENV=${NODE_ENV}
ENV SENTRY_DSN=${SENTRY_DSN}

CMD ["node", "./dist/apps/unified-database-service/main.js"]

0 comments on commit 73a0c7f

Please sign in to comment.