пересборка api/worker без удаления orphan-контейнеров;
проверка /api/health и docker compose ps.
Добавлены инструкции по Actions secrets: PROD_HOST, PROD_USER, PROD_SSH_KEY, PROD_KNOWN_HOSTS. Версия сервиса повышена до 0.8.2.
Добавляет CD для production:
- запуск после push в main или вручную;
- backup PostgreSQL перед обновлением;
- release-архив без .env;
- пересборка api/worker без удаления orphan-контейнеров;
- проверка /api/health и docker compose ps.
Добавлены инструкции по Actions secrets: PROD_HOST, PROD_USER, PROD_SSH_KEY, PROD_KNOWN_HOSTS. Версия сервиса повышена до 0.8.2.
Ревью полного diff PR #40. Найдены блокирующие замечания:
P1 — deploy не зависит от CI (.gitea/workflows/deploy.yml:3-6): push в main запускает production deploy независимо от результата pytest из CI. Исправить через общий workflow с needs: test или deploy по успешному workflow_run.
P1 — гонка deploy-запусков (:19, :34, :52-59): фиксированный /tmp/miem-workers-release.tar и отсутствие concurrency позволяют двум запускам перезаписать архив и одновременно пересобрать production. Нужны уникальный путь на run и concurrency group с последовательным выполнением.
P1 — CD неработоспособен без Actions secrets: PROD_HOST, PROD_USER, PROD_SSH_KEY, PROD_KNOWN_HOSTS должны быть настроены до merge; сейчас через пользователя agent их установить нельзя.
git diff --check и node --check app/static/admin.js проходят. Полный pytest в текущем окружении зависает на API TestClient, поэтому approve после исправлений и успешного CI не ставлю.
Ревью полного diff PR #40. Найдены блокирующие замечания:
1. **P1 — deploy не зависит от CI** (`.gitea/workflows/deploy.yml:3-6`): push в `main` запускает production deploy независимо от результата pytest из CI. Исправить через общий workflow с `needs: test` или deploy по успешному `workflow_run`.
2. **P1 — гонка deploy-запусков** (`:19`, `:34`, `:52-59`): фиксированный `/tmp/miem-workers-release.tar` и отсутствие `concurrency` позволяют двум запускам перезаписать архив и одновременно пересобрать production. Нужны уникальный путь на run и concurrency group с последовательным выполнением.
3. **P1 — CD неработоспособен без Actions secrets**: `PROD_HOST`, `PROD_USER`, `PROD_SSH_KEY`, `PROD_KNOWN_HOSTS` должны быть настроены до merge; сейчас через пользователя `agent` их установить нельзя.
`git diff --check` и `node --check app/static/admin.js` проходят. Полный pytest в текущем окружении зависает на API TestClient, поэтому approve после исправлений и успешного CI не ставлю.
deploy теперь находится в CI workflow и имеет needs: test; на PR deploy не запускается, на main — только после успешного pytest/node check.
добавлена concurrency-группа production-deploy с последовательным выполнением.
release archive получил уникальное имя по github.run_id; независимый .gitea/workflows/deploy.yml удалён.
README синхронизирован с новой схемой.
git diff --check и node --check app/static/admin.js проходят. Code-level замечаний по текущему diff нет. Перед merge остаётся операционный prerequisite: настроить Actions secrets PROD_HOST, PROD_USER, PROD_SSH_KEY, PROD_KNOWN_HOSTS; без них deploy job закономерно завершится ошибкой.
Повторное ревью после коммита `880ac6c`:
- deploy теперь находится в CI workflow и имеет `needs: test`; на PR deploy не запускается, на main — только после успешного pytest/node check.
- добавлена `concurrency`-группа `production-deploy` с последовательным выполнением.
- release archive получил уникальное имя по `github.run_id`; независимый `.gitea/workflows/deploy.yml` удалён.
- README синхронизирован с новой схемой.
`git diff --check` и `node --check app/static/admin.js` проходят. Code-level замечаний по текущему diff нет. Перед merge остаётся операционный prerequisite: настроить Actions secrets `PROD_HOST`, `PROD_USER`, `PROD_SSH_KEY`, `PROD_KNOWN_HOSTS`; без них deploy job закономерно завершится ошибкой.
admin
merged commit 5d89ae3891 into main2026-09-11 12:17:49 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Добавляет CD для production:
Добавлены инструкции по Actions secrets: PROD_HOST, PROD_USER, PROD_SSH_KEY, PROD_KNOWN_HOSTS. Версия сервиса повышена до 0.8.2.
Ревью полного diff PR #40. Найдены блокирующие замечания:
.gitea/workflows/deploy.yml:3-6): push вmainзапускает production deploy независимо от результата pytest из CI. Исправить через общий workflow сneeds: testили deploy по успешномуworkflow_run.:19,:34,:52-59): фиксированный/tmp/miem-workers-release.tarи отсутствиеconcurrencyпозволяют двум запускам перезаписать архив и одновременно пересобрать production. Нужны уникальный путь на run и concurrency group с последовательным выполнением.PROD_HOST,PROD_USER,PROD_SSH_KEY,PROD_KNOWN_HOSTSдолжны быть настроены до merge; сейчас через пользователяagentих установить нельзя.git diff --checkиnode --check app/static/admin.jsпроходят. Полный pytest в текущем окружении зависает на API TestClient, поэтому approve после исправлений и успешного CI не ставлю.Повторное ревью после коммита
880ac6c:needs: test; на PR deploy не запускается, на main — только после успешного pytest/node check.concurrency-группаproduction-deployс последовательным выполнением.github.run_id; независимый.gitea/workflows/deploy.ymlудалён.git diff --checkиnode --check app/static/admin.jsпроходят. Code-level замечаний по текущему diff нет. Перед merge остаётся операционный prerequisite: настроить Actions secretsPROD_HOST,PROD_USER,PROD_SSH_KEY,PROD_KNOWN_HOSTS; без них deploy job закономерно завершится ошибкой.