926 migrate beta deploy to use ghcr instead of docker hub 2 (#945)

* Adding distinct github image deployable.

* Creating a dockerignore to avoid files too big.

* Adjusting dockerfile to only consume the minimal fields.

* Bonus quicksetup fix, for speedup. Migrations should be made at build time.

* Adding in a sqlite change or two for speedup.

Author: augustjohnson

.dockerignore

@@ -0,0 +1,17 @@
+.git
+__pycache__
+*.pyc
+*.pyo
+.venv
+.env
+*.sqlite3
+.github
+.gitignore
+README.md
+CONTRIBUTING.md
+CODE_OF_CONDUCT.md
+LICENSE
+*.md
+.pytest_cache
+.ruff_cache
+node_modules

.github/workflows/build_and_deploy.yml

@@ -4,6 +4,15 @@ on:
     branches:
       - staging
       - main
+    paths-ignore:
+      - 'README.md'
+      - 'docs/**'
+      - 'scripts/**'
+      - '.github/ISSUE_TEMPLATE/**'
+
+concurrency:
+  group: deploy-${{ github.ref_name }}
+  cancel-in-progress: true
 
 jobs:
   build:
@@ -14,21 +23,32 @@ jobs:
         with:
           username: ${{ secrets.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
-      - name: Set Docker image tag
-        run: | 
+      - name: Login to GHCR
+        uses: docker/login-action@v2
+        with:
+          registry: ghcr.io
+          username: ${{ github.actor }}
+          password: ${{ secrets.GITHUB_TOKEN }}
+      - name: Set image tags
+        run: |
           echo "GITHUB_REF = $GITHUB_REF" > version.py
           echo "GITHUB_SHA = $GITHUB_SHA" >> version.py
-        
+
           if [[ $GITHUB_REF == refs/heads/main ]]; then
-            echo "DOCKER_IMAGE_TAG=latest" >> $GITHUB_ENV
+            echo "IMAGE_TAGS=eepmoody/open5e-api:latest" >> $GITHUB_ENV
           elif [[ $GITHUB_REF == refs/heads/staging ]]; then
-            echo "DOCKER_IMAGE_TAG=staging" >> $GITHUB_ENV
+            {
+              echo "IMAGE_TAGS<<EOF"
+              echo "eepmoody/open5e-api:staging"
+              echo "ghcr.io/open5e/open5e-api:beta"
+              echo "EOF"
+            } >> $GITHUB_ENV
           fi
       - name: Build and push
         uses: docker/build-push-action@v7
         with:
           push: true
-          tags: eepmoody/open5e-api:${{ env.DOCKER_IMAGE_TAG }}
+          tags: ${{ env.IMAGE_TAGS }}
 
   deploy:
     runs-on: ubuntu-latest

.github/workflows/pr_validation.yml

@@ -21,6 +21,8 @@ jobs:
         run: uv python install 3.11
       - name: Install dependencies
         run: uv sync
+      - name: Check migrations are up to date
+        run: uv run python manage.py makemigrations --check --dry-run
       - name: Run migrations
         run: uv run python manage.py quicksetup --noindex
       - name: Run Tests

Dockerfile

@@ -14,8 +14,16 @@ FROM python:3.11-slim
 
 WORKDIR /opt/services/open5e-api
 
-COPY --from=builder /build/.venv /opt/services/open5e-api/.venv
-COPY --from=builder /build /opt/services/open5e-api
+COPY --from=builder /build/.venv ./.venv
+COPY --from=builder /build/server ./server
+COPY --from=builder /build/api ./api
+COPY --from=builder /build/api_v2 ./api_v2
+COPY --from=builder /build/search ./search
+COPY --from=builder /build/templates ./templates
+COPY --from=builder /build/staticfiles ./staticfiles
+COPY --from=builder /build/db.sqlite3 ./db.sqlite3
+COPY --from=builder /build/manage.py ./manage.py
+COPY --from=builder /build/newrelic.ini ./newrelic.ini
 
 ENV PATH="/opt/services/open5e-api/.venv/bin:$PATH"
 

api/management/commands/quicksetup.py

@@ -94,7 +94,6 @@ def migrate_db() -> None:
     This command is added primarily to assist in local development, because
     checking out and changing branches results in unclean model/dbs."""
 
-    call_command('makemigrations')
     call_command('migrate')
 
 def is_dirty() ->None:

server/settings.py

@@ -121,6 +121,14 @@
     "default": {
         "ENGINE": "django.db.backends.sqlite3",
         "NAME": os.path.join(BASE_DIR, "db.sqlite3"),
+        "OPTIONS": {
+            "init_command": (
+                "PRAGMA journal_mode=WAL;"
+                "PRAGMA synchronous=NORMAL;"
+                "PRAGMA cache_size=-65536;"
+                "PRAGMA temp_store=MEMORY;"
+            ),
+        },
     }
 }