lekss361/gendesign

Fork 0

[P2] Миграции tradein: порядок (до рестарта) + реконсиляция 152 vs 131 + дубль-префиксы #2216

New issue

Closed

opened 2026-07-02 19:19:14 +00:00 by lekss361 · 1 comment

lekss361 commented

2026-07-02 19:19:14 +00:00

Owner

Эпик: #2200

Находки (Backend/DB, Medium, verified):

Миграции применяются ПОСЛЕ рестарта app-контейнеров: deploy-tradein.yml:320 up -d ПЕРЕД циклом :369-387; при сбое миграции новый код бесконечно на старой схеме (500/тихо-неверно). deploy.md утверждает обратный порядок (верно для main, ложно для tradein).
История схемы нереспроизводима: прод _schema_migrations=152 vs ls data/sql=131 файл (21 переименована/удалена); трекинг по bare-filename → переименование применённого файла ре-выполнит его на деплое.
6 дубль-префиксов (084/108/113/121/124/130) → неоднозначный порядок для людей.

Acceptance:

Миграции применяются ДО up -d новых контейнеров (или с rollback-веткой на сбое).
Реконсилировать 152 vs 131 (задокументировать/восстановить недостающие имена; убедиться, что fresh-bootstrap = прод-lineage).
Разрулить дубль-префиксы; закрепить, что переименование применённой миграции запрещено.

Scope: .forgejo/workflows/deploy-tradein.yml, tradein-mvp/backend/data/sql/.

Эпик: #2200 **Находки (Backend/DB, Medium, verified):** - Миграции применяются ПОСЛЕ рестарта app-контейнеров: `deploy-tradein.yml:320 up -d` ПЕРЕД циклом `:369-387`; при сбое миграции новый код бесконечно на старой схеме (500/тихо-неверно). `deploy.md` утверждает обратный порядок (верно для main, ложно для tradein). - История схемы нереспроизводима: прод `_schema_migrations`=152 vs `ls data/sql`=131 файл (21 переименована/удалена); трекинг по bare-filename → переименование применённого файла ре-выполнит его на деплое. - 6 дубль-префиксов (084/108/113/121/124/130) → неоднозначный порядок для людей. **Acceptance:** - [ ] Миграции применяются ДО `up -d` новых контейнеров (или с rollback-веткой на сбое). - [ ] Реконсилировать 152 vs 131 (задокументировать/восстановить недостающие имена; убедиться, что fresh-bootstrap = прод-lineage). - [ ] Разрулить дубль-префиксы; закрепить, что переименование применённой миграции запрещено. Scope: `.forgejo/workflows/deploy-tradein.yml`, `tradein-mvp/backend/data/sql/`.

lekss361 added the

labels 2026-07-02 19:20:27 +00:00

lekss361 referenced this issue

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

EPIC: Мера (trade-in) — ремедиация полного аудита 2026-07-02 #2200

bot-backend referenced this issue from a commit

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

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

lekss361 referenced this issue

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

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

lekss361 referenced this issue from a commit

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

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

lekss361 commented

2026-07-02 21:10:49 +00:00

Author

Owner

Сделано в PR #2228 (смержен; первый деплой с новым порядком прошёл success, health 200).

Acceptance:

✅ Миграции ДО up -d новых контейнеров: pull → up только postgres → pg_isready-цикл (30×2s, fail=exit 1) → блок миграций → up app-контейнеров. Сбой миграции = exit 1 до нового кода (старые контейнеры остаются на консистентной паре код↔схема). docker restart tradein-backend стал не нужен и удалён.
✅ Реконсиляция 152 vs 131 — не актуальна: прод _schema_migrations и репо data/sql полностью совпадают (154 == 154, diff пуст, проверено на проде 2026-07-02; аудит снят со стейл-ветки).
✅ Дубль-префиксы + запрет переименования: _manifest_applied.txt + test_migrations_manifest.py (pre-merge CI): red при rename/rm применённой миграции и при новом дубль-NN; 6 исторических дублей (084/108/113/121/124/130) grandfathered. Переименовывать их НЕЛЬЗЯ (tracking по bare-filename → повторное выполнение на проде) — теперь это машинно-запрещено, а не только договорённость.

Minor из ревью (health-check не валит job при мёртвом backend — pre-existing) закрывается в #2214.

Сделано в PR #2228 (смержен; первый деплой с новым порядком прошёл success, health 200). Acceptance: - ✅ **Миграции ДО `up -d` новых контейнеров:** pull → up только postgres → pg_isready-цикл (30×2s, fail=exit 1) → блок миграций → up app-контейнеров. Сбой миграции = exit 1 до нового кода (старые контейнеры остаются на консистентной паре код↔схема). `docker restart tradein-backend` стал не нужен и удалён. - ✅ **Реконсиляция 152 vs 131 — не актуальна:** прод `_schema_migrations` и репо `data/sql` полностью совпадают (154 == 154, diff пуст, проверено на проде 2026-07-02; аудит снят со стейл-ветки). - ✅ **Дубль-префиксы + запрет переименования:** `_manifest_applied.txt` + `test_migrations_manifest.py` (pre-merge CI): red при rename/rm применённой миграции и при новом дубль-NN; 6 исторических дублей (084/108/113/121/124/130) grandfathered. Переименовывать их НЕЛЬЗЯ (tracking по bare-filename → повторное выполнение на проде) — теперь это машинно-запрещено, а не только договорённость. Minor из ревью (health-check не валит job при мёртвом backend — pre-existing) закрывается в #2214.

lekss361 closed this issue

2026-07-02 21:10:50 +00:00

lekss361 referenced this issue

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

fix(tradein/ops): health-check валит деплой + mem-лимиты + log-rotation (#2214) #2231

lekss361 referenced this issue

2026-07-02 21:23:50 +00:00

EPIC: Мера (trade-in) — ремедиация полного аудита 2026-07-02 #2200

lekss361 referenced this issue

2026-07-03 05:53:17 +00:00

fix(tradein): оставить domklik активным — отключить его TTL-деактивацию (#2204) #2253

lekss361 referenced this issue