DockerFiles and Docker Compose for multiple containers by Suryanshg · Pull Request #31 · ShafathZ/AniZenithProject

Suryanshg · 2026-04-03T21:07:44Z

No description provided.

…le because of cuda dependencies

… issue earlier along with healthcheck stuff not working. All sorted now

# Conflicts: # frontend/app.py # uv.lock

… docker COPY for prometheus folder. Additionally added #!/bin/bash line to sh scripts. Fixed merge conflicts, and tested working locally and with docker containers

…leanup. Also added a deployment script for the compose.yml and related stuff

…oard for extra credit

Copilot

Pull request overview

This PR introduces Docker/Docker Compose assets to run AniZenith as a multi-container deployment (frontend, backend, monitoring stack), and adjusts Python dependency/index configuration to support installing PyTorch CPU builds.

Changes:

Add Dockerfiles + docker/compose.yml for frontend/backend, plus Prometheus/Grafana/cAdvisor/node-exporter/ngrok services.
Update FastAPI apps to use environment variables for host/port and add a backend /health endpoint for container healthchecks.
Reconfigure uv/lockfiles and requirements to pull torch from the PyTorch CPU index.

Reviewed changes

Copilot reviewed 17 out of 22 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
`uv.lock`	Updates resolved artifacts/sources (notably pulling many packages from the PyTorch CPU index).
`README.md`	Adds Case Study 3 VM access and links; wraps Case Study 2 content in `<details>`.
`pyproject.toml`	Adds a `uv` index/source mapping intended for PyTorch CPU wheels.
`prometheus/prometheus_middleware.py`	Removes standalone Prometheus HTTP server start; exposes `/metrics` via the app router + middleware.
`grafana_dashboards/anizenith_monitoring_dashboard.json`	Adds a Grafana dashboard for container CPU/memory monitoring via Prometheus.
`frontend/requirements.txt`	Adds a separate requirements file for the frontend container.
`frontend/docker/.gitkeep`	Placeholder to keep the directory in Git.
`frontend/app.py`	Switches frontend runtime config to env vars; adjusts proxy backend URL construction; uses Prometheus middleware/router.
`docker/run.sh`	Adds a helper script to pull/down/up the compose stack.
`docker/prometheus.yml`	Adds Prometheus scrape config for frontend/backend/cAdvisor/node-exporter.
`docker/frontend.dockerfile`	Adds a Dockerfile to build/run the frontend container.
`docker/compose.yml`	Adds a multi-service compose stack including monitoring and tunneling.
`docker/build.sh`	Adds a helper script to build and push backend/frontend images.
`docker/backend.dockerfile`	Adds a Dockerfile to build/run the backend container (includes curl + PyTorch CPU extra index).
`backend/requirements.txt`	Adjusts pinned backend dependencies (including PyTorch CPU wheel + related version changes).
`backend/docker/.gitkeep`	Placeholder to keep the directory in Git.
`backend/app.py`	Adds `/health` endpoint; switches runtime config to env vars; uses Prometheus middleware/router.
`automation_scripts/establish_new_ssh_keys_for_backend.sh`	Updates SSH port used for key establishment.
`automation_scripts/deploy_containers.sh`	Adds an automated deployment script to copy compose/env files and start containers on the VM.
`.gitignore`	Ignores `*.env` and `.run/`.
`.github/workflows/run_tests.yml`	Updates CI to install backend requirements before running pytest.
`.dockerignore`	Ignores `*.env` and Python cache/bytecode from Docker build context.

Comments suppressed due to low confidence (4)

frontend/app.py:26

This f-string uses nested double quotes (os.getenv("FRONTEND_PORT")) inside a double-quoted f-string, which is a Python syntax error. Use single quotes for the env var key (or escape the quotes) so the module can import/run.
frontend/app.py:97
Same quoting issue as above: nested double quotes inside a double-quoted f-string makes this line invalid Python. Switch the inner getenv keys to single quotes (or otherwise avoid quote nesting).
frontend/app.py:138
int(os.getenv("FRONTEND_PORT")) will raise if the env var is unset/empty, and host=os.getenv("FRONTEND_HOSTNAME") may bind to nothing (or default to 127.0.0.1), which will break access from other containers. Consider providing safe defaults (e.g., host 0.0.0.0 and port 7002) or validating env vars at startup with a clear error.
backend/requirements.txt:45
Including pip==24.0 in the application requirements will force a pip downgrade during installs (even though the Dockerfile upgrades pip just before). This can cause non-reproducible builds and makes dependency installation harder to reason about; prefer managing pip via the base image/build step rather than pinning it as an app dependency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-03T21:13:39Z

+
+[[tool.uv.index]]
+name = "pytorch-cpu"
+url = "https://download.pytorch.org/whl/cpu"


The added PyTorch CPU index is intended for torch, but the lockfile now resolves many unrelated packages from https://download.pytorch.org/whl/cpu (and downgrades some, e.g. certifi/urllib3). Configure the index as explicit/secondary (so only torch uses it) and re-lock, to avoid accidental global dependency changes.

Suggested change

url = "https://download.pytorch.org/whl/cpu"

url = "https://download.pytorch.org/whl/cpu"

explicit = true

Addressed in commit: 1fdbba4

Copilot · 2026-04-03T21:13:40Z

+# Start from the base python:3.12.3-slim which also has debian:12-slim as base layer
+FROM python:3.12.3-slim
+
+# TODO: Add comment here


Leftover placeholder comment (# TODO: Add comment here). If this is meant to document why curl is installed (healthcheck), please replace the TODO with an explanatory comment or remove it.

Suggested change

# TODO: Add comment here

# Install curl for container health checks and service diagnostics

Addressed in commit: 1fdbba4

…dashboard

ShafathZ

Needs few more useful comments, but code accepted

ShafathZ · 2026-04-07T19:42:13Z

+
+# Install libraries using requirements.txt
+RUN pip install --upgrade pip
+RUN pip install -r backend/requirements.txt --extra-index-url https://download.pytorch.org/whl/cpu --no-cache-dir


nit: Write comment that --extra-index-url is here since we are ensuring cpu version is installed (since Linux defaults to cuda version on PyPI)

Good catch. Addressed in a27774a

ShafathZ · 2026-04-07T19:45:13Z

+    image: gcr.io/cadvisor/cadvisor:latest
+    container_name: anizenith_cadvisor
+    volumes:
+      - /:/rootfs:ro


I know professor wrote the code, but needs comments for why we require all of these volumes (example comment: cadvisor requires data from: root (/), sys (/sys), docker (/var/lib/docker), ...

Good catch. Addressed in a27774a

ShafathZ · 2026-04-07T19:46:14Z

+
+# Maps prometheus and grafana data to a volume outside the container (in case container is destroyed)
+volumes:
+  # Used to persist Prometheus time-series metrics across container restarts


Add TODO to map to local user drive, but outside container drives (if possible)

I added a separate TODO, as the issue is different.
Addressed in a27774a

…ys helps :)

ashirwani04 and others added 11 commits March 28, 2026 15:54

added docker files and docker compose, need to test on docker containers

4cf3006

Frontend image building working but backend image building was terrib…

1e3d7e5

…le because of cuda dependencies

Attempted fix by removing torch-cpu

8b2ff39

Fixed uv lock with cpu again

3392476

Modified dockerfiles and compose yaml as we had some weird networking…

704f123

… issue earlier along with healthcheck stuff not working. All sorted now

Merge remote-tracking branch 'origin/main' into docker

4466ae0

# Conflicts: # frontend/app.py # uv.lock

Reverse merged main into docker branch with prometheus changes. Added…

9a0e603

… docker COPY for prometheus folder. Additionally added #!/bin/bash line to sh scripts. Fixed merge conflicts, and tested working locally and with docker containers

Added prometheus container setup

f71217c

Added Ngrok, cadvisor, node-exporter containers, and performed some c…

d74d815

…leanup. Also added a deployment script for the compose.yml and related stuff

Added Comments in Docker compose.yml and added a simple grafana dashb…

289eb19

…oard for extra credit

Finalized .env export for running frontend and backend

b975c17

Suryanshg requested review from ShafathZ, ashirwani04 and Copilot April 3, 2026 21:07

Copilot started reviewing on behalf of Suryanshg April 3, 2026 21:08 View session

Trying to fix failing unit tests becoz of some weird pypi issue

97e177b

Copilot AI reviewed Apr 3, 2026

View reviewed changes

Suryanshg added 2 commits April 6, 2026 11:50

Added ngrok.yml and configured endpoints and traffic policy on ngrok …

dc81053

…dashboard

Added Ngrok Link for grafana

e5f7f56

Repository owner deleted a comment from Copilot AI Apr 6, 2026

Suryanshg added 2 commits April 6, 2026 12:26

Addressing PR comments by Copilot

1fdbba4

Cleaned up README.md

8995204

ashirwani04 approved these changes Apr 6, 2026

View reviewed changes

ShafathZ approved these changes Apr 7, 2026

View reviewed changes

Added inline comments to adddress PR comments, good documenation alwa…

a27774a

…ys helps :)

ShafathZ approved these changes Apr 8, 2026

View reviewed changes

ShafathZ merged commit 70c32f8 into main Apr 8, 2026
2 checks passed

ShafathZ deleted the docker branch April 17, 2026 22:46

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

DockerFiles and Docker Compose for multiple containers#31

DockerFiles and Docker Compose for multiple containers#31
ShafathZ merged 17 commits into
mainfrom
docker

Suryanshg commented Apr 3, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 3, 2026

Uh oh!

Suryanshg Apr 6, 2026

Uh oh!

Copilot AI Apr 3, 2026

Uh oh!

Suryanshg Apr 6, 2026

Uh oh!

ShafathZ left a comment

Uh oh!

ShafathZ Apr 7, 2026

Uh oh!

Suryanshg Apr 7, 2026

Uh oh!

ShafathZ Apr 7, 2026

Uh oh!

Suryanshg Apr 7, 2026

Uh oh!

ShafathZ Apr 7, 2026

Uh oh!

Suryanshg Apr 7, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

	url = "https://download.pytorch.org/whl/cpu"
	url = "https://download.pytorch.org/whl/cpu"
	explicit = true

	# TODO: Add comment here
	# Install curl for container health checks and service diagnostics

Conversation

Suryanshg commented Apr 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ShafathZ left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants