Merge pull request #157 from the-mama-ai/fix-default-args

fix(docker): remove ARG PORT
datopian · Mar 29, 2024 · 75b046b · 75b046b
2 parents b15675c + bc5ef89
commit 75b046b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/.dockerignore b/.dockerignore
@@ -7,3 +7,5 @@
 .pytest_cache
 .mypy_cache
 /.idea
+.tox
+.make-cache
diff --git a/Dockerfile b/Dockerfile
@@ -55,12 +55,9 @@ WORKDIR /app
 
 ENV UWSGI_MODULE "giftless.wsgi_entrypoint"
 
-ARG PORT=5000
-EXPOSE $PORT
-
 ENTRYPOINT ["tini", "uwsgi", "--"]
+CMD ["-s", "127.0.0.1:5000", "-M", "-T", "--threads", "2", "-p", "2", \
+     "--manage-script-name", "--callable", "app"]
+
 # TODO remove this STOPSIGNAL override after uwsgi>=2.1
 STOPSIGNAL SIGQUIT
-
-CMD ["-s", "127.0.0.1:${PORT}", "-M", "-T", "--threads", "2", "-p", "2", \
-     "--manage-script-name", "--callable", "app"]