Skip to content

Commit

Permalink
Merge pull request #2717 from danswer-ai/bugfix/docker-legacy-key-val…
Browse files Browse the repository at this point in the history
…ue-format

Fix all LegacyKeyValueFormat docker warnings
  • Loading branch information
rkuo-danswer authored and Richard Kuo [bot] committed Oct 9, 2024
1 parent acaff41 commit 1646acc
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion backend/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ COPY ./scripts/force_delete_connector_by_id.py /app/scripts/force_delete_connect
# Put logo in assets
COPY ./assets /app/assets

ENV PYTHONPATH /app
ENV PYTHONPATH=/app

# Default command which does nothing
# This container is used by api server and background which specify their own CMD
Expand Down
2 changes: 1 addition & 1 deletion backend/Dockerfile.model_server
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,6 @@ COPY ./shared_configs /app/shared_configs
# Model Server main code
COPY ./model_server /app/model_server

ENV PYTHONPATH /app
ENV PYTHONPATH=/app

CMD ["uvicorn", "model_server.main:app", "--host", "0.0.0.0", "--port", "9000"]
2 changes: 1 addition & 1 deletion backend/tests/integration/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,6 @@ RUN pip install --no-cache-dir --upgrade \
-r /tmp/dev-requirements.txt
COPY ./tests/integration /app/tests/integration

ENV PYTHONPATH /app
ENV PYTHONPATH=/app

CMD ["pytest", "-s", "/app/tests/integration"]
6 changes: 3 additions & 3 deletions web/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,10 @@ COPY . .
RUN npm ci

# needed to get the `standalone` dir we expect later
ENV NEXT_PRIVATE_STANDALONE true
ENV NEXT_PRIVATE_STANDALONE=true

# Disable automatic telemetry collection
ENV NEXT_TELEMETRY_DISABLED 1
ENV NEXT_TELEMETRY_DISABLED=1

# Environment variables must be present at build time
# https://github.com/vercel/next.js/discussions/14030
Expand Down Expand Up @@ -77,7 +77,7 @@ RUN rm -rf /usr/local/lib/node_modules
# ENV NODE_ENV production

# Disable automatic telemetry collection
ENV NEXT_TELEMETRY_DISABLED 1
ENV NEXT_TELEMETRY_DISABLED=1

# Don't run production as root
RUN addgroup --system --gid 1001 nodejs
Expand Down

0 comments on commit 1646acc

Please sign in to comment.