ci: печатать код возврата pytest явно (#2871) #2875
No reviewers
Labels
No labels
Fable 5 ревью
GG-форсайт
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
feedback/max
generative
needs-discussion
needs-human
observability
pause-bots
performance
priority/p0
priority/p1
priority/p2
priority/p3
scope/backend
scope/db
scope/devops
scope/frontend
scope/qa
scrapers
security
site-finder
stage/1
stage/2
status/blocked
status/done
status/needs-analysis
status/needs-fix
status/qa
status/ready
status/review
status/wip
tech-debt
tradein
ux
week ревью 1
wontfix
ИРД
вторичка
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: lekss361/gendesign#2875
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "ci/2871-pytest-exit-code"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Что показали маркеры из #2872
Прогон 7530 на ветке #2865 — все шаги отчитались успешно, и job всё равно красная:
Ни одного
::error, ни одного❌во всём логе. Оба подозреваемых из #2872 —чистые. Значит я подозревал не тех.
Остался один невидимый кандидат — сам pytest
Сводка pytest печатается до выхода процесса. Ненулевой код возврата после зелёной
сводки (внутренняя ошибка на teardown, сбой плагина, ошибка записи coverage.xml) выглядит
ровно так, как мы видим: «тесты прошли, job упала неизвестно где». Проверить это нечем,
потому что шаг код возврата не печатает.
Печатаем явно, гейт сохраняем через
exit $rc.|| rc=$?, а не отдельная строкаrc=$?— подset -eприсваивание после упавшейкоманды просто не выполнится и код снова потеряется. Третий раз за сегодня эта же
ловушка (в #2870 и #2872 тоже).
Границы
Это по-прежнему диагностика, а не починка. Если в следующем прогоне окажется
### pytest вернул код 0, значит виновата не работа шага, а учёт самого act_runner,и разбираться надо уже с раннером — но тогда это будет установлено, а не предположено.
Параллельно гоняю ту же команду локально на той же ветке с явным
echo $?— сравню.yaml.safe_load— валиденcheck yaml— passedRefs #2871