Skip to content

fix react dev celery service name - #676

Merged
TThanos3000 merged 1 commit into
masterfrom
fix/dev-075-react-dev-celery-service
Jul 31, 2026
Merged

fix react dev celery service name#676
TThanos3000 merged 1 commit into
masterfrom
fix/dev-075-react-dev-celery-service

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Что исправлено

  • Автодеплой приведён в соответствие с фактической конфигурацией React-dev.
  • Compose service фонового обработчика изменён с celery на celerys.
  • Название сервиса зафиксировано в одной константе CELERY_SERVICE.
  • Поиск контейнера продолжает выполняться через Docker Compose labels.
  • Build, deploy, rollback и проверка статуса используют web и celerys.
  • Redis, nginx, database и production не затрагиваются.
  • Документация обновлена в соответствии с фактическими labels сервера.

Причина

Первый реальный запуск DEV-075 безопасно остановился на preflight-проверке:

Expected Compose service is missing: celery.

Проверка labels React-dev показала фактические services:

web
celerys
redis

Код, миграции и контейнеры при этом не изменялись.

Проверка

  • bash -n scripts/deploy_react_dev.sh — успешно.
  • Обязательные Compose services: web, celerys, redis.
  • Build выполняется только для web и celerys.
  • Deploy и rollback пересоздают только web и celerys.
  • git diff --check — успешно.
  • Изменены только deploy script и документация.
  • Post-merge Backend PostgreSQL CI — успешно.

Уточнение после первого зелёного workflow

Backend React-dev deploy #2 завершился зелёным, но лог показал, что docker compose run прочитал оставшуюся часть deploy-скрипта из stdin. Фактически были подтверждены сборка image и Django system check, но миграции, пересоздание web/celerys и финальный HTTPS health-check этим запуском не подтверждены.

Нужен отдельный исправляющий PR: запускать временные Compose containers без TTY и с stdin, подключённым к /dev/null, затем повторно подтвердить полный post-merge deploy.

Риски и ограничения

  • Автоматический rollback миграций не выполняется.
  • Новые миграции должны оставаться backward-compatible с предыдущим image.
  • Production и старый backend /root/api не затрагиваются.

Roadmap

Roadmap-IDs: DEV-075
Roadmap-Partial: DEV-075

@TThanos3000
TThanos3000 merged commit accd9d5 into master Jul 31, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant