chore(ops): выставить +x на docker-prune.sh и restore.sh #2889
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#2889
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "chore/ops-scripts-exec-bit"
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?
Проблема
Хвост от #2887/#2888, замеченный при проверке прода после деплоя.
ops/docker-prune.shиops/restore.shзакоммичены как100644, хотя соседи по каталогу —100755:На VM оба файла лежат с
+x, поэтомуgit statusв/opt/gendesignпоказывает их как изменённые постоянно:Контент идентичен —
1 file changed, 0 insertions(+), 0 deletions(-), разница только в бите.Почему это стоит починить
Функционально ничего не ломается: cron зовёт
bash /opt/gendesign/ops/docker-prune.sh,restore.shзапускают руками так же, — бит не участвует. Уrestore.shэтот перекос живёт давно.Чинить стоит не ради бита, а ради сигнала.
git statusна проде — единственный дешёвый способ увидеть, что файл правили руками мимо пайплайна. Пока в нём висят две вечные строки, шум становится фоном, и на его месте перестаёшь замечать настоящую правку.Что сделано
git update-index --chmod=+xна оба файла — режим приведён к тому, что реально на диске и что уже стоит у соседних скриптов. Изменений контента нет, diff состоит только из смены mode.Test plan
git ls-files -s ops/— оба теперь100755, в одном ряду сbackup.shиuptime-healthcheck.shgit status --shortв/opt/gendesignпо этим двум файлам чистыйДеплой на этот PR не сработает:
ops/restore.shне входит вpaths:deploy.yml, аops/docker-prune.shпопал туда только что (#2888). Режим доедет со следующим деплоем от любого другого изменения — торопиться некуда, на работу скриптов это не влияет.Оба скрипта закоммичены как 100644, хотя соседние по каталогу backup.sh и uptime-healthcheck.sh — 100755. На прод-VM файлы лежат с +x, поэтому `git status` в /opt/gendesign постоянно показывает их как modified: M ops/docker-prune.sh M ops/restore.sh (old mode 100644 / new mode 100755, контент идентичен) Функционально это ничего не ломает: cron зовёт docker-prune.sh через `bash <путь>`, restore.sh запускают руками так же. Проблема в другом — постоянный M в прод-репозитории обесценивает единственный дешёвый сигнал, по которому видно ручную правку файла на проде. Шум надо убирать, а не привыкать к нему. Приводим режим к тому, что реально на диске и что уже стоит у соседей.