Merge branch 'dev' (staging/no-tag)

Dockerfile

@@ -86,8 +86,9 @@ EOT
 # See <https://hynek.me/articles/docker-signals/>.
 STOPSIGNAL SIGINT
 
-COPY --from=build --chown=app:app /app /app
+COPY --from=build --chown=app:app --chmod=u+rw /app /app
 
+COPY --from=ghcr.io/astral-sh/uv:0.7.2 /uv /app/bin/uv
 COPY --chown=app:app --chmod=700 /scripts/entrypoint.sh /entrypoint.sh
 COPY --chown=app:app --chmod=700 /scripts/deploy.sh /deploy.sh
 COPY --chown=app:app --chmod=700 /manage.py /app/manage.py

docker-compose.yml

@@ -1,30 +1,41 @@
 services:
   web:
     build: .
-    #command: ["django-admin", "runserver", "0.0.0.0:8000"]
+    command: bash -c "cd /code && uv pip install --editable . --no-deps && python manage.py runserver 0.0.0.0:8000"
     ports:
-      - 8000:8000
-    restart: always
+      - "8000:8000"
+    restart: unless-stopped
+    environment:
+      - DEBUG=true
     volumes:
       - ./seed:/seed:ro
-      # DANGEROUS DUE TO DOCKERFILE PACKAGE BUILDING/INSTALLATION
-      # will need to use editable package instead somehow
-      #- ./src/pkmntrade_club:/app/lib/python3.12/site-packages/pkmntrade_club:ro
+      - ./:/code
     depends_on:
       db:
         condition: service_healthy
+      redis:
+        condition: service_started
   celery:
     build: .
-    command: ["celery", "-A", "pkmntrade_club.django_project", "worker", "-l", "INFO", "-B", "-E"]
-    restart: always
+    command: bash -c "cd /code && uv pip install --editable . --no-deps && celery -A pkmntrade_club.django_project worker -l INFO -B -E"
+    restart: unless-stopped
+    volumes:
+      - ./:/code
     depends_on:
       db:
         condition: service_healthy
+      redis:
+        condition: service_started
   redis:
     image: redis:latest
     restart: always
     ports:
       - 6379:6379
+    healthcheck:
+      test: ["CMD", "redis-cli", "ping"]
+      interval: 10s
+      timeout: 5s
+      retries: 5
   db:
     image: postgres:16
     restart: always
@@ -37,6 +48,5 @@ services:
       interval: 10s
       timeout: 5s
       retries: 5
-
 volumes:
   postgres_data:

scripts/entrypoint.sh

@@ -8,10 +8,8 @@ fi
 
 /deploy.sh
 
-# show command version before running it
-$1 --version
-
 if [ "$1" == "granian" ]; then
+    granian --version
     echo "Appending static files path to granian command (requires granian >= 2.3.0)"
     STATIC_ROOT=$(python -c 'import os; import pkmntrade_club; from django.conf import settings; print(settings.STATIC_ROOT)')
     set -- "$@" --static-path-mount "$STATIC_ROOT"

scripts/prebuild.sh

@@ -1,11 +1,12 @@
 #!/bin/bash
 
+cd src/pkmntrade_club/
+
 # Remove all files in staticfiles except .gitkeep
 if [ -d "staticfiles" ]; then
     find staticfiles -type f ! -name '.gitkeep' -delete
-    find staticfiles -type d -empty -delete
 fi
 
 # Build the tailwind theme css
-cd src/pkmntrade_club/theme/static_src
+cd theme/static_src
 npm install . && npm run build

scripts/reset-db_make-migrations_seed-data.sh

@@ -9,18 +9,14 @@ source .env
 set +a
 uv run manage.py makemigrations --noinput
 
-echo "Resetting database... "
+echo "Resetting dev database... "
 docker compose down \
   && docker volume rm -f pkmntradeclub_postgres_data \
   && ./scripts/rebuild-and-run.sh
 
 # Wait for the database to be ready.
-echo "Waiting 15 seconds for the database to be ready, and migrations to be autorun..."
+echo "Waiting 15 seconds for migrations to be auto-run..."
 sleep 15
-
-echo "Creating cache table..."
-docker compose exec -it web bash -c "django-admin createcachetable django_cache"
-
 echo "Loading seed data..."
 docker compose exec -it web bash -c "django-admin loaddata /seed/0*"
 

server/docker-compose_staging.yml

@@ -10,13 +10,14 @@ services:
       - DEBUG=False
       - DISABLE_SIGNUPS=True
       - PUBLIC_HOST=staging.pkmntrade.club
+      - ALLOWED_HOSTS=staging.pkmntrade.club,127.0.0.1
     labels:
       - "enable_gatekeeper=true"
     deploy:
       mode: replicated
       replicas: 2
     # healthcheck:
-    #   test: ["CMD", "curl", "-f", "http://127.0.0.1:8000"]
+    #   test: ["CMD", "curl", "-f", "http://127.0.0.1:8000/health"]
     #   interval: 30s
     #   timeout: 10s
     #   retries: 3
@@ -27,4 +28,5 @@ services:
       - DEBUG=False
       - DISABLE_SIGNUPS=True
       - PUBLIC_HOST=staging.pkmntrade.club
+      - ALLOWED_HOSTS=staging.pkmntrade.club,127.0.0.1
     command: ["celery", "-A", "pkmntrade_club.django_project", "worker", "-l", "INFO", "-B", "-E"]

server/docker-compose_web.yml

@@ -9,18 +9,18 @@ services:
     image: ghcr.io/xe/x/httpdebug
     entrypoint: ["/ko-app/httpdebug", "--bind", ":8000"]
     #image: badbl0cks/pkmntrade-club:stable
-    #command: ["granian", "--interface", "wsgi", "pkmntrade_club.django_project.wsgi:app", "--host", "0.0.0.0", "--port", "8000", "--workers", "1", "--workers-kill-timeout", "180", "--access-log"] 
     environment:
       - DEBUG=False
       - DISABLE_SIGNUPS=True
       - PUBLIC_HOST=pkmntrade.club
+      - ALLOWED_HOSTS=pkmntrade.club,127.0.0.1
     labels:
       - "enable_gatekeeper=true"
     deploy:
       mode: replicated
       replicas: 4
     # healthcheck:
-    #   test: ["CMD", "curl", "-f", "http://127.0.0.1:8000"]
+    #   test: ["CMD", "curl", "-f", "http://127.0.0.1:8000/health"]
     #   interval: 30s
     #   timeout: 10s
     #   retries: 3
@@ -32,4 +32,5 @@ services:
   #     - DEBUG=False
   #     - DISABLE_SIGNUPS=True
   #     - PUBLIC_HOST=pkmntrade.club
+  #     - ALLOWED_HOSTS=pkmntrade.club,127.0.0.1
   #   command: ["celery", "-A", "pkmntrade_club.django_project", "worker", "-l", "INFO", "-B", "-E"]

src/pkmntrade_club/home/urls.py

@@ -1,6 +1,6 @@
 from django.urls import path
 
-from .views import HomePageView, HealthCheckView
+from .views import HomePageView
 
 urlpatterns = [
     path("", HomePageView.as_view(), name="home"),

src/pkmntrade_club/home/views.py

@@ -139,29 +139,3 @@ class HomePageView(TemplateView):
     def get(self, request, *args, **kwargs):
         """Override get method to add caching"""
         return super().get(request, *args, **kwargs)
-
-class HealthCheckView(View):
-    def get(self, request, *args, **kwargs):
-
-        try:
-            from django.db import connection
-            connection.cursor().execute("SELECT 1")
-        except Exception as e:
-            return HttpResponse("Database connection failed", status=500)
-
-        try:
-            from pkmntrade_club.trades.models import TradeOffer
-            with contextlib.redirect_stdout(None):
-                print(TradeOffer.objects.count())
-        except Exception as e:
-            return HttpResponse("DB models not reachable, but db is reachable", status=500)
-        
-        try:
-            from django.core.cache import cache
-            cache.set("test", "test")
-            with contextlib.redirect_stdout(None):
-                print(cache.get("test"))
-        except Exception as e:
-            return HttpResponse("Cache not reachable", status=500)
-        
-        return HttpResponse("OK/HEALTHY")

src/pkmntrade_club/theme/templates/base.html

@@ -38,8 +38,8 @@
   <link rel="stylesheet" href="{% static 'css/base.css' %}">
 
   <!-- Floating UI -->
-  <script src="{% static 'js/floating-ui_core@1.6.9.9.min.js' %}"></script>
-  <script src="{% static 'js/floating-ui_dom@1.6.13.13.min.js' %}"></script>
+  <script src="{% static 'js/floating-ui_core-1.6.9.9.min.js' %}"></script>
+  <script src="{% static 'js/floating-ui_dom-1.6.13.13.min.js' %}"></script>
 
   <script defer src="{% static 'js/card-multiselect.js' %}"></script>
   <link rel="stylesheet" href="{% static 'css/card-multiselect.css' %}">
@@ -130,13 +130,13 @@
   </div>
   
   <!-- Alpine Plugins -->
-  <script defer src="{% static 'js/alpinejs.collapse@3.14.8.min.js' %}"></script>
+  <script defer src="{% static 'js/alpinejs.collapse-3.14.8.min.js' %}"></script>
   
   <!-- Alpine Core -->
-  <script defer src="{% static 'js/alpinejs@3.14.8.min.js' %}"></script>
+  <script defer src="{% static 'js/alpinejs-3.14.8.min.js' %}"></script>
 
-  <!-- 100% privacy-first, no tracking analytics -->
-  <script data-goatcounter="https://stats.pkmntrade.club/count" async src="/static/js/count@v4.js"></script>
+  <!-- Goatcounter: 100% privacy-first, no tracking analytics -->
+  <script data-goatcounter="https://stats.pkmntrade.club/count" async src="{% static 'js/count-v4.js' %}"></script>
   
   {% block javascript %}{% endblock %}
 </body>