fix(tradein/deploy): atomic recreate + graceful scraper drain + startup-reap (#1951) #2388

Merged

lekss361 merged 1 commit from fix/tradein-1951-deploy-atomicity into main

2026-07-04 06:49:07 +00:00

lekss361 commented

2026-07-04 06:48:22 +00:00

Owner

Summary

Атомарность: browser/backend/frontend/scraper пересоздаются в один docker compose up -d --no-deps $SERVICES вместо двух раздельных вызовов.
Graceful drain: перед recreate scraper — poll scrape_runs.status='running' до 5 мин (30×10s); WARNING+proceed на таймауте (существующий SIGTERM-drain + stop_grace_period 120s остаются финальной страховкой).
Startup-reap: orphaned running-строки (heartbeat старше DB-clock checkpoint'а, взятого прямо перед stop) помечаются cancelled сразу вместо ожидания 6h zombie-reaper'а. Реализовано ЦЕЛИКОМ в deploy-workflow (shell), НЕ в scheduler_main.py/scrapers — избегает коллизии с активной scraper_kit-миграцией (#2277).
Review fixup: tr -d '[:space:]' ломал внутренний пробел в timestamp-литерале (CAST(...AS timestamptz) молча падал на КАЖДОМ деплое) → заменено на sed-trim только leading/trailing. Drain-loop больше не путает неудачный psql-read с подтверждённым running_count=0 (иначе транзиентный docker-exec/DB glitch тихо пропускал весь drain-wait).

Test plan

YAML валиден (yaml.safe_load), docker compose config -q проходит
Live-repro обоих критических багов в throwaway postgres:16 контейнере (code-review) — подтверждена причина, подтверждён фикс
code-review: ❌MAJOR (2 критических) → оба пофикшены, готово к смёржу
Post-deploy: сверить _manifest_applied/логи следующего деплоя на факт срабатывания startup-reap (если найдутся orphaned rows)

Refs #1951

## Summary - **Атомарность**: browser/backend/frontend/scraper пересоздаются в один `docker compose up -d --no-deps $SERVICES` вместо двух раздельных вызовов. - **Graceful drain**: перед recreate scraper — poll `scrape_runs.status='running'` до 5 мин (30×10s); WARNING+proceed на таймауте (существующий SIGTERM-drain + `stop_grace_period 120s` остаются финальной страховкой). - **Startup-reap**: orphaned running-строки (heartbeat старше DB-clock checkpoint'а, взятого прямо перед stop) помечаются `cancelled` сразу вместо ожидания 6h zombie-reaper'а. Реализовано ЦЕЛИКОМ в deploy-workflow (shell), НЕ в `scheduler_main.py`/scrapers — избегает коллизии с активной scraper_kit-миграцией (#2277). - **Review fixup**: `tr -d '[:space:]'` ломал внутренний пробел в timestamp-литерале (`CAST(...AS timestamptz)` молча падал на КАЖДОМ деплое) → заменено на `sed`-trim только leading/trailing. Drain-loop больше не путает неудачный psql-read с подтверждённым `running_count=0` (иначе транзиентный docker-exec/DB glitch тихо пропускал весь drain-wait). ## Test plan - [x] YAML валиден (`yaml.safe_load`), `docker compose config -q` проходит - [x] Live-repro обоих критических багов в throwaway `postgres:16` контейнере (code-review) — подтверждена причина, подтверждён фикс - [x] code-review: ❌MAJOR (2 критических) → оба пофикшены, готово к смёржу - [ ] Post-deploy: сверить `_manifest_applied`/логи следующего деплоя на факт срабатывания startup-reap (если найдутся orphaned rows) Refs #1951

Rows
Columns