fix(tradein/deploy): миграции ДО подъёма app-контейнеров + manifest-инвариант (#2216) #2228

Merged

lekss361 merged 1 commit from fix/tradein-migrations-before-up into main

2026-07-02 21:03:38 +00:00

lekss361 commented

2026-07-02 20:51:47 +00:00

Owner

Summary

Порядок деплоя: pull → up -d --no-deps postgres (только БД) → pg_isready-цикл (30×2s, fail=exit 1) → весь блок миграций (внутренности байт-идентичны, только перемещены) → reader-password bootstrap → up -d browser backend frontend (+условный scraper) → Caddy reload → health → cleanup. Сбой миграции = exit 1 до подъёма нового кода — старые контейнеры остаются на консистентной паре код↔схема. docker restart tradein-backend удалён (backend стартует после миграций, FDW server уже существует; история сохранена в комментарии).
Реконсиляция «152 vs 131» не актуальна: прод _schema_migrations и репо data/sql совпадают (154 == 154, diff пуст; проверено на проде 2026-07-02 — аудит снят со стейл-ветки).
Инвариант против rename-trap: _manifest_applied.txt (154 имени) + test_migrations_manifest.py — red при переименовании/удалении применённой миграции (tracking по bare-filename → rename = повторное выполнение на проде) и при дубль-NN-префиксе новых файлов; 6 исторических дублей (084/108/113/121/124/130) grandfathered. Manifest дополняется автором в том же PR (подсказка в assert).

Как проверено

Полный локальный pytest: 2977 passed, 6 skipped, 1 deselected (включая 4 новых manifest-теста; negative-пробы rename/dup-prefix дают red — проверено воркером).
YAML валиден; ruff чисто. code-reviewer: ✅ APPROVE — построчная сверка перемещённых блоков (потерянных строк/env нет), pg_isready-цикл корректен под set -e, первый прогон нового порядка на текущем проде no-op-безопасен.
Minor из ревью (health-check не валит job при мёртвом backend — pre-existing) = предмет #2214, будет закрыт там.

Refs #2216

## Summary - **Порядок деплоя:** `pull` → `up -d --no-deps postgres` (только БД) → `pg_isready`-цикл (30×2s, fail=exit 1) → весь блок миграций (внутренности байт-идентичны, только перемещены) → reader-password bootstrap → `up -d browser backend frontend` (+условный scraper) → Caddy reload → health → cleanup. Сбой миграции = exit 1 **до** подъёма нового кода — старые контейнеры остаются на консистентной паре код↔схема. `docker restart tradein-backend` удалён (backend стартует после миграций, FDW server уже существует; история сохранена в комментарии). - **Реконсиляция «152 vs 131» не актуальна:** прод `_schema_migrations` и репо `data/sql` совпадают (154 == 154, diff пуст; проверено на проде 2026-07-02 — аудит снят со стейл-ветки). - **Инвариант против rename-trap:** `_manifest_applied.txt` (154 имени) + `test_migrations_manifest.py` — red при переименовании/удалении применённой миграции (tracking по bare-filename → rename = повторное выполнение на проде) и при дубль-NN-префиксе новых файлов; 6 исторических дублей (084/108/113/121/124/130) grandfathered. Manifest дополняется автором в том же PR (подсказка в assert). ## Как проверено - Полный локальный pytest: **2977 passed, 6 skipped, 1 deselected** (включая 4 новых manifest-теста; negative-пробы rename/dup-prefix дают red — проверено воркером). - YAML валиден; ruff чисто. code-reviewer: ✅ APPROVE — построчная сверка перемещённых блоков (потерянных строк/env нет), pg_isready-цикл корректен под set -e, **первый прогон нового порядка на текущем проде no-op-безопасен**. - Minor из ревью (health-check не валит job при мёртвом backend — pre-existing) = предмет #2214, будет закрыт там. Refs #2216

Rows
Columns