Add docker live reloading to Cleezy (#21)

Author: evanugarte

Dockerfile

@@ -1,15 +1,13 @@
 FROM python:3.9-slim-buster
 
-WORKDIR /src
+WORKDIR /app
 
 RUN apt-get update && apt-get install -y default-libmysqlclient-dev
 
 COPY requirements.txt .
 
 RUN pip install -r requirements.txt
 
-COPY . .
-
 EXPOSE 8000
 
 ENTRYPOINT ["python", "server.py"]

docker-compose.dev.yml

@@ -0,0 +1,22 @@
+version: '3.7'
+services:
+  app:
+    container_name: cleezy-app-dev
+    build:
+      context: .
+      dockerfile: Dockerfile
+    command:
+       - --database-file-path=/tmp/urldatabase.db
+       - --port=8000
+       - -vvv
+    ports:
+      - 8000:8000
+    volumes:
+      - cleezy_data_dev:/tmp/
+      - ./server.py:/app/server.py
+      - ./modules:/app/modules
+    environment:
+      - WATCHFILES_FORCE_POLLING=true
+
+volumes:
+  cleezy_data_dev:

docker-compose.yml

@@ -10,6 +10,8 @@ services:
        - -vvv
     volumes:
       - cleezy_data:/tmp/
+      - ./server.py:/app/server.py
+      - ./modules:/app/modules
   prometheus:
     image: prom/prometheus:latest
     restart: always

args.py modules/args.pyargs.py renamed to modules/args.py

@@ -6,11 +6,12 @@
 import prometheus_client
 import uvicorn
 
-from args import get_args
-from generate_alias import generate_alias
-import sqlite_helpers
-from constants import HttpResponse, http_code_to_enum
-from metrics import MetricsHandler
+from modules.args import get_args
+from modules.generate_alias import generate_alias
+import modules.sqlite_helpers as sqlite_helpers
+from modules.constants import HttpResponse, http_code_to_enum
+from modules.metrics import MetricsHandler
+
 
 app = FastAPI()
 args = get_args()
@@ -105,8 +106,18 @@ def get_metrics():
     level= logging.ERROR - (args.verbose*10),
 )
 
-if __name__ == "__main__":
-    logging.info(f"running on {args.host}, listening on port {args.port}")
+# we have a separate __name__ check here due to how FastAPI starts
+# a server. the file is first ran (where __name__ == "__main__")
+# and then calls `uvicorn.run`. the call to run() reruns the file,
+# this time __name__ == "server". the separate __name__ if statement
+# is so the thread references the same instance as the global
+# metrics_handler referenced by the rest of the file. otherwise,
+# the thread interacts with an instance different than the one the
+# server uses
+if __name__ == "server":
     initial_url_count = sqlite_helpers.get_number_of_entries(DATABASE_FILE)
     MetricsHandler.url_count.inc(initial_url_count)
-    uvicorn.run(app, host=args.host, port=args.port)
+
+if __name__ == "__main__":
+    logging.info(f"running on {args.host}, listening on port {args.port}")
+    uvicorn.run("server:app", host=args.host, port=args.port, reload=True)