default docker image to port 8080

Author: isabelizimm

vetiver/tests/test_write_docker.py

@@ -14,26 +14,26 @@ def test_vetiver_write_app_no_rspm():
     contents = open(file).read()
     os.remove(file)
     assert(contents == f"""# # Generated by the vetiver package; edit with care
-#
+# start with python base image
 FROM python:{py_version}
 
-#
+# create directory in container for vetiver files
 WORKDIR /vetiver
 
-#
+# copy  and install requirements
 COPY vetiver_requirements.txt /vetiver/requirements.txt
 
 #
 RUN pip install --no-cache-dir --upgrade -r /vetiver/requirements.txt
 
-#
+# copy app file
 COPY app.py /vetiver/app
 
-#
-EXPOSE 80
+# expose port
+EXPOSE 8080
 
-#
-CMD ["uvicorn", "app.app:api", "--host", "0.0.0.0", "--port", "80"]
+# run vetiver API
+CMD ["uvicorn", "app.app:api", "--host", "0.0.0.0", "--port", "8080"]
 """)
 
 def test_vetiver_write_app_rspm():
@@ -44,25 +44,25 @@ def test_vetiver_write_app_rspm():
     py_version = str(sys.version_info.major) + "." + str(sys.version_info.minor)
 
     assert(contents == f"""# # Generated by the vetiver package; edit with care
-#
+# start with python base image
 FROM python:{py_version}
 
-#
+# create directory in container for vetiver files
 WORKDIR /vetiver
 
-#
+# copy  and install requirements
 COPY vetiver_requirements.txt /vetiver/requirements.txt
 
 #
 RUN pip config set global.index-url https://colorado.rstudio.com/rspm/pypi/latest/simple
 RUN pip install --no-cache-dir --upgrade -r /vetiver/requirements.txt
 
-#
+# copy app file
 COPY app.py /vetiver/app
 
-#
-EXPOSE 80
+# expose port
+EXPOSE 8080
 
-#
-CMD ["uvicorn", "app.app:api", "--host", "0.0.0.0", "--port", "80"]
+# run vetiver API
+CMD ["uvicorn", "app.app:api", "--host", "0.0.0.0", "--port", "8080"]
 """)

vetiver/write_docker.py

@@ -4,7 +4,7 @@ def vetiver_write_docker(app_file: str = "app.py",
     path: str = "./",
     rspm_env: bool = False,
     host: str = "0.0.0.0",
-    port: str = "80"):
+    port: str = "8080"):
     """Writes a Dockerfile to run VetiverAPI in a container
 
     Args
@@ -29,25 +29,25 @@ def vetiver_write_docker(app_file: str = "app.py",
 
 
     docker_script = f"""# # Generated by the vetiver package; edit with care
-#
+# start with python base image
 FROM python:{py_version}
 
-#
+# create directory in container for vetiver files
 WORKDIR /vetiver
 
-#
+# copy  and install requirements
 COPY vetiver_requirements.txt /vetiver/requirements.txt
 
 #{rspm}
 RUN pip install --no-cache-dir --upgrade -r /vetiver/requirements.txt
 
-#
+# copy app file
 COPY {app_file} /vetiver/app
 
-#
+# expose port
 EXPOSE {port}
 
-#
+# run vetiver API
 CMD ["uvicorn", "app.app:api", "--host", "{host}", "--port", "{port}"]
 """