Add "latest" integer table in database & switch to blue-green deployment strategy; add migration script #37

Merged

marfavi merged 10 commits from migration into main

2026-03-11 19:27:38 +01:00

marfavi commented

2026-03-10 13:38:12 +01:00

Owner

Fixes #30
Fixes #16

Fixes #30 Fixes #16

marfavi added 8 commits

2026-03-10 13:38:12 +01:00

feat(deploy): implement blue-green deployment with Nginx switching bd40f256c0

Previously the deploy script stopped the running container before
starting the new one, causing downtime on every deploy.

Changes:
- docker-compose.yml: remove container_name (required for two parallel stacks), bind ports to 127.0.0.1 only, add healthcheck
- docker-compose.test.yml: move test port from 5002 → 5003 to avoid collision with the green stack
- Makefile: add blue/green variants of run and stop targets
- deploy.sh: replace stop-then-start with a full blue-green cutover
- add default nginx config files
- deploy/README.md: document Nginx install

add table for "latest" int and add zero-downtime migration 1a2f589569

change nginx upstream location 3173ca8746

add .DS_Store to .gitignore 2a35e3c3ba

feat: add health check endpoint to simulator API ac2c394eab

Extract one-time migration from deploy.sh into migrate.sh c3063c32b8

Add migration.typ documenting the migration plan and deployment stack, and write a script that follows this plan (migrate.sh). Remove initial migration logic from deploy.sh.

deploy.sh now requires nginx to already be running (via migrate.sh) and only handles blue-green switching.

- Add migrate.sh with pre-flight checks, cleanup trap, and health verification via /api/health/live
- Simplify deploy.sh by cleaning up comments
- Update README.md to reference migrate.sh

Add command availability checks in migration script 17bc53c0c1

add confirmation prompt for migration when nginx is running on port 8080 and ensure write permissions for the database cbd4ee7e9a

marfavi referenced this pull request

2026-03-10 13:40:21 +01:00

fix: "latest" integer in AppState isn’t persistent between runs issue #25

marfavi added 1 commit

2026-03-10 13:41:14 +01:00

Remove ambigious character 04c28d423d

balthazar approved these changes

2026-03-10 14:08:37 +01:00

balthazar left a comment

Good structuring with functions.

Good with piping into nothing for clean output

I was a bit stumped by choice of port 8080, but ultimately it is just another temporary port for the healthcheck, maybe some docs on that and how nginx listens to that.

There are some changes to things other than migration and deployment that confuses me, and maybe should have been split out into another PR as this one is already quite big.

Ultimately, LGTM 🚀

Good structuring with functions. Good with piping into nothing for clean output I was a bit stumped by choice of port 8080, but ultimately it is just another temporary port for the healthcheck, maybe some docs on that and how nginx listens to that. There are some changes to things other than migration and deployment that confuses me, and maybe should have been split out into another PR as this one is already quite big. Ultimately, LGTM 🚀

marfavi added 1 commit

2026-03-11 19:26:54 +01:00

fix default minitwit upstream port to 5002 9674deccb9