Skip to content

Commit b577b9c

Browse files
Enhance Dockerfile comments to clarify architectural decision for using Python/uvicorn for frontend serving
1 parent dbdb613 commit b577b9c

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

src/frontend/Dockerfile

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,15 @@ RUN npm run build
1717

1818
# Runtime stage
1919
# ARCHITECTURAL DECISION: Using Python/uvicorn instead of nginx for frontend serving
20-
# Reasons:
21-
# 1. Unified deployment model with backend (both use Python/uvicorn)
22-
# 2. Consistent container app configuration in Azure
23-
# 3. Simplified orchestration and monitoring
24-
# 4. frontend_server.py may handle API proxying or server-side logic
20+
#
21+
# This approach provides:
22+
# 1. Unified deployment model - both frontend/backend use identical Python/uvicorn runtime
23+
# 2. Consistent Azure Container Apps configuration and resource management
24+
# 3. Simplified CI/CD pipeline with single container technology stack
25+
# 4. frontend_server.py enables API proxying, authentication, or future server-side features
26+
# 5. Reduced operational complexity - same monitoring, logging, and scaling patterns
27+
#
28+
# Trade-off: Larger container size vs operational simplicity benefits
2529
FROM python:3.11-slim
2630

2731
WORKDIR /app

0 commit comments

Comments
 (0)