fix: Move game creator to Django (#1798)

* Move game creator logic to Django

* Merge master

* Merge master

* Finish merging conflicts

* Move serializers to models.py

* Fix tests

* Fix tests again

* Fix tests again again

* Clean pipfiles and dependencies

* Clean up, increase local fleet to 15 and reduce retry from 60 to 10

* Try adding back pytest as test package for ci

* Add back all dev packages to setup.py

* Test removing the patch method (not needed?)

* Clean tests

* Merge conflicts

* Try using prints instead of logger

* Remove final logger

* Replace views loggers with print

* Recreate game service on worksheet change

* Cleanup

* Feedback

* Delete unused endpoint

* Fix test

* Add URL log

.github/workflows/ci.yml

@@ -82,28 +82,6 @@ jobs:
       - name: Upload coverage to Codecov
         uses: codecov/codecov-action@v3
 
-  test-aimmo-game-creator:
-    name: Test aimmo-game-creator
-    runs-on: ubuntu-20.04
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-        with:
-          fetch-depth: 0
-      - name: Build docker image
-        run: |
-          cd aimmo-game-creator
-          docker build -t ocadotechnology/aimmo-game-creator:test --target tester .
-      - name: Run tests
-        run: docker run -i -p 5000:5000 ocadotechnology/aimmo-game-creator:test /bin/bash -c "coverage run setup.py test && coverage xml"
-      - name: Get the coverage data from the container
-        run: |
-          read CONTAINERID <<< $(docker ps --all -q | head -1)
-          docker cp -a $CONTAINERID:.coverage $(pwd)/.coverage
-          docker cp -a $CONTAINERID:coverage.xml $(pwd)/coverage.xml
-      - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@v3
-
   test-aimmo-game-worker:
     name: Test aimmo-game-worker
     runs-on: ubuntu-20.04

.github/workflows/publish-docker-images.yml

@@ -42,17 +42,6 @@ jobs:
           push: true
           cache-from: type=local,src=/tmp/.buildx-cache
           cache-to: type=local,dest=/tmp/.buildx-cache
-      - name: Build and push aimmo-game-creator image
-        id: game_creator
-        uses: docker/build-push-action@v3
-        with:
-          context: aimmo-game-creator
-          file: aimmo-game-creator/Dockerfile
-          tags: ocadotechnology/aimmo-game-creator:${{ steps.docker_tag.outputs.tag }}
-          target: runner
-          push: true
-          cache-from: type=local,src=/tmp/.buildx-cache
-          cache-to: type=local,dest=/tmp/.buildx-cache
       - name: Image digest
         run: |
           echo ${{ steps.game.outputs.digest }}

Pipfile

@@ -4,22 +4,26 @@ verify_ssl = true
 name = "pypi"
 
 [dev-packages]
+black = "*"
+codeforlife-portal = "*"
+django-import-export = "*"
+django-test-migrations = "==1.2.0"
 pytest = "~=6.2"
+pytest-cov = "*"
 pytest-django = "==4.5.2"
 pytest-pythonpath = "*"
-codeforlife-portal = "*"
-pytest-cov = "*"
-black = "*"
-django-test-migrations = "==1.2.0"
-django-import-export = "*"
 
 [packages]
 aimmo = {editable = true, path = "."}
-kubernetes = "==26.1.0"
-eventlet = "*"
+django-formtools = "==2.2"
+django-otp = "==1.0.2"
 docker = "<6"
-wheel = "*"
+eventlet = "*"
 importlib-metadata = "<5" # Using version 5 causes an issue when trying to run pytest. Not sure why, linked to: https://stackoverflow.com/questions/73929564/entrypoints-object-has-no-attribute-get-digital-ocean
+kubernetes = "==26.1.0"
+pyyaml = "==5.4.1"
+setuptools = "==62.1.0"
+wheel = "*"
 
 [requires]
 python_version = "3.8"

agones/fleet.yml

@@ -3,7 +3,7 @@ kind: Fleet
 metadata:
   name: aimmo-game
 spec:
-  replicas: 3
+  replicas: 15
   strategy:
     type: Recreate
   template: