From 44e81f9bc8d17eb8beb70ff91317d4e6b52e125d Mon Sep 17 00:00:00 2001
From: Sanjay S Kumar <35188255+Quantaindew@users.noreply.github.com>
Date: Thu, 11 Apr 2024 19:14:14 +0530
Subject: [PATCH] Added extra documentation to mention alignment of port
 variables

By default it's better to have port 8080, 80 gives errors

Signed-off-by: Sanjay S Kumar <35188255+Quantaindew@users.noreply.github.com>
---
 docs/builds/dockerfile/python.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/docs/builds/dockerfile/python.md b/docs/builds/dockerfile/python.md
index d6c7de4..c1e43a8 100644
--- a/docs/builds/dockerfile/python.md
+++ b/docs/builds/dockerfile/python.md
@@ -8,6 +8,7 @@ Below is an example Dockerfile based off a Python application using FastAPI and
 
 :::info
 Keep in mind you'll need to alter this to work with your specific file structure.
+Exposed Port must match PORT variable defined in enviroment varibles, here it's 8080.
 :::
 
 ```bash title="/Dockerfile"
@@ -23,5 +24,5 @@ COPY --from=requirements-stage /tmp/requirements.txt /code/requirements.txt
 RUN pip install --no-cache-dir --upgrade -r /code/requirements.txt
 COPY ./app /code/app
 
-CMD ["uvicorn", "app.main:app", "--proxy-headers", "--host", "0.0.0.0", "--port", "80"]
+CMD ["uvicorn", "app.main:app", "--proxy-headers", "--host", "0.0.0.0", "--port", "8080"]
 ```