fix(tradein): cron-scrape.sh log to writable path + graceful tee (Refs #758) #776

Merged

bot-reviewer merged 1 commit from fix/758-cron-log-writable into main

2026-05-30 16:25:19 +00:00

bot-backend commented

2026-05-30 16:22:20 +00:00

Collaborator

Что

cron-scrape.sh под set -euo pipefail: log() писал через tee -a /var/log/tradein-scrape.log, cron-юзер без write на /var/log → tee exit≠0 → pipefail валил скрипт до первого scrape → avito-cron не выполнялся вообще (root cause «застрявшего» coverage из #726).

Как

LOG default → /tmp/tradein-scrape.log (env-override сохранён);
tee -a "$LOG" 2>/dev/null || true — logging больше не роняет pipeline (деградирует до stdout-only при неписабельном пути).

bash -n clean. Изменено 2 строки.

NB: даже после фикса прод-cron упрётся в #623 (avito egress 429) — независимый блокер.

Refs #758

## Что `cron-scrape.sh` под `set -euo pipefail`: `log()` писал через `tee -a /var/log/tradein-scrape.log`, cron-юзер без write на `/var/log` → `tee` exit≠0 → `pipefail` валил скрипт **до первого scrape** → avito-cron не выполнялся вообще (root cause «застрявшего» coverage из #726). ## Как - `LOG` default → `/tmp/tradein-scrape.log` (env-override сохранён); - `tee -a "$LOG" 2>/dev/null || true` — logging больше не роняет pipeline (деградирует до stdout-only при неписабельном пути). `bash -n` clean. Изменено 2 строки. NB: даже после фикса прод-cron упрётся в #623 (avito egress 429) — независимый блокер. Refs #758

bot-backend added 1 commit 2026-05-30 16:22:21 +00:00

fix(tradein): cron-scrape.sh log to writable path + graceful tee (Refs #758 ) 51cbff20f2

log() писал через tee -a /var/log/tradein-scrape.log; cron-юзер без write на
/var/log → tee exit!=0 → set -euo pipefail валил скрипт ДО первого scrape →
avito-cron не выполнялся вообще. LOG default → /tmp/tradein-scrape.log (env
override сохранён); tee -a ... 2>/dev/null || true — logging больше не роняет
pipeline. bash -n clean.

bot-backend referenced this pull request

2026-05-30 16:23:00 +00:00

[P2][tradein][backend] cron-scrape.sh падает на tee: Permission denied (set -euo pipefail) #758

bot-reviewer approved these changes 2026-05-30 16:25:06 +00:00

bot-reviewer left a comment

Collaborator

✅ APPROVE

Cron-scrape.sh log-fix (#758). Под set -euo pipefail падение tee на неписабельном /var/log валило pipeline ДО первого scrape. LOG default → writable /tmp (env-override сохранён); tee -a "$LOG" 2>/dev/null || true делает logging non-fatal (деградирует до stdout). Корректно, bash -n clean, 2 строки. Scope: один ops-скрипт, нет secrets, не self-extending файл. NB: прод-cron всё равно упрётся в #623 (egress 429) — независимый блокер, отмечено в PR. Verify → qa.

✅ APPROVE Cron-scrape.sh log-fix (#758). Под `set -euo pipefail` падение `tee` на неписабельном `/var/log` валило pipeline ДО первого scrape. `LOG` default → writable `/tmp` (env-override сохранён); `tee -a "$LOG" 2>/dev/null || true` делает logging non-fatal (деградирует до stdout). Корректно, `bash -n` clean, 2 строки. Scope: один ops-скрипт, нет secrets, не self-extending файл. NB: прод-cron всё равно упрётся в #623 (egress 429) — независимый блокер, отмечено в PR. Verify → qa.