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

Merged

lekss361 merged 1 commit from fix/tradein-observability-hardening into main

2026-07-02 21:29:31 +00:00

lekss361 commented

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

Owner

Summary

Health-check больше не декоративен: 30 фейлов /health подряд → exit 1 до записи success-маркера .tradein-deployed-sha — мёртвый backend не помечается задеплоенным, changes-job следующего прогона берёт корректную базу. Порядок миграций #2216 не задет; set -e семантика сохранена (curl в if-условии).
mem-лимиты всем 5 tradein-сервисам от прод-замеров (docker stats): postgres 3g (idle 656MiB; дефолтный shared_buffers=128MB — запас на PostGIS-спайки), backend 768m (WeasyPrint 200-500MB поверх idle 100MiB), browser 2560m/3g (swap-cushion, BROWSER_CONCURRENCY=1), scraper 640m, frontend 384m. memswap_limit == mem_limit — tradein перестаёт докидывать в общий своп; restart: unless-stopped у всех → OOM-kill = авто-recovery, не мёртвый сервис (проверено ревью).
Log-rotation: json-file 20m×3 каждому сервису (YAML anchor) — закрывает unbounded-рост логов.
Premise аудита «коробка 4GB» устарел: фактически 11.96G RAM (замер в комментарии diff).

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

docker compose config exit 0; YAML валиден; anchor резолвится во все 5 сервисов.
code-reviewer: ⚠️ MINOR → учтено в этом же PR (postgres 2g→3g, backend 640m→768m); restart policy у всех сервисов verified — BLOCK-условие не сработало.
⚠️ Первый деплой пересоздаст ВСЕ tradein-контейнеры (включая scraper — прервёт бегущий sweep; stop_grace_period 120s; допустимо разово).

Оставшееся по #2214 (needs-human, вне PR)

Off-box uptime-probe: нужен внешний хост/сервис (вариант (а) UptimeRobot/BetterUptime free tier — 5-мин HTTP+keyword, zero-infra; вариант (б) docker-compose.uptime.yml на другом VPS). Мониторить https://gendsgn.ru/ и /trade-in; /health наружу не проброшен (только docker exec). Решение и доступ — за человеком.

Refs #2214

## Summary - **Health-check больше не декоративен:** 30 фейлов `/health` подряд → `exit 1` **до** записи success-маркера `.tradein-deployed-sha` — мёртвый backend не помечается задеплоенным, changes-job следующего прогона берёт корректную базу. Порядок миграций #2216 не задет; set -e семантика сохранена (curl в if-условии). - **mem-лимиты всем 5 tradein-сервисам** от прод-замеров (`docker stats`): postgres **3g** (idle 656MiB; дефолтный shared_buffers=128MB — запас на PostGIS-спайки), backend **768m** (WeasyPrint 200-500MB поверх idle 100MiB), browser 2560m/3g (swap-cushion, BROWSER_CONCURRENCY=1), scraper 640m, frontend 384m. `memswap_limit == mem_limit` — tradein перестаёт докидывать в общий своп; `restart: unless-stopped` у всех → OOM-kill = авто-recovery, не мёртвый сервис (проверено ревью). - **Log-rotation:** json-file 20m×3 каждому сервису (YAML anchor) — закрывает unbounded-рост логов. - **Premise аудита «коробка 4GB» устарел:** фактически 11.96G RAM (замер в комментарии diff). ## Как проверено - `docker compose config` exit 0; YAML валиден; anchor резолвится во все 5 сервисов. - code-reviewer: ⚠️ MINOR → учтено в этом же PR (postgres 2g→3g, backend 640m→768m); restart policy у всех сервисов verified — BLOCK-условие не сработало. - ⚠️ Первый деплой пересоздаст ВСЕ tradein-контейнеры (включая scraper — прервёт бегущий sweep; stop_grace_period 120s; допустимо разово). ## Оставшееся по #2214 (needs-human, вне PR) Off-box uptime-probe: нужен внешний хост/сервис (вариант (а) UptimeRobot/BetterUptime free tier — 5-мин HTTP+keyword, zero-infra; вариант (б) docker-compose.uptime.yml на другом VPS). Мониторить https://gendsgn.ru/ и /trade-in; /health наружу не проброшен (только docker exec). Решение и доступ — за человеком. Refs #2214

Rows
Columns