fix(ops): деплой триггерится на любой ops/*.sh, а не только docker-prune.sh (#2203)
All checks were successful
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / changes (pull_request) Successful in 9s
CI / frontend-tests (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 8s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m6s
CI / backend-tests (pull_request) Successful in 17m11s
All checks were successful
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / changes (pull_request) Successful in 9s
CI / frontend-tests (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 8s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m6s
CI / backend-tests (pull_request) Successful in 17m11s
paths-filter в deploy.yml знал только про ops/docker-prune.sh (#2887) — правка ops/backup.sh или новый ops/restore-drill.sh из этого же PR не долетели бы до /opt/gendesign: деплой не триггерится -> git reset --hard origin/main не исполняется -> cron на VM месяцами крутит старую версию, молча. Точечный список сам по себе и есть баг: #2887 добавил только тот файл, о котором тогда шла речь, и следующий новый ops-скрипт (backup.sh) остался за бортом. Глоб ops/*.sh закрывает класс целиком — не матчит подпути (ops/db-bootstrap/**, ops/glitchtip-auth-forwarder/**), у них свои explicit триггеры уже есть, дублирования нет.
This commit is contained in:
parent
7691db8d2c
commit
2d2336cd51
1 changed files with 7 additions and 1 deletions
|
|
@ -32,7 +32,13 @@ on:
|
||||||
# по cron из /opt/gendesign/ops/, куда попадает только через `git reset --hard`
|
# по cron из /opt/gendesign/ops/, куда попадает только через `git reset --hard`
|
||||||
# шага деплоя. Без этой строки правка скрипта лежала бы в main, а cron месяцами
|
# шага деплоя. Без этой строки правка скрипта лежала бы в main, а cron месяцами
|
||||||
# исполнял бы старую версию — молча и без единого сигнала.
|
# исполнял бы старую версию — молча и без единого сигнала.
|
||||||
- "ops/docker-prune.sh"
|
# Глоб, а не точечный список (#2203): класс бага — «любой ops-скрипт,
|
||||||
|
# запускаемый по cron с VM», не только docker-prune.sh. Сейчас сюда попадают
|
||||||
|
# backup.sh, restore-drill.sh, restore.sh, uptime-healthcheck.sh — точечное
|
||||||
|
# перечисление пришлось бы дополнять при каждом новом скрипте, и про это
|
||||||
|
# снова забыли бы (см. как этот самый комментарий выше был точечным про
|
||||||
|
# docker-prune.sh и не спас backup.sh). Глоб закрывает класс целиком.
|
||||||
|
- "ops/*.sh"
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
|
|
||||||
# #2950: ОБЩАЯ группа с deploy-tradein.yml — не опечатка и не копипаста.
|
# #2950: ОБЩАЯ группа с deploy-tradein.yml — не опечатка и не копипаста.
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue