Compare commits
9
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
446208c8ad | ||
|
|
0c8f1f790b | ||
|
|
95a72437ec | ||
|
|
eda5694270 | ||
|
|
c443190bf4 | ||
|
|
93dd7ee0a3 | ||
|
|
daf7653635 | ||
|
|
76f9cf2e06 | ||
|
|
3c7c546aa7 |
@@ -24,11 +24,24 @@ GIT_SSH_KEY_B64= # (secret) для Docker: тот же ключ
|
|||||||
GIT_UPLOADS_URL=git@ssh.sourcecraft.dev:rogachevegorushka/tgit-uploads.git
|
GIT_UPLOADS_URL=git@ssh.sourcecraft.dev:rogachevegorushka/tgit-uploads.git
|
||||||
# GIT_CLONE_DIR=data/uploads # локальный клон репозитория загрузок (в Docker — volume /data/uploads)
|
# GIT_CLONE_DIR=data/uploads # локальный клон репозитория загрузок (в Docker — volume /data/uploads)
|
||||||
|
|
||||||
|
# --- CI репозитория загрузок: деплой ждёт зелёный check (#11, ADR-016) ---
|
||||||
|
CI_PROVIDER=none # none — разворачивать сразу после push | gitea — ждать Gitea Actions
|
||||||
|
# CI_API_URL=https://gitea.24studbank.ru
|
||||||
|
CI_TOKEN= # (secret) токен Gitea с чтением репозитория загрузок (его Actions)
|
||||||
|
# CI_REPO=HSE/tgit-uploads
|
||||||
|
# CI_WORKFLOW=check.yml
|
||||||
|
# CI_RUN_TIMEOUT_MINUTES=30
|
||||||
|
|
||||||
# --- Docker: раздел «Статус» (#14) ---
|
# --- Docker: раздел «Статус» (#14) ---
|
||||||
# В Compose задан сам: http://docker-proxy:2375. Без него (make run) «Статус» отвечает «Docker недоступен».
|
# В Compose задан сам: http://docker-proxy:2375. Без него (make run) «Статус» отвечает «Docker недоступен».
|
||||||
# DOCKER_URL=
|
# DOCKER_URL=
|
||||||
# DOCKER_LABEL=tgit.managed=true
|
# DOCKER_LABEL=tgit.managed=true
|
||||||
|
|
||||||
|
# --- Runner: деплой сервисов из репозитория загрузок (#13, infra/runner/) ---
|
||||||
|
# RUNNER_HOST=host.docker.internal # пусто — бот сервисы не разворачивает
|
||||||
|
# RUNNER_USER=tgit-deploy
|
||||||
|
# RUNNER_SSH_KEY_B64= # (secret) base64 ключа, прибитого к tgit-runner в authorized_keys
|
||||||
|
|
||||||
# --- LLM (#17) ---
|
# --- LLM (#17) ---
|
||||||
LLM_PROVIDER=mock # mock | openai_compat; при mock агент логов выключен (#18)
|
LLM_PROVIDER=mock # mock | openai_compat; при mock агент логов выключен (#18)
|
||||||
LLM_BASE_URL=
|
LLM_BASE_URL=
|
||||||
|
|||||||
@@ -12,7 +12,7 @@
|
|||||||
|---|---|
|
|---|---|
|
||||||
| [`docs/SPEC.md`](docs/SPEC.md) | ТЗ: требования A-, M-, C-, S-, L-, AL- |
|
| [`docs/SPEC.md`](docs/SPEC.md) | ТЗ: требования A-, M-, C-, S-, L-, AL- |
|
||||||
| [`docs/ARCHITECTURE.md`](docs/ARCHITECTURE.md) | модули, потоки данных, хранилище |
|
| [`docs/ARCHITECTURE.md`](docs/ARCHITECTURE.md) | модули, потоки данных, хранилище |
|
||||||
| [`docs/DECISIONS.md`](docs/DECISIONS.md) | решения ADR-001…015 с альтернативами |
|
| [`docs/DECISIONS.md`](docs/DECISIONS.md) | решения ADR-001…016 с альтернативами |
|
||||||
| [`docs/dev/DEPLOY.md`](docs/dev/DEPLOY.md) | CI/CD и деплой на сервер |
|
| [`docs/dev/DEPLOY.md`](docs/dev/DEPLOY.md) | CI/CD и деплой на сервер |
|
||||||
| [`examples/README.md`](examples/README.md) | демо-сервисы и сценарии для проверки |
|
| [`examples/README.md`](examples/README.md) | демо-сервисы и сценарии для проверки |
|
||||||
|
|
||||||
@@ -41,9 +41,11 @@
|
|||||||
| **📜 Логи** | хвост 50/200 строк или 5 мин/1 ч/24 ч; свой grep (`-i -v -E -c -C N`), выполняется в Python, не в shell; длинный ответ — файлом; журнал поисков | L-1…L-6 |
|
| **📜 Логи** | хвост 50/200 строк или 5 мин/1 ч/24 ч; свой grep (`-i -v -E -c -C N`), выполняется в Python, не в shell; длинный ответ — файлом; журнал поисков | L-1…L-6 |
|
||||||
| **🤖 Агент** | вопрос словами → фильтр → LLM-страж → LLM строит только `LogQuery` → валидация → grep; показывает построенную команду и строки; атака — отказ и 🛡 админам; 30 вопросов в сутки | L-10…L-17 |
|
| **🤖 Агент** | вопрос словами → фильтр → LLM-страж → LLM строит только `LogQuery` → валидация → grep; показывает построенную команду и строки; атака — отказ и 🛡 админам; 30 вопросов в сутки | L-10…L-17 |
|
||||||
| **🚨 Алерты** | 💥 краш, 🧠 OOM, 🔁 цикл рестартов, 🩺 unhealthy, 🔥 перегрузка, 💾 мало места, 🔌 Docker недоступен, 🛡 атака; всем участникам, с хвостом лога и кнопками; повторы — счётчик, «✅ восстановлено» ответом; список за 7 дней | AL-1…AL-6 |
|
| **🚨 Алерты** | 💥 краш, 🧠 OOM, 🔁 цикл рестартов, 🩺 unhealthy, 🔥 перегрузка, 💾 мало места, 🔌 Docker недоступен, 🛡 атака; всем участникам, с хвостом лога и кнопками; повторы — счётчик, «✅ восстановлено» ответом; список за 7 дней | AL-1…AL-6 |
|
||||||
|
| **⏳ CI/CD после push** | сообщение о push обновляется на месте: CI ⏳ → ✅/❌ → CD 🚀/⛔; CI репозитория загрузок — Gitea Actions (compile · ruff · bandit · pytest), бот находит прогон по sha и деплоит только после зелёного; автору — «✅ CI пройден» со ссылкой, упавший шаг с логом (длинный — файлом), «🚀 CD: сервисы … развёрнуты»; таймаут 30 мин; после рестарта бот продолжает с того же места | C-24, C-30…C-34, #11 |
|
||||||
|
| **🚀 Деплой сервисов** | после CI бот отправляет изменённые `services/<name>/` в `tgit-runner` на хосте: сборка образа, запуск в песочнице (uid 10001, read-only, без capabilities и сети, 128 МБ / 0.25 CPU / 64 процесса); старая версия заменяется только здоровой новой, иначе ⛔ с логом; «♻️ Перезапустить» в «Статусе» для админа | #13, S-5 |
|
||||||
| **CI/CD бота** | Gitea Actions: ruff, mypy strict, миграции up/down/up на PostgreSQL, pytest с порогом покрытия 80 %; автодеплой `dev` на сервер с проверкой health | ADR-015 |
|
| **CI/CD бота** | Gitea Actions: ruff, mypy strict, миграции up/down/up на PostgreSQL, pytest с порогом покрытия 80 %; автодеплой `dev` на сервер с проверкой health | ADR-015 |
|
||||||
|
|
||||||
Качество: 296 тестов, покрытие 92 %, `mypy --strict`, `ruff`. Telegram в тестах — фейковая сессия aiogram,
|
Качество: 357 тестов, покрытие 92 %, `mypy --strict`, `ruff`. Telegram в тестах — фейковая сессия aiogram,
|
||||||
Docker — фейковый Engine API на `httpx.MockTransport`, LLM — сценарный фейк.
|
Docker — фейковый Engine API на `httpx.MockTransport`, LLM — сценарный фейк.
|
||||||
|
|
||||||
## 3. Развёртывание и запуск
|
## 3. Развёртывание и запуск
|
||||||
@@ -106,6 +108,8 @@ CD из Gitea Actions разворачивает `dev` на сервер скр
|
|||||||
| `TELEGRAM_PROXY_URL` | нет | HTTP/SOCKS-прокси до Telegram (если сервер не видит api.telegram.org) |
|
| `TELEGRAM_PROXY_URL` | нет | HTTP/SOCKS-прокси до Telegram (если сервер не видит api.telegram.org) |
|
||||||
| `GIT_UPLOADS_URL` | да | репозиторий загрузок: путь, `ssh://…` или `git@…` |
|
| `GIT_UPLOADS_URL` | да | репозиторий загрузок: путь, `ssh://…` или `git@…` |
|
||||||
| `GIT_SSH_KEY_B64` / `GIT_SSH_KEY_PATH` | для SSH | deploy-ключ бота |
|
| `GIT_SSH_KEY_B64` / `GIT_SSH_KEY_PATH` | для SSH | deploy-ключ бота |
|
||||||
|
| `CI_PROVIDER`, `CI_TOKEN`, `CI_REPO` | нет | CI репозитория загрузок, которого ждёт деплой (`gitea`); при `none` сервисы разворачиваются сразу после push |
|
||||||
|
| `RUNNER_HOST`, `RUNNER_SSH_KEY_B64` | нет | `tgit-runner` на хосте; без него сервисы не разворачиваются |
|
||||||
| `DATABASE_URL` | нет | по умолчанию SQLite; в Compose — PostgreSQL |
|
| `DATABASE_URL` | нет | по умолчанию SQLite; в Compose — PostgreSQL |
|
||||||
| `DOCKER_URL` | нет | docker-socket-proxy; без него «Статус», «Логи» и алерты отвечают «Docker недоступен» |
|
| `DOCKER_URL` | нет | docker-socket-proxy; без него «Статус», «Логи» и алерты отвечают «Docker недоступен» |
|
||||||
| `LLM_PROVIDER`, `LLM_BASE_URL`, `LLM_MODEL`, `LLM_API_KEY` | нет | LLM для агента; при `mock` агент выключен |
|
| `LLM_PROVIDER`, `LLM_BASE_URL`, `LLM_MODEL`, `LLM_API_KEY` | нет | LLM для агента; при `mock` агент выключен |
|
||||||
@@ -136,7 +140,8 @@ make cov # pytest с покрытием (порог 80 %)
|
|||||||
|
|
||||||
- **В Телеграме**: экраны разделов, файлы (`.log` для длинных результатов, исходник для больших файлов),
|
- **В Телеграме**: экраны разделов, файлы (`.log` для длинных результатов, исходник для больших файлов),
|
||||||
QR-приглашения PNG, алерты.
|
QR-приглашения PNG, алерты.
|
||||||
- **В репозитории загрузок**: коммиты от `tgit-bot` с автором из Телеграма; push в `main` запускает CI репозитория.
|
- **В репозитории загрузок**: коммиты от `tgit-bot` с автором из Телеграма; push в `main` запускает CI репозитория
|
||||||
|
(`.gitea/workflows/check.yml`).
|
||||||
- **В БД** (13 таблиц, миграции Alembic): участники и приглашения, черновики, состояние диалогов, снапшоты
|
- **В БД** (13 таблиц, миграции Alembic): участники и приглашения, черновики, состояние диалогов, снапшоты
|
||||||
контейнеров, алерты и их доставки, журналы поисков по логам и вопросов агенту.
|
контейнеров, алерты и их доставки, журналы поисков по логам и вопросов агенту.
|
||||||
|
|
||||||
@@ -148,7 +153,9 @@ make cov # pytest с покрытием (порог 80 %)
|
|||||||
| git (SSH) | чтение и коммиты в репозиторий загрузок | `git` подпроцессом со своим ключом, без shell |
|
| git (SSH) | чтение и коммиты в репозиторий загрузок | `git` подпроцессом со своим ключом, без shell |
|
||||||
| Docker Engine API | контейнеры, stats, логи | только GET через `docker-socket-proxy` |
|
| Docker Engine API | контейнеры, stats, логи | только GET через `docker-socket-proxy` |
|
||||||
| OpenAI-совместимый `/chat/completions` | страж и планировщик агента | httpx, `response_format: json_schema` |
|
| OpenAI-совместимый `/chat/completions` | страж и планировщик агента | httpx, `response_format: json_schema` |
|
||||||
| SourceCraft REST API | статус прогонов CI репозитория загрузок | httpx, клиент готов, в UI — #11 |
|
| SSH → `tgit-runner` | деплой и перезапуск сервисов на хосте | `ssh` подпроцессом, ключ с forced command |
|
||||||
|
| Gitea Actions API | прогон CI по sha, упавший шаг, лог джоба | httpx с ретраями, `app/pipeline/gitea.py` |
|
||||||
|
| SourceCraft REST API | прогоны CI — на будущее возвращение (ADR-016) | httpx, клиент готов, к наблюдателю не подключён |
|
||||||
|
|
||||||
## 5. Ограничения и особенности
|
## 5. Ограничения и особенности
|
||||||
|
|
||||||
@@ -176,9 +183,8 @@ make cov # pytest с покрытием (порог 80 %)
|
|||||||
| повтор алерта | в течение 10 мин — счётчик, а не новое сообщение |
|
| повтор алерта | в течение 10 мин — счётчик, а не новое сообщение |
|
||||||
|
|
||||||
**Что пока не сделано** (открытые задачи):
|
**Что пока не сделано** (открытые задачи):
|
||||||
- запуск сервисов из репозитория загрузок в песочнице (`--cap-drop ALL`, read-only FS, лимиты) и перезапуск из бота —
|
- CI репозитория загрузок в SourceCraft: сейчас — Gitea Actions (ADR-016); SourceCraft подключится второй
|
||||||
`tgit-runner`, #13. Сейчас сервисы поднимаются Compose, как в `examples/`;
|
реализацией `CIProvider`, когда он примет ключ бота;
|
||||||
- уведомления о прогонах CI/CD репозитория загрузок и алерты 📦/⛔ — #11, ждут возврата к SourceCraft;
|
|
||||||
- «🔕 Тишина на час» для алертов (P2);
|
- «🔕 Тишина на час» для алертов (P2);
|
||||||
- prod-окружение: нужен отдельный бот, деплой `main` пропускается, пока его нет.
|
- prod-окружение: нужен отдельный бот, деплой `main` пропускается, пока его нет.
|
||||||
|
|
||||||
@@ -193,12 +199,14 @@ app/
|
|||||||
logs/ grep-движок и журнал
|
logs/ grep-движок и журнал
|
||||||
agent/ фильтр, страж, планировщик, схема LogQuery
|
agent/ фильтр, страж, планировщик, схема LogQuery
|
||||||
alerts/ детекторы, сервис алертов, рассылка, монитор
|
alerts/ детекторы, сервис алертов, рассылка, монитор
|
||||||
|
pipeline/ наблюдатель CI/CD после push, клиент Gitea Actions
|
||||||
|
runner/ SSH-клиент tgit-runner и деплой сервисов
|
||||||
llm/ провайдеры: OpenAI-совместимый, mock
|
llm/ провайдеры: OpenAI-совместимый, mock
|
||||||
sourcecraft/ клиент API прогонов
|
sourcecraft/ клиент API прогонов
|
||||||
db/ модели, миграции Alembic, очистка
|
db/ модели, миграции Alembic, очистка
|
||||||
config/ тексты интерфейса, пороги проверок и алертов, правила и промпты агента
|
config/ тексты интерфейса, пороги проверок и алертов, правила и промпты агента
|
||||||
tests/ pytest; фейки Telegram, Docker и LLM
|
tests/ pytest; фейки Telegram, Docker и LLM
|
||||||
examples/ демо-сервисы и сценарии
|
examples/ демо-сервисы и сценарии
|
||||||
infra/ деплой на сервер, шаблон репозитория загрузок
|
infra/ деплой бота, tgit-runner (песочница сервисов), шаблон репозитория загрузок
|
||||||
docs/ ТЗ, архитектура, ADR, процесс
|
docs/ ТЗ, архитектура, ADR, процесс
|
||||||
```
|
```
|
||||||
|
|||||||
+15
-11
@@ -18,6 +18,7 @@ from aiogram.utils.keyboard import InlineKeyboardBuilder
|
|||||||
|
|
||||||
from app.bot.menu import Nav, NavAction, free_input
|
from app.bot.menu import Nav, NavAction, free_input
|
||||||
from app.db.models import ChangeOp, User
|
from app.db.models import ChangeOp, User
|
||||||
|
from app.pipeline.watcher import PipelineWatcher, initial_status, render
|
||||||
from app.repo.drafts import DraftFull, DraftService, DraftView
|
from app.repo.drafts import DraftFull, DraftService, DraftView
|
||||||
from app.repo.git import Entry, GitError, UploadsRepo
|
from app.repo.git import Entry, GitError, UploadsRepo
|
||||||
from app.repo.publisher import Publisher, PushResult, PushStatus
|
from app.repo.publisher import Publisher, PushResult, PushStatus
|
||||||
@@ -212,7 +213,12 @@ def to_dir_keyboard(texts: Texts) -> InlineKeyboardMarkup:
|
|||||||
|
|
||||||
|
|
||||||
def build_router(
|
def build_router(
|
||||||
texts: Texts, repo: UploadsRepo, drafts: DraftService, publisher: Publisher, commit_url_template: str = ""
|
texts: Texts,
|
||||||
|
repo: UploadsRepo,
|
||||||
|
drafts: DraftService,
|
||||||
|
publisher: Publisher,
|
||||||
|
commit_url_template: str = "",
|
||||||
|
pipeline: PipelineWatcher | None = None,
|
||||||
) -> Router:
|
) -> Router:
|
||||||
router = Router(name="commit")
|
router = Router(name="commit")
|
||||||
t = texts.commit
|
t = texts.commit
|
||||||
@@ -285,22 +291,20 @@ def build_router(
|
|||||||
result: PushResult = await publisher.push(member, draft, draft.message, overwrite=overwrite)
|
result: PushResult = await publisher.push(member, draft, draft.message, overwrite=overwrite)
|
||||||
match result.status:
|
match result.status:
|
||||||
case PushStatus.OK:
|
case PushStatus.OK:
|
||||||
assert result.sha is not None
|
paths = [c.path for c in draft.changes]
|
||||||
first_line = draft.message.splitlines()[0]
|
status = pipeline.initial_status(paths) if pipeline else initial_status(False, False, paths)
|
||||||
lines = [t.pushed.format(sha=result.sha[:7], message=first_line)]
|
push = await publisher.record(member, result, draft.message, draft, msg.chat.id, msg.message_id, status)
|
||||||
if commit_url_template:
|
await msg.edit_text(render(texts, push, commit_url_template=commit_url_template))
|
||||||
lines.append(commit_url_template.format(sha=result.sha))
|
|
||||||
lines.append(t.ci_pending)
|
|
||||||
await msg.edit_text("\n".join(lines))
|
|
||||||
await publisher.record(member, result, draft.message, draft, msg.chat.id, msg.message_id)
|
|
||||||
await drafts.clear(member.tg_id)
|
await drafts.clear(member.tg_id)
|
||||||
await state.update_data(commit_virtual=[])
|
await state.update_data(commit_virtual=[])
|
||||||
|
if pipeline:
|
||||||
|
pipeline.wake()
|
||||||
case PushStatus.CONFLICT:
|
case PushStatus.CONFLICT:
|
||||||
builder = InlineKeyboardBuilder()
|
builder = InlineKeyboardBuilder()
|
||||||
builder.button(text=t.overwrite, callback_data=CommitCb(a=Act.PUSH_FORCE))
|
builder.button(text=t.overwrite, callback_data=CommitCb(a=Act.PUSH_FORCE))
|
||||||
builder.button(text=texts.menu.cancel, callback_data=CommitCb(a=Act.DRAFT))
|
builder.button(text=texts.menu.cancel, callback_data=CommitCb(a=Act.DRAFT))
|
||||||
paths = "\n".join(f"/{p}" for p in result.conflicts)
|
conflicts = "\n".join(f"/{p}" for p in result.conflicts)
|
||||||
await msg.edit_text(t.conflict.format(paths=paths), reply_markup=builder.as_markup())
|
await msg.edit_text(t.conflict.format(paths=conflicts), reply_markup=builder.as_markup())
|
||||||
case PushStatus.EMPTY:
|
case PushStatus.EMPTY:
|
||||||
await drafts.clear(member.tg_id)
|
await drafts.clear(member.tg_id)
|
||||||
await msg.edit_text(t.push_empty)
|
await msg.edit_text(t.push_empty)
|
||||||
|
|||||||
@@ -13,7 +13,8 @@ from aiogram.utils.keyboard import InlineKeyboardBuilder
|
|||||||
|
|
||||||
from app.bot.menu import Nav, NavAction
|
from app.bot.menu import Nav, NavAction
|
||||||
from app.bot.routers.commit import human_size
|
from app.bot.routers.commit import human_size
|
||||||
from app.db.models import utcnow
|
from app.db.models import Role, User, utcnow
|
||||||
|
from app.runner.client import RunnerClient
|
||||||
from app.runtime.checks import Checker, CheckName, ContainerReport
|
from app.runtime.checks import Checker, CheckName, ContainerReport
|
||||||
from app.runtime.docker import Container, Details, DockerClient, DockerError, Stats
|
from app.runtime.docker import Container, Details, DockerClient, DockerError, Stats
|
||||||
from app.texts import StatusTexts, Texts
|
from app.texts import StatusTexts, Texts
|
||||||
@@ -27,6 +28,7 @@ class StatusAct(StrEnum):
|
|||||||
LIST = "l"
|
LIST = "l"
|
||||||
CARD = "c"
|
CARD = "c"
|
||||||
CHECK = "k"
|
CHECK = "k"
|
||||||
|
RESTART = "r"
|
||||||
|
|
||||||
|
|
||||||
class StatusCb(CallbackData, prefix="s"):
|
class StatusCb(CallbackData, prefix="s"):
|
||||||
@@ -75,7 +77,10 @@ def render_list(texts: Texts, containers: list[Container], now: datetime) -> tup
|
|||||||
return _clip("\n".join(lines)), kb.as_markup()
|
return _clip("\n".join(lines)), kb.as_markup()
|
||||||
|
|
||||||
|
|
||||||
def render_card(texts: Texts, d: Details, stats: Stats | None, now: datetime) -> tuple[str, InlineKeyboardMarkup]:
|
def render_card(
|
||||||
|
texts: Texts, d: Details, stats: Stats | None, now: datetime, can_restart: bool = False
|
||||||
|
) -> tuple[str, InlineKeyboardMarkup]:
|
||||||
|
"""can_restart: an admin looking at a service deployed by tgit-runner (S-5)."""
|
||||||
t = texts.status
|
t = texts.status
|
||||||
running = d.state == "running"
|
running = d.state == "running"
|
||||||
uptime = duration(t, (now - d.started_at).total_seconds()) if running and d.started_at else t.none
|
uptime = duration(t, (now - d.started_at).total_seconds()) if running and d.started_at else t.none
|
||||||
@@ -105,11 +110,18 @@ def render_card(texts: Texts, d: Details, stats: Stats | None, now: datetime) ->
|
|||||||
kb = InlineKeyboardBuilder()
|
kb = InlineKeyboardBuilder()
|
||||||
kb.button(text=t.check_button, callback_data=StatusCb(a=StatusAct.CHECK, id=d.id[:12]))
|
kb.button(text=t.check_button, callback_data=StatusCb(a=StatusAct.CHECK, id=d.id[:12]))
|
||||||
kb.button(text=t.refresh, callback_data=StatusCb(a=StatusAct.CARD, id=d.id[:12]))
|
kb.button(text=t.refresh, callback_data=StatusCb(a=StatusAct.CARD, id=d.id[:12]))
|
||||||
|
if can_restart and runner_service(d):
|
||||||
|
kb.button(text=t.restart, callback_data=StatusCb(a=StatusAct.RESTART, id=d.id[:12]))
|
||||||
kb.button(text=t.to_list, callback_data=StatusCb(a=StatusAct.LIST))
|
kb.button(text=t.to_list, callback_data=StatusCb(a=StatusAct.LIST))
|
||||||
kb.adjust(2, 1)
|
kb.adjust(2, 1, 1)
|
||||||
return text, kb.as_markup()
|
return text, kb.as_markup()
|
||||||
|
|
||||||
|
|
||||||
|
def runner_service(d: Details) -> str | None:
|
||||||
|
"""Only services tgit-runner deployed can be restarted through it — not the bot, its DB or the proxy."""
|
||||||
|
return d.labels.get("tgit.service") or None
|
||||||
|
|
||||||
|
|
||||||
def render_report(
|
def render_report(
|
||||||
texts: Texts, reports: list[ContainerReport], now: datetime, back_id: str = ""
|
texts: Texts, reports: list[ContainerReport], now: datetime, back_id: str = ""
|
||||||
) -> tuple[str, InlineKeyboardMarkup]:
|
) -> tuple[str, InlineKeyboardMarkup]:
|
||||||
@@ -141,7 +153,9 @@ def render_report(
|
|||||||
return _clip("\n".join(lines)), kb.as_markup()
|
return _clip("\n".join(lines)), kb.as_markup()
|
||||||
|
|
||||||
|
|
||||||
def build_router(texts: Texts, docker: DockerClient | None, checker: Checker | None) -> Router:
|
def build_router(
|
||||||
|
texts: Texts, docker: DockerClient | None, checker: Checker | None, runner: RunnerClient | None = None
|
||||||
|
) -> Router:
|
||||||
router = Router(name="status")
|
router = Router(name="status")
|
||||||
t = texts.status
|
t = texts.status
|
||||||
|
|
||||||
@@ -150,7 +164,7 @@ def build_router(texts: Texts, docker: DockerClient | None, checker: Checker | N
|
|||||||
raise DockerError("docker is not configured (DOCKER_URL)")
|
raise DockerError("docker is not configured (DOCKER_URL)")
|
||||||
return render_list(texts, await docker.containers(), utcnow())
|
return render_list(texts, await docker.containers(), utcnow())
|
||||||
|
|
||||||
async def card_screen(short_id: str) -> tuple[str, InlineKeyboardMarkup]:
|
async def card_screen(short_id: str, is_admin: bool = False) -> tuple[str, InlineKeyboardMarkup]:
|
||||||
if docker is None:
|
if docker is None:
|
||||||
raise DockerError("docker is not configured (DOCKER_URL)")
|
raise DockerError("docker is not configured (DOCKER_URL)")
|
||||||
cid = await docker.resolve(short_id)
|
cid = await docker.resolve(short_id)
|
||||||
@@ -161,7 +175,7 @@ def build_router(texts: Texts, docker: DockerClient | None, checker: Checker | N
|
|||||||
stats = await docker.stats(cid)
|
stats = await docker.stats(cid)
|
||||||
except DockerError:
|
except DockerError:
|
||||||
log.warning("stats %s unavailable", short_id)
|
log.warning("stats %s unavailable", short_id)
|
||||||
return render_card(texts, details, stats, utcnow())
|
return render_card(texts, details, stats, utcnow(), can_restart=is_admin and runner is not None)
|
||||||
|
|
||||||
@router.message(F.text == texts.menu.status)
|
@router.message(F.text == texts.menu.status)
|
||||||
async def open_status(message: Message, state: FSMContext) -> None:
|
async def open_status(message: Message, state: FSMContext) -> None:
|
||||||
@@ -175,7 +189,7 @@ def build_router(texts: Texts, docker: DockerClient | None, checker: Checker | N
|
|||||||
await message.answer(text, reply_markup=kb)
|
await message.answer(text, reply_markup=kb)
|
||||||
|
|
||||||
@router.callback_query(StatusCb.filter())
|
@router.callback_query(StatusCb.filter())
|
||||||
async def on_action(query: CallbackQuery, callback_data: StatusCb) -> None:
|
async def on_action(query: CallbackQuery, callback_data: StatusCb, member: User) -> None:
|
||||||
if not isinstance(query.message, Message):
|
if not isinstance(query.message, Message):
|
||||||
await query.answer()
|
await query.answer()
|
||||||
return
|
return
|
||||||
@@ -185,7 +199,23 @@ def build_router(texts: Texts, docker: DockerClient | None, checker: Checker | N
|
|||||||
case StatusAct.LIST:
|
case StatusAct.LIST:
|
||||||
text, kb = await list_screen()
|
text, kb = await list_screen()
|
||||||
case StatusAct.CARD:
|
case StatusAct.CARD:
|
||||||
text, kb = await card_screen(callback_data.id)
|
text, kb = await card_screen(callback_data.id, member.role is Role.ADMIN)
|
||||||
|
case StatusAct.RESTART:
|
||||||
|
if docker is None or runner is None or member.role is not Role.ADMIN:
|
||||||
|
await query.answer()
|
||||||
|
return
|
||||||
|
service = runner_service(await docker.inspect(await docker.resolve(callback_data.id)))
|
||||||
|
if service is None:
|
||||||
|
await query.answer()
|
||||||
|
return
|
||||||
|
result = await runner.restart(service)
|
||||||
|
if not result.ok:
|
||||||
|
await query.answer(t.restart_failed.format(reason=result.summary)[:200], show_alert=True)
|
||||||
|
return
|
||||||
|
await query.answer(t.restarted)
|
||||||
|
text, kb = await card_screen(callback_data.id, True)
|
||||||
|
await _edit(msg, text, kb)
|
||||||
|
return
|
||||||
case StatusAct.CHECK:
|
case StatusAct.CHECK:
|
||||||
if docker is None or checker is None:
|
if docker is None or checker is None:
|
||||||
raise DockerError("docker is not configured (DOCKER_URL)")
|
raise DockerError("docker is not configured (DOCKER_URL)")
|
||||||
|
|||||||
@@ -66,6 +66,23 @@ class SourceCraftSettings(BaseSettings):
|
|||||||
max_attempts: int = Field(default=5, ge=1)
|
max_attempts: int = Field(default=5, ge=1)
|
||||||
|
|
||||||
|
|
||||||
|
class CISettings(BaseSettings):
|
||||||
|
model_config = SettingsConfigDict(**_ENV, env_prefix="CI_")
|
||||||
|
|
||||||
|
# CI of the uploads repo the bot waits for before deploying (#11). none — deploy right after push, no CI status.
|
||||||
|
provider: Literal["none", "gitea"] = "none"
|
||||||
|
api_url: str = "https://gitea.24studbank.ru"
|
||||||
|
# Read access to the uploads repo's Actions is enough
|
||||||
|
token: SecretStr = SecretStr("")
|
||||||
|
repo: str = "HSE/tgit-uploads"
|
||||||
|
workflow: str = "check.yml"
|
||||||
|
poll_interval: float = Field(default=10.0, gt=0)
|
||||||
|
max_poll_interval: float = Field(default=60.0, gt=0) # backoff while the API is failing
|
||||||
|
run_timeout_minutes: float = Field(default=30.0, gt=0)
|
||||||
|
timeout: float = Field(default=30.0, gt=0)
|
||||||
|
max_attempts: int = Field(default=5, ge=1)
|
||||||
|
|
||||||
|
|
||||||
class GitSettings(BaseSettings):
|
class GitSettings(BaseSettings):
|
||||||
model_config = SettingsConfigDict(**_ENV, env_prefix="GIT_")
|
model_config = SettingsConfigDict(**_ENV, env_prefix="GIT_")
|
||||||
|
|
||||||
@@ -98,6 +115,18 @@ class DockerSettings(BaseSettings):
|
|||||||
max_log_lines: int = Field(default=50_000, gt=0)
|
max_log_lines: int = Field(default=50_000, gt=0)
|
||||||
|
|
||||||
|
|
||||||
|
class RunnerSettings(BaseSettings):
|
||||||
|
model_config = SettingsConfigDict(**_ENV, env_prefix="RUNNER_")
|
||||||
|
|
||||||
|
# SSH to tgit-runner on the host (ADR-006); empty host — services are not deployed from the bot
|
||||||
|
host: str = ""
|
||||||
|
port: int = 22
|
||||||
|
user: str = "tgit-deploy"
|
||||||
|
# The bot's runner key, base64; authorized_keys pins it to `command="tgit-runner"` (infra/runner/)
|
||||||
|
ssh_key_b64: SecretStr | None = None
|
||||||
|
timeout: float = Field(default=300.0, gt=0)
|
||||||
|
|
||||||
|
|
||||||
class RuntimeSettings(BaseSettings):
|
class RuntimeSettings(BaseSettings):
|
||||||
model_config = _ENV
|
model_config = _ENV
|
||||||
|
|
||||||
@@ -119,5 +148,8 @@ class Settings(BaseSettings):
|
|||||||
llm: LLMSettings = Field(default_factory=LLMSettings)
|
llm: LLMSettings = Field(default_factory=LLMSettings)
|
||||||
sourcecraft: SourceCraftSettings = Field(default_factory=SourceCraftSettings)
|
sourcecraft: SourceCraftSettings = Field(default_factory=SourceCraftSettings)
|
||||||
git: GitSettings = Field(default_factory=GitSettings)
|
git: GitSettings = Field(default_factory=GitSettings)
|
||||||
|
# not `ci`: Settings has no env prefix, and CI runners export CI=true
|
||||||
|
uploads_ci: CISettings = Field(default_factory=CISettings)
|
||||||
docker: DockerSettings = Field(default_factory=DockerSettings)
|
docker: DockerSettings = Field(default_factory=DockerSettings)
|
||||||
|
runner: RunnerSettings = Field(default_factory=RunnerSettings)
|
||||||
runtime: RuntimeSettings = Field(default_factory=RuntimeSettings)
|
runtime: RuntimeSettings = Field(default_factory=RuntimeSettings)
|
||||||
|
|||||||
@@ -0,0 +1,25 @@
|
|||||||
|
"""pushes made before the PipelineWatcher (#11): nobody waits for their CI/CD — mark them skipped
|
||||||
|
|
||||||
|
Revision ID: 0002
|
||||||
|
Revises: 0001
|
||||||
|
Create Date: 2026-10-19 12:00:00
|
||||||
|
"""
|
||||||
|
|
||||||
|
from collections.abc import Sequence
|
||||||
|
|
||||||
|
from alembic import op
|
||||||
|
|
||||||
|
revision: str = "0002"
|
||||||
|
down_revision: str | None = "0001"
|
||||||
|
branch_labels: str | Sequence[str] | None = None
|
||||||
|
depends_on: str | Sequence[str] | None = None
|
||||||
|
|
||||||
|
|
||||||
|
def upgrade() -> None:
|
||||||
|
# Otherwise the first watcher tick would time them out and message their authors about long-gone pushes
|
||||||
|
op.execute("UPDATE pushes SET ci_status = 'skipped' WHERE ci_status IN ('pending', 'running')")
|
||||||
|
op.execute("UPDATE pushes SET cd_status = 'skipped' WHERE cd_status IN ('pending', 'running')")
|
||||||
|
|
||||||
|
|
||||||
|
def downgrade() -> None:
|
||||||
|
pass # data only: nothing to undo
|
||||||
+43
-4
@@ -3,6 +3,7 @@
|
|||||||
import asyncio
|
import asyncio
|
||||||
import contextlib
|
import contextlib
|
||||||
import logging
|
import logging
|
||||||
|
from datetime import timedelta
|
||||||
|
|
||||||
from aiogram import Bot, Dispatcher
|
from aiogram import Bot, Dispatcher
|
||||||
from aiogram.client.session.aiohttp import AiohttpSession
|
from aiogram.client.session.aiohttp import AiohttpSession
|
||||||
@@ -26,9 +27,13 @@ from app.health import heartbeat
|
|||||||
from app.llm.factory import make_provider
|
from app.llm.factory import make_provider
|
||||||
from app.logs.journal import LogJournal
|
from app.logs.journal import LogJournal
|
||||||
from app.logs_setup import setup_logging
|
from app.logs_setup import setup_logging
|
||||||
|
from app.pipeline.factory import make_ci
|
||||||
|
from app.pipeline.watcher import PipelineWatcher
|
||||||
from app.repo.drafts import DraftService
|
from app.repo.drafts import DraftService
|
||||||
from app.repo.git import UploadsRepo
|
from app.repo.git import UploadsRepo
|
||||||
from app.repo.publisher import Publisher
|
from app.repo.publisher import Publisher
|
||||||
|
from app.runner.client import RunnerClient
|
||||||
|
from app.runner.deployer import Deployer
|
||||||
from app.runtime.checks import Checker, load_checks
|
from app.runtime.checks import Checker, load_checks
|
||||||
from app.runtime.docker import DockerClient
|
from app.runtime.docker import DockerClient
|
||||||
from app.runtime.snapshots import SnapshotStore, snapshot_loop
|
from app.runtime.snapshots import SnapshotStore, snapshot_loop
|
||||||
@@ -47,6 +52,8 @@ def build_dispatcher(
|
|||||||
checker: Checker | None = None,
|
checker: Checker | None = None,
|
||||||
agent: Agent | None = None,
|
agent: Agent | None = None,
|
||||||
alerts: AlertService | None = None,
|
alerts: AlertService | None = None,
|
||||||
|
runner: RunnerClient | None = None,
|
||||||
|
pipeline: PipelineWatcher | None = None,
|
||||||
) -> Dispatcher:
|
) -> Dispatcher:
|
||||||
dp = Dispatcher(storage=make_storage(sessions))
|
dp = Dispatcher(storage=make_storage(sessions))
|
||||||
gate = AccessMiddleware(access, texts)
|
gate = AccessMiddleware(access, texts)
|
||||||
@@ -55,9 +62,11 @@ def build_dispatcher(
|
|||||||
dp.callback_query.outer_middleware(gate)
|
dp.callback_query.outer_middleware(gate)
|
||||||
dp.include_router(members.build_router(texts, access))
|
dp.include_router(members.build_router(texts, access))
|
||||||
dp.include_router(
|
dp.include_router(
|
||||||
commit.build_router(texts, repo, DraftService(sessions), Publisher(repo, sessions), commit_url_template)
|
commit.build_router(
|
||||||
|
texts, repo, DraftService(sessions), Publisher(repo, sessions), commit_url_template, pipeline
|
||||||
|
)
|
||||||
)
|
)
|
||||||
dp.include_router(status.build_router(texts, docker, checker))
|
dp.include_router(status.build_router(texts, docker, checker, runner))
|
||||||
dp.include_router(logs.build_router(texts, docker, LogJournal(sessions), agent, alerts))
|
dp.include_router(logs.build_router(texts, docker, LogJournal(sessions), agent, alerts))
|
||||||
dp.include_router(alerts_router.build_router(texts, alerts, docker))
|
dp.include_router(alerts_router.build_router(texts, alerts, docker))
|
||||||
dp.include_router(common.build_router(texts, access))
|
dp.include_router(common.build_router(texts, access))
|
||||||
@@ -89,34 +98,60 @@ async def run(settings: Settings) -> None:
|
|||||||
alerts_cfg,
|
alerts_cfg,
|
||||||
log_source=(lambda cid, n: docker.logs(cid, tail=n)) if docker else None,
|
log_source=(lambda cid, n: docker.logs(cid, tail=n)) if docker else None,
|
||||||
)
|
)
|
||||||
|
repo = UploadsRepo(settings.git)
|
||||||
|
runner = RunnerClient(settings.runner) if settings.runner.host else None
|
||||||
|
ci = make_ci(settings.uploads_ci)
|
||||||
|
deployer = Deployer(repo, runner, alerts) if runner else None
|
||||||
|
# Without CI and runner there is nothing to watch: the push message just says what was pushed
|
||||||
|
pipeline = (
|
||||||
|
PipelineWatcher(
|
||||||
|
sessions,
|
||||||
|
bot,
|
||||||
|
texts,
|
||||||
|
ci,
|
||||||
|
deployer,
|
||||||
|
commit_url_template=settings.git.commit_url_template,
|
||||||
|
poll_interval=settings.uploads_ci.poll_interval,
|
||||||
|
max_poll_interval=settings.uploads_ci.max_poll_interval,
|
||||||
|
run_timeout=timedelta(minutes=settings.uploads_ci.run_timeout_minutes),
|
||||||
|
)
|
||||||
|
if ci or deployer
|
||||||
|
else None
|
||||||
|
)
|
||||||
dp = build_dispatcher(
|
dp = build_dispatcher(
|
||||||
texts,
|
texts,
|
||||||
sessions,
|
sessions,
|
||||||
access,
|
access,
|
||||||
UploadsRepo(settings.git),
|
repo,
|
||||||
settings.git.commit_url_template,
|
settings.git.commit_url_template,
|
||||||
docker,
|
docker,
|
||||||
checker,
|
checker,
|
||||||
agent,
|
agent,
|
||||||
alerts,
|
alerts,
|
||||||
|
runner,
|
||||||
|
pipeline,
|
||||||
)
|
)
|
||||||
rt = settings.runtime
|
rt = settings.runtime
|
||||||
background = [
|
background = [
|
||||||
asyncio.create_task(heartbeat(rt.heartbeat_file, rt.heartbeat_interval)),
|
asyncio.create_task(heartbeat(rt.heartbeat_file, rt.heartbeat_interval)),
|
||||||
asyncio.create_task(cleanup_loop(sessions)),
|
asyncio.create_task(cleanup_loop(sessions)),
|
||||||
]
|
]
|
||||||
|
if pipeline:
|
||||||
|
background.append(asyncio.create_task(pipeline.run()))
|
||||||
if docker:
|
if docker:
|
||||||
monitor = AlertMonitor(alerts, store, alerts_cfg)
|
monitor = AlertMonitor(alerts, store, alerts_cfg)
|
||||||
background.append(
|
background.append(
|
||||||
asyncio.create_task(snapshot_loop(docker, store, settings.docker.snapshot_interval, monitor.on_round))
|
asyncio.create_task(snapshot_loop(docker, store, settings.docker.snapshot_interval, monitor.on_round))
|
||||||
)
|
)
|
||||||
log.info(
|
log.info(
|
||||||
"tgit %s starting, %d bootstrap admin(s), telegram %s, docker %s, agent %s",
|
"tgit %s starting, %d bootstrap admin(s), telegram %s, docker %s, agent %s, ci %s, runner %s",
|
||||||
__version__,
|
__version__,
|
||||||
len(settings.access.admin_ids),
|
len(settings.access.admin_ids),
|
||||||
"via proxy" if proxy else "direct",
|
"via proxy" if proxy else "direct",
|
||||||
"on" if docker else "off",
|
"on" if docker else "off",
|
||||||
settings.llm.provider if agent else "off",
|
settings.llm.provider if agent else "off",
|
||||||
|
settings.uploads_ci.provider if ci else "off",
|
||||||
|
settings.runner.host or "off",
|
||||||
)
|
)
|
||||||
try:
|
try:
|
||||||
await dp.start_polling(bot, handle_signals=True)
|
await dp.start_polling(bot, handle_signals=True)
|
||||||
@@ -126,6 +161,10 @@ async def run(settings: Settings) -> None:
|
|||||||
for task in background:
|
for task in background:
|
||||||
with contextlib.suppress(asyncio.CancelledError):
|
with contextlib.suppress(asyncio.CancelledError):
|
||||||
await task
|
await task
|
||||||
|
if pipeline:
|
||||||
|
await pipeline.aclose()
|
||||||
|
if ci:
|
||||||
|
await ci.aclose()
|
||||||
await bot.session.close()
|
await bot.session.close()
|
||||||
if docker:
|
if docker:
|
||||||
await docker.aclose()
|
await docker.aclose()
|
||||||
|
|||||||
@@ -0,0 +1 @@
|
|||||||
|
"""CI/CD of the uploads repo after a push: wait for the CI run, then deploy, telling the author (#11)."""
|
||||||
@@ -0,0 +1,10 @@
|
|||||||
|
from app.config import CISettings
|
||||||
|
from app.pipeline.gitea import GiteaCI
|
||||||
|
from app.pipeline.provider import CIProvider
|
||||||
|
|
||||||
|
|
||||||
|
def make_ci(settings: CISettings) -> CIProvider | None:
|
||||||
|
"""None — no CI to wait for: services deploy right after push."""
|
||||||
|
if settings.provider == "gitea":
|
||||||
|
return GiteaCI(settings)
|
||||||
|
return None
|
||||||
@@ -0,0 +1,118 @@
|
|||||||
|
"""Gitea Actions as the uploads CI (ADR-016). Unlike SourceCraft, a Gitea run carries `head_sha` — no meta cube."""
|
||||||
|
|
||||||
|
import asyncio
|
||||||
|
import logging
|
||||||
|
import re
|
||||||
|
from collections.abc import Awaitable, Callable
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
import httpx
|
||||||
|
|
||||||
|
from app.config import CISettings
|
||||||
|
from app.db.models import PipelineStatus
|
||||||
|
from app.pipeline.provider import CIRun
|
||||||
|
|
||||||
|
log = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
_TIMESTAMP = re.compile(r"^\d{4}-\d\d-\d\dT[\d:.]+Z ?", re.MULTILINE)
|
||||||
|
_FAILED = {"failure", "cancelled"}
|
||||||
|
|
||||||
|
Sleep = Callable[[float], Awaitable[None]]
|
||||||
|
|
||||||
|
|
||||||
|
class GiteaError(Exception):
|
||||||
|
pass
|
||||||
|
|
||||||
|
|
||||||
|
def _status(status: str, conclusion: str | None) -> PipelineStatus:
|
||||||
|
if status != "completed":
|
||||||
|
return PipelineStatus.RUNNING if status == "in_progress" else PipelineStatus.PENDING # queued · waiting
|
||||||
|
if conclusion == "success":
|
||||||
|
return PipelineStatus.SUCCESS
|
||||||
|
if conclusion == "skipped":
|
||||||
|
return PipelineStatus.SKIPPED
|
||||||
|
return PipelineStatus.FAILED # failure · cancelled
|
||||||
|
|
||||||
|
|
||||||
|
class GiteaCI:
|
||||||
|
def __init__(
|
||||||
|
self,
|
||||||
|
settings: CISettings,
|
||||||
|
transport: httpx.AsyncBaseTransport | None = None,
|
||||||
|
sleep: Sleep = asyncio.sleep,
|
||||||
|
) -> None:
|
||||||
|
self._s = settings
|
||||||
|
self._sleep = sleep
|
||||||
|
self._base = f"/api/v1/repos/{settings.repo}/actions"
|
||||||
|
self._http = httpx.AsyncClient(
|
||||||
|
base_url=settings.api_url,
|
||||||
|
headers={"Authorization": f"token {settings.token.get_secret_value()}"},
|
||||||
|
timeout=settings.timeout,
|
||||||
|
transport=transport,
|
||||||
|
)
|
||||||
|
|
||||||
|
async def aclose(self) -> None:
|
||||||
|
await self._http.aclose()
|
||||||
|
|
||||||
|
async def find(self, sha: str) -> CIRun | None:
|
||||||
|
data = await self._get(f"{self._base}/runs", params={"head_sha": sha, "limit": 20})
|
||||||
|
# path is "<file>@<ref>"; other workflows of the repo (if any) don't gate the deploy
|
||||||
|
runs = [r for r in data.get("workflow_runs") or [] if str(r.get("path", "")).split("@")[0] == self._s.workflow]
|
||||||
|
if not runs:
|
||||||
|
return None
|
||||||
|
run = max(runs, key=lambda r: int(r["id"])) # a re-run is a new id
|
||||||
|
status = _status(str(run.get("status")), run.get("conclusion"))
|
||||||
|
failed_step = await self._failed_step(run["id"]) if status is PipelineStatus.FAILED else None
|
||||||
|
return CIRun(str(run["id"]), status, str(run.get("html_url", "")), failed_step)
|
||||||
|
|
||||||
|
async def failure_log(self, run: CIRun) -> str:
|
||||||
|
job = await self._failed_job(run.id)
|
||||||
|
if job is None:
|
||||||
|
return ""
|
||||||
|
text = await self._get_text(f"{self._base}/jobs/{job['id']}/logs")
|
||||||
|
return _TIMESTAMP.sub("", text)
|
||||||
|
|
||||||
|
async def _failed_job(self, run_id: object) -> dict[str, Any] | None:
|
||||||
|
data = await self._get(f"{self._base}/runs/{run_id}/jobs")
|
||||||
|
jobs = data.get("jobs") or []
|
||||||
|
return next((j for j in jobs if j.get("conclusion") in _FAILED), None)
|
||||||
|
|
||||||
|
async def _failed_step(self, run_id: object) -> str | None:
|
||||||
|
job = await self._failed_job(run_id)
|
||||||
|
if job is None:
|
||||||
|
return None
|
||||||
|
step = next((s for s in job.get("steps") or [] if s.get("conclusion") in _FAILED), None)
|
||||||
|
if job.get("conclusion") == "cancelled" and step is None:
|
||||||
|
return f"{job.get('name')} / cancelled"
|
||||||
|
return f"{job.get('name')} / {step['name']}" if step else str(job.get("name"))
|
||||||
|
|
||||||
|
async def _request(self, path: str, params: dict[str, Any] | None = None) -> httpx.Response:
|
||||||
|
attempts = self._s.max_attempts
|
||||||
|
for attempt in range(1, attempts + 1):
|
||||||
|
try:
|
||||||
|
response = await self._http.get(path, params=params)
|
||||||
|
except httpx.TransportError as e:
|
||||||
|
if attempt == attempts:
|
||||||
|
raise GiteaError(f"{path}: {type(e).__name__}") from e
|
||||||
|
log.warning("gitea %s: %s, retry %d/%d", path, type(e).__name__, attempt, attempts - 1)
|
||||||
|
else:
|
||||||
|
if response.status_code < 500 or attempt == attempts:
|
||||||
|
if response.status_code >= 400:
|
||||||
|
raise GiteaError(f"{path}: HTTP {response.status_code}")
|
||||||
|
return response
|
||||||
|
log.warning("gitea %s: HTTP %d, retry %d/%d", path, response.status_code, attempt, attempts - 1)
|
||||||
|
await self._sleep(min(0.5 * 2 ** (attempt - 1), 8.0))
|
||||||
|
raise AssertionError("unreachable")
|
||||||
|
|
||||||
|
async def _get(self, path: str, params: dict[str, Any] | None = None) -> dict[str, Any]:
|
||||||
|
response = await self._request(path, params)
|
||||||
|
try:
|
||||||
|
body = response.json()
|
||||||
|
except ValueError as e:
|
||||||
|
raise GiteaError(f"{path}: not JSON") from e
|
||||||
|
if not isinstance(body, dict):
|
||||||
|
raise GiteaError(f"{path}: expected a JSON object")
|
||||||
|
return body
|
||||||
|
|
||||||
|
async def _get_text(self, path: str) -> str:
|
||||||
|
return (await self._request(path)).text
|
||||||
@@ -0,0 +1,26 @@
|
|||||||
|
"""What the watcher needs from a CI: the run of a commit and the log of what failed. Vendors live next to it."""
|
||||||
|
|
||||||
|
from dataclasses import dataclass
|
||||||
|
from typing import Protocol
|
||||||
|
|
||||||
|
from app.db.models import PipelineStatus
|
||||||
|
|
||||||
|
|
||||||
|
@dataclass(frozen=True)
|
||||||
|
class CIRun:
|
||||||
|
id: str
|
||||||
|
status: PipelineStatus # pending · running · success · failed · skipped
|
||||||
|
url: str
|
||||||
|
failed_step: str | None = None # "job / step" of the first failure
|
||||||
|
|
||||||
|
|
||||||
|
class CIProvider(Protocol):
|
||||||
|
async def find(self, sha: str) -> CIRun | None:
|
||||||
|
"""The newest run of the uploads workflow for this commit; None — not started (yet)."""
|
||||||
|
...
|
||||||
|
|
||||||
|
async def failure_log(self, run: CIRun) -> str:
|
||||||
|
"""Log of the failed job, plain text without runner timestamps."""
|
||||||
|
...
|
||||||
|
|
||||||
|
async def aclose(self) -> None: ...
|
||||||
@@ -0,0 +1,361 @@
|
|||||||
|
"""PipelineWatcher: after a push — wait for the uploads CI, then deploy through tgit-runner (SPEC C-24, C-30…C-34).
|
||||||
|
|
||||||
|
The truth is in the DB (`pushes.ci_status/cd_status`, `pipeline_runs`), so a restarted bot picks up where it stopped.
|
||||||
|
The CI is polled only while some push waits for it (ADR-003); with nothing to wait for the loop sleeps until `wake()`.
|
||||||
|
|
||||||
|
The push message is edited in place (CI ⏳ → ✅/❌ → CD ⏳ → 🚀/⛔); the author also gets separate messages for the
|
||||||
|
outcomes — an edit doesn't notify. A failed deploy is an alert ⛔ for the whole team (raised by the Deployer).
|
||||||
|
"""
|
||||||
|
|
||||||
|
import asyncio
|
||||||
|
import contextlib
|
||||||
|
import html
|
||||||
|
import logging
|
||||||
|
from collections.abc import Awaitable, Callable
|
||||||
|
from datetime import datetime, timedelta
|
||||||
|
|
||||||
|
from aiogram import Bot
|
||||||
|
from aiogram.enums import ParseMode
|
||||||
|
from aiogram.exceptions import TelegramAPIError, TelegramBadRequest
|
||||||
|
from aiogram.types import BufferedInputFile, ReplyParameters
|
||||||
|
from sqlalchemy import or_, select, update
|
||||||
|
|
||||||
|
from app.db.models import PipelineRun, PipelineStatus, Push, Stage, as_utc, utcnow
|
||||||
|
from app.db.session import SessionMaker
|
||||||
|
from app.pipeline.provider import CIProvider, CIRun
|
||||||
|
from app.runner.deployer import Deployer, Outcome, changed_services
|
||||||
|
from app.texts import Texts
|
||||||
|
|
||||||
|
log = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
WAITING = (PipelineStatus.PENDING, PipelineStatus.RUNNING)
|
||||||
|
PASSED = (PipelineStatus.SUCCESS, PipelineStatus.SKIPPED)
|
||||||
|
RUNNER = "tgit-runner" # run_slug of the CD stage: the deploy is the bot's own, not a CI run
|
||||||
|
LOG_CHARS = 3000 # inline tail; the whole log goes as a file when it is longer
|
||||||
|
MAX_LOG_BYTES = 5 * 1024 * 1024 # SPEC L-5
|
||||||
|
|
||||||
|
|
||||||
|
def initial_status(ci: bool, deploys: bool, paths: list[str]) -> tuple[PipelineStatus, PipelineStatus]:
|
||||||
|
"""(ci, cd) of a fresh push: SKIPPED where there is nothing to wait for."""
|
||||||
|
cd = PipelineStatus.PENDING if deploys and changed_services(paths) else PipelineStatus.SKIPPED
|
||||||
|
return (PipelineStatus.PENDING if ci else PipelineStatus.SKIPPED), cd
|
||||||
|
|
||||||
|
|
||||||
|
def render(
|
||||||
|
texts: Texts,
|
||||||
|
push: Push,
|
||||||
|
ci_run: PipelineRun | None = None,
|
||||||
|
cd_run: PipelineRun | None = None,
|
||||||
|
commit_url_template: str = "",
|
||||||
|
timeout_minutes: float = 30,
|
||||||
|
) -> str:
|
||||||
|
"""The push message: what was pushed, then a line per stage that applies to it."""
|
||||||
|
t = texts.pipeline
|
||||||
|
first_line = push.message.splitlines()[0] if push.message else ""
|
||||||
|
lines = [texts.commit.pushed.format(sha=push.sha[:7], message=first_line)]
|
||||||
|
if commit_url_template:
|
||||||
|
lines.append(commit_url_template.format(sha=push.sha))
|
||||||
|
match push.ci_status:
|
||||||
|
case PipelineStatus.PENDING:
|
||||||
|
lines.append(t.ci_pending)
|
||||||
|
case PipelineStatus.RUNNING:
|
||||||
|
lines.append(t.ci_running)
|
||||||
|
case PipelineStatus.SUCCESS:
|
||||||
|
lines.append(t.ci_success)
|
||||||
|
case PipelineStatus.FAILED:
|
||||||
|
lines.append(t.ci_failed.format(step=(ci_run and ci_run.failed_cube) or "?"))
|
||||||
|
case PipelineStatus.TIMEOUT:
|
||||||
|
lines.append(t.ci_timeout.format(minutes=round(timeout_minutes)))
|
||||||
|
if ci_run is not None and ci_run.url:
|
||||||
|
lines.append(ci_run.url)
|
||||||
|
|
||||||
|
services = ", ".join(changed_services(push.paths))
|
||||||
|
failed = (cd_run.failed_cube or "") if cd_run else ""
|
||||||
|
match push.cd_status:
|
||||||
|
case PipelineStatus.PENDING if push.ci_status in WAITING:
|
||||||
|
lines.append(t.cd_waiting.format(services=services))
|
||||||
|
case PipelineStatus.PENDING | PipelineStatus.RUNNING:
|
||||||
|
lines.append(t.cd_running.format(services=services))
|
||||||
|
case PipelineStatus.SUCCESS:
|
||||||
|
lines.append(t.cd_success.format(services=services))
|
||||||
|
case PipelineStatus.FAILED:
|
||||||
|
lines.append(t.cd_failed.format(services=failed.replace(",", ", ")))
|
||||||
|
case PipelineStatus.SKIPPED if services and push.ci_status not in PASSED:
|
||||||
|
lines.append(t.cd_not_run)
|
||||||
|
case PipelineStatus.SKIPPED if cd_run is not None:
|
||||||
|
lines.append(t.cd_superseded)
|
||||||
|
return "\n".join(lines)
|
||||||
|
|
||||||
|
|
||||||
|
class PipelineWatcher:
|
||||||
|
def __init__(
|
||||||
|
self,
|
||||||
|
sessions: SessionMaker,
|
||||||
|
bot: Bot,
|
||||||
|
texts: Texts,
|
||||||
|
ci: CIProvider | None,
|
||||||
|
deployer: Deployer | None,
|
||||||
|
*,
|
||||||
|
commit_url_template: str = "",
|
||||||
|
poll_interval: float = 10.0,
|
||||||
|
max_poll_interval: float = 60.0,
|
||||||
|
run_timeout: timedelta = timedelta(minutes=30),
|
||||||
|
clock: Callable[[], datetime] = utcnow,
|
||||||
|
) -> None:
|
||||||
|
self._sessions = sessions
|
||||||
|
self._bot = bot
|
||||||
|
self._texts = texts
|
||||||
|
self._ci = ci
|
||||||
|
self._deployer = deployer
|
||||||
|
self._url = commit_url_template
|
||||||
|
self._interval = poll_interval
|
||||||
|
self._max_interval = max_poll_interval
|
||||||
|
self._timeout = run_timeout
|
||||||
|
self._clock = clock
|
||||||
|
self._wake = asyncio.Event()
|
||||||
|
self._deploying: dict[int, asyncio.Task[None]] = {}
|
||||||
|
|
||||||
|
def initial_status(self, paths: list[str]) -> tuple[PipelineStatus, PipelineStatus]:
|
||||||
|
return initial_status(self._ci is not None, self._deployer is not None, paths)
|
||||||
|
|
||||||
|
def wake(self) -> None:
|
||||||
|
"""A new push: look at it now rather than after the current pause."""
|
||||||
|
self._wake.set()
|
||||||
|
|
||||||
|
async def run(self) -> None:
|
||||||
|
delay = self._interval
|
||||||
|
while True:
|
||||||
|
self._wake.clear() # before reading the DB: a push recorded during the tick wakes the next wait
|
||||||
|
try:
|
||||||
|
polling = await self.tick()
|
||||||
|
delay = self._interval
|
||||||
|
except Exception:
|
||||||
|
log.exception("pipeline watcher tick")
|
||||||
|
polling = True
|
||||||
|
delay = min(delay * 2, self._max_interval)
|
||||||
|
if polling:
|
||||||
|
with contextlib.suppress(TimeoutError):
|
||||||
|
await asyncio.wait_for(self._wake.wait(), timeout=delay)
|
||||||
|
else:
|
||||||
|
await self._wake.wait()
|
||||||
|
|
||||||
|
async def tick(self) -> bool:
|
||||||
|
"""One pass over unfinished pushes. True — some push still waits for CI, poll again soon."""
|
||||||
|
async with self._sessions() as s:
|
||||||
|
pushes = list(
|
||||||
|
await s.scalars(
|
||||||
|
select(Push)
|
||||||
|
.where(
|
||||||
|
or_(
|
||||||
|
Push.ci_status.in_(WAITING),
|
||||||
|
Push.ci_status.in_(PASSED) & Push.cd_status.in_(WAITING),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
.order_by(Push.id)
|
||||||
|
)
|
||||||
|
)
|
||||||
|
polling = False
|
||||||
|
for push in pushes:
|
||||||
|
if push.ci_status in WAITING:
|
||||||
|
await self._poll_ci(push)
|
||||||
|
polling |= push.ci_status in WAITING
|
||||||
|
elif push.id not in self._deploying:
|
||||||
|
await self._start_cd(push)
|
||||||
|
return polling
|
||||||
|
|
||||||
|
async def drain(self) -> None:
|
||||||
|
"""Wait for the deploys in flight."""
|
||||||
|
while self._deploying:
|
||||||
|
await asyncio.gather(*self._deploying.values(), return_exceptions=True)
|
||||||
|
|
||||||
|
async def aclose(self) -> None:
|
||||||
|
"""Stop deploys in flight: their push stays `cd=running` in the DB, and the next start deploys it again."""
|
||||||
|
tasks = list(self._deploying.values())
|
||||||
|
for task in tasks:
|
||||||
|
task.cancel()
|
||||||
|
await asyncio.gather(*tasks, return_exceptions=True)
|
||||||
|
|
||||||
|
# ---- CI ------------------------------------------------------------------------------------------------------
|
||||||
|
|
||||||
|
async def _poll_ci(self, push: Push) -> None:
|
||||||
|
if self._ci is None: # CI switched off since the push: don't keep it waiting forever
|
||||||
|
await self._set(push, ci=PipelineStatus.SKIPPED)
|
||||||
|
await self._refresh(push)
|
||||||
|
return
|
||||||
|
run = await self._ci.find(push.sha)
|
||||||
|
if run is None or run.status in WAITING:
|
||||||
|
if self._clock() - as_utc(push.created_at) > self._timeout:
|
||||||
|
await self._finish_ci(push, PipelineStatus.TIMEOUT, run)
|
||||||
|
elif run is not None and run.status is not push.ci_status:
|
||||||
|
await self._save_run(push, Stage.CI, run.id, run.status, None, run.url)
|
||||||
|
await self._set(push, ci=run.status)
|
||||||
|
await self._refresh(push)
|
||||||
|
return
|
||||||
|
await self._finish_ci(push, run.status, run)
|
||||||
|
|
||||||
|
async def _finish_ci(self, push: Push, status: PipelineStatus, run: CIRun | None) -> None:
|
||||||
|
if run is not None:
|
||||||
|
await self._save_run(push, Stage.CI, run.id, status, run.failed_step, run.url)
|
||||||
|
passed = status in PASSED
|
||||||
|
cd = push.cd_status if passed or push.cd_status is not PipelineStatus.PENDING else PipelineStatus.SKIPPED
|
||||||
|
await self._set(push, ci=status, cd=cd)
|
||||||
|
await self._refresh(push)
|
||||||
|
t = self._texts.pipeline
|
||||||
|
sha = push.sha[:7]
|
||||||
|
url = html.escape(run.url) if run else ""
|
||||||
|
if passed:
|
||||||
|
await self._notify(push, t.notify_ci_ok.format(sha=sha, url=url))
|
||||||
|
if push.cd_status is PipelineStatus.PENDING:
|
||||||
|
await self._start_cd(push)
|
||||||
|
elif status is PipelineStatus.TIMEOUT:
|
||||||
|
minutes = round(self._timeout.total_seconds() / 60)
|
||||||
|
await self._notify(push, t.notify_ci_timeout.format(sha=sha, minutes=minutes))
|
||||||
|
else:
|
||||||
|
assert run is not None
|
||||||
|
await self._notify_failure(push, run)
|
||||||
|
|
||||||
|
async def _notify_failure(self, push: Push, run: CIRun) -> None:
|
||||||
|
t = self._texts.pipeline
|
||||||
|
step = html.escape(run.failed_step or "?")
|
||||||
|
text = t.notify_ci_failed.format(step=step, sha=push.sha[:7], url=html.escape(run.url))
|
||||||
|
assert self._ci is not None
|
||||||
|
try:
|
||||||
|
full = await self._ci.failure_log(run)
|
||||||
|
except Exception as e: # the verdict matters more than its log
|
||||||
|
log.warning("ci log for %s: %s", push.sha[:7], e)
|
||||||
|
full = ""
|
||||||
|
tail = full.strip()[-LOG_CHARS:]
|
||||||
|
if tail:
|
||||||
|
text += f"\n<pre>{html.escape(tail)}</pre>"
|
||||||
|
await self._notify(push, text)
|
||||||
|
if len(full.strip()) > LOG_CHARS:
|
||||||
|
data = full.encode()[-MAX_LOG_BYTES:]
|
||||||
|
await self._send(
|
||||||
|
lambda: self._bot.send_document(
|
||||||
|
push.chat_id,
|
||||||
|
BufferedInputFile(data, f"ci-{push.sha[:7]}.log"),
|
||||||
|
caption=t.log_file,
|
||||||
|
reply_parameters=self._reply(push),
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
# ---- CD ------------------------------------------------------------------------------------------------------
|
||||||
|
|
||||||
|
async def _start_cd(self, push: Push) -> None:
|
||||||
|
if self._deployer is None: # the runner was switched off since the push
|
||||||
|
await self._set(push, cd=PipelineStatus.SKIPPED)
|
||||||
|
await self._refresh(push)
|
||||||
|
return
|
||||||
|
services = await self._not_superseded(push)
|
||||||
|
if not services:
|
||||||
|
await self._save_run(push, Stage.CD, RUNNER, PipelineStatus.SKIPPED, None, None)
|
||||||
|
await self._set(push, cd=PipelineStatus.SKIPPED)
|
||||||
|
await self._refresh(push)
|
||||||
|
return
|
||||||
|
await self._set(push, cd=PipelineStatus.RUNNING)
|
||||||
|
await self._refresh(push)
|
||||||
|
task = asyncio.create_task(self._deploy(push, services))
|
||||||
|
self._deploying[push.id] = task
|
||||||
|
task.add_done_callback(lambda _: self._deploying.pop(push.id, None))
|
||||||
|
|
||||||
|
async def _not_superseded(self, push: Push) -> list[str]:
|
||||||
|
"""Services of this push that no newer passed push will deploy anyway (CI runs may finish out of order)."""
|
||||||
|
services = changed_services(push.paths)
|
||||||
|
async with self._sessions() as s:
|
||||||
|
newer = await s.scalars(select(Push.paths).where(Push.id > push.id, Push.ci_status.in_(PASSED)))
|
||||||
|
taken = {svc for paths in newer for svc in changed_services(paths)}
|
||||||
|
return [svc for svc in services if svc not in taken]
|
||||||
|
|
||||||
|
async def _deploy(self, push: Push, services: list[str]) -> None:
|
||||||
|
assert self._deployer is not None
|
||||||
|
outcomes: list[Outcome] = []
|
||||||
|
|
||||||
|
async def report(o: Outcome) -> None:
|
||||||
|
outcomes.append(o)
|
||||||
|
t = self._texts.pipeline
|
||||||
|
name = html.escape(o.service)
|
||||||
|
if o.result.ok and o.removed:
|
||||||
|
await self._notify(push, t.removed.format(service=name))
|
||||||
|
elif not o.result.ok:
|
||||||
|
text = t.deploy_failed.format(service=name, reason=html.escape(o.result.summary))
|
||||||
|
if o.result.tail:
|
||||||
|
text += f"\n<pre>{html.escape(o.result.tail[-LOG_CHARS:])}</pre>"
|
||||||
|
await self._notify(push, text)
|
||||||
|
|
||||||
|
try:
|
||||||
|
await self._deployer.deploy(push.sha, services, report)
|
||||||
|
except Exception:
|
||||||
|
log.exception("deploy of push %s", push.sha[:7])
|
||||||
|
failed = [o.service for o in outcomes if not o.result.ok] + services[len(outcomes) :]
|
||||||
|
deployed = [o.service for o in outcomes if o.result.ok and not o.removed]
|
||||||
|
status = PipelineStatus.FAILED if failed else PipelineStatus.SUCCESS
|
||||||
|
await self._save_run(push, Stage.CD, RUNNER, status, ",".join(failed)[:100] or None, None)
|
||||||
|
await self._set(push, cd=status)
|
||||||
|
await self._refresh(push)
|
||||||
|
if deployed:
|
||||||
|
names = ", ".join(deployed)
|
||||||
|
await self._notify(push, self._texts.pipeline.notify_cd_ok.format(services=names, sha=push.sha[:7]))
|
||||||
|
|
||||||
|
# ---- state and messages --------------------------------------------------------------------------------------
|
||||||
|
|
||||||
|
async def _set(self, push: Push, ci: PipelineStatus | None = None, cd: PipelineStatus | None = None) -> None:
|
||||||
|
values = {}
|
||||||
|
if ci is not None:
|
||||||
|
values["ci_status"] = push.ci_status = ci
|
||||||
|
if cd is not None:
|
||||||
|
values["cd_status"] = push.cd_status = cd
|
||||||
|
async with self._sessions.begin() as s:
|
||||||
|
await s.execute(update(Push).where(Push.id == push.id).values(**values))
|
||||||
|
|
||||||
|
async def _save_run(
|
||||||
|
self, push: Push, stage: Stage, slug: str, status: PipelineStatus, failed: str | None, url: str | None
|
||||||
|
) -> None:
|
||||||
|
async with self._sessions.begin() as s:
|
||||||
|
row = await s.scalar(select(PipelineRun).where(PipelineRun.push_id == push.id, PipelineRun.stage == stage))
|
||||||
|
if row is None:
|
||||||
|
row = PipelineRun(push_id=push.id, stage=stage)
|
||||||
|
s.add(row)
|
||||||
|
row.run_slug, row.status, row.failed_cube, row.url = slug, status, failed, url
|
||||||
|
|
||||||
|
async def _runs(self, push: Push) -> dict[Stage, PipelineRun]:
|
||||||
|
async with self._sessions() as s:
|
||||||
|
rows = await s.scalars(select(PipelineRun).where(PipelineRun.push_id == push.id))
|
||||||
|
return {r.stage: r for r in rows}
|
||||||
|
|
||||||
|
async def _refresh(self, push: Push) -> None:
|
||||||
|
"""Re-render the push message from the DB."""
|
||||||
|
if push.message_id is None:
|
||||||
|
return
|
||||||
|
runs = await self._runs(push)
|
||||||
|
text = render(
|
||||||
|
self._texts,
|
||||||
|
push,
|
||||||
|
runs.get(Stage.CI),
|
||||||
|
runs.get(Stage.CD),
|
||||||
|
self._url,
|
||||||
|
self._timeout.total_seconds() / 60,
|
||||||
|
)
|
||||||
|
message_id = push.message_id
|
||||||
|
await self._send(lambda: self._bot.edit_message_text(text, chat_id=push.chat_id, message_id=message_id))
|
||||||
|
|
||||||
|
def _reply(self, push: Push) -> ReplyParameters | None:
|
||||||
|
if push.message_id is None:
|
||||||
|
return None
|
||||||
|
return ReplyParameters(message_id=push.message_id, allow_sending_without_reply=True)
|
||||||
|
|
||||||
|
async def _notify(self, push: Push, text: str) -> None:
|
||||||
|
await self._send(
|
||||||
|
lambda: self._bot.send_message(
|
||||||
|
push.chat_id, text, parse_mode=ParseMode.HTML, reply_parameters=self._reply(push)
|
||||||
|
)
|
||||||
|
)
|
||||||
|
|
||||||
|
@staticmethod
|
||||||
|
async def _send(call: Callable[[], Awaitable[object]]) -> None:
|
||||||
|
"""The status in the DB is what matters: a message that didn't get through is logged, not retried."""
|
||||||
|
try:
|
||||||
|
await call()
|
||||||
|
except TelegramBadRequest as e: # not modified · deleted by the user · too old to edit
|
||||||
|
log.info("pipeline message: %s", e)
|
||||||
|
except TelegramAPIError as e: # blocked by the user · flood control
|
||||||
|
log.warning("pipeline message: %s", e)
|
||||||
+24
-14
@@ -13,7 +13,7 @@ from pathlib import Path
|
|||||||
|
|
||||||
from sqlalchemy import func, select
|
from sqlalchemy import func, select
|
||||||
|
|
||||||
from app.db.models import ChangeOp, Push, User, utcnow
|
from app.db.models import ChangeOp, PipelineStatus, Push, User, utcnow
|
||||||
from app.db.session import SessionMaker
|
from app.db.session import SessionMaker
|
||||||
from app.repo.drafts import DraftView
|
from app.repo.drafts import DraftView
|
||||||
from app.repo.git import GitError, UploadsRepo
|
from app.repo.git import GitError, UploadsRepo
|
||||||
@@ -67,21 +67,31 @@ class Publisher:
|
|||||||
return PushResult(PushStatus.FAILED)
|
return PushResult(PushStatus.FAILED)
|
||||||
|
|
||||||
async def record(
|
async def record(
|
||||||
self, member: User, result: PushResult, message: str, draft: DraftView, chat_id: int, message_id: int
|
self,
|
||||||
) -> None:
|
member: User,
|
||||||
|
result: PushResult,
|
||||||
|
message: str,
|
||||||
|
draft: DraftView,
|
||||||
|
chat_id: int,
|
||||||
|
message_id: int,
|
||||||
|
status: tuple[PipelineStatus, PipelineStatus],
|
||||||
|
) -> Push:
|
||||||
|
"""status — (ci, cd) to start from; the PipelineWatcher takes it from there (#11)."""
|
||||||
assert result.sha is not None
|
assert result.sha is not None
|
||||||
|
push = Push(
|
||||||
|
user_id=member.tg_id,
|
||||||
|
sha=result.sha,
|
||||||
|
message=message,
|
||||||
|
paths=[c.path for c in draft.changes],
|
||||||
|
chat_id=chat_id,
|
||||||
|
message_id=message_id,
|
||||||
|
ci_status=status[0],
|
||||||
|
cd_status=status[1],
|
||||||
|
created_at=self._clock(),
|
||||||
|
)
|
||||||
async with self._sessions.begin() as s:
|
async with self._sessions.begin() as s:
|
||||||
s.add(
|
s.add(push)
|
||||||
Push(
|
return push
|
||||||
user_id=member.tg_id,
|
|
||||||
sha=result.sha,
|
|
||||||
message=message,
|
|
||||||
paths=[c.path for c in draft.changes],
|
|
||||||
chat_id=chat_id,
|
|
||||||
message_id=message_id,
|
|
||||||
created_at=self._clock(),
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
async def _push(self, member: User, draft: DraftView, message: str, overwrite: bool) -> PushResult:
|
async def _push(self, member: User, draft: DraftView, message: str, overwrite: bool) -> PushResult:
|
||||||
repo = self._repo
|
repo = self._repo
|
||||||
|
|||||||
@@ -8,7 +8,7 @@ from dataclasses import dataclass
|
|||||||
MAX_FILE_BYTES = 100 * 1024
|
MAX_FILE_BYTES = 100 * 1024
|
||||||
NAME = re.compile(r"^[A-Za-z0-9._-]{1,100}$")
|
NAME = re.compile(r"^[A-Za-z0-9._-]{1,100}$")
|
||||||
# Bot users can't touch the CI of the uploads repo or its helper scripts (SPEC C-16, ADR-014)
|
# Bot users can't touch the CI of the uploads repo or its helper scripts (SPEC C-16, ADR-014)
|
||||||
PROTECTED_PREFIXES = (".sourcecraft/", ".tgit/", ".git/")
|
PROTECTED_PREFIXES = (".sourcecraft/", ".gitea/", ".tgit/", ".git/")
|
||||||
PROTECTED_FILES = {"ruff.toml"}
|
PROTECTED_FILES = {"ruff.toml"}
|
||||||
_FENCE = re.compile(r"^\s*```[\w+-]*[ \t]*\n(.*?)\n?```\s*$", re.DOTALL)
|
_FENCE = re.compile(r"^\s*```[\w+-]*[ \t]*\n(.*?)\n?```\s*$", re.DOTALL)
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1 @@
|
|||||||
|
"""Deploying services of the uploads repo: the bot ships `git archive` to `tgit-runner` on the host (#13)."""
|
||||||
@@ -0,0 +1,85 @@
|
|||||||
|
"""SSH to `tgit-runner`: the key can run nothing else (forced command), so the «command» is just its arguments."""
|
||||||
|
|
||||||
|
import asyncio
|
||||||
|
import base64
|
||||||
|
import logging
|
||||||
|
import os
|
||||||
|
import re
|
||||||
|
import tempfile
|
||||||
|
from dataclasses import dataclass
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
from app.config import RunnerSettings
|
||||||
|
|
||||||
|
log = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
SERVICE = re.compile(r"^[a-z][a-z0-9-]{1,30}$") # the runner checks the same — refuse early, with a clear reason
|
||||||
|
SHA = re.compile(r"^[0-9a-f]{7,40}$")
|
||||||
|
|
||||||
|
|
||||||
|
@dataclass(frozen=True)
|
||||||
|
class RunnerResult:
|
||||||
|
ok: bool
|
||||||
|
code: int # 0 ok · 2 bad request · 3 not healthy · 4 limit · 5 build failed · 255 ssh · -1 timeout
|
||||||
|
summary: str # the runner's last line without OK/FAIL
|
||||||
|
output: str # everything, for logs and the user's log tail
|
||||||
|
|
||||||
|
@property
|
||||||
|
def tail(self) -> str:
|
||||||
|
lines = self.output.strip().splitlines()[:-1]
|
||||||
|
return "\n".join(lines[-20:])
|
||||||
|
|
||||||
|
|
||||||
|
class RunnerClient:
|
||||||
|
def __init__(self, settings: RunnerSettings) -> None:
|
||||||
|
self._s = settings
|
||||||
|
self._key = self._materialize_key()
|
||||||
|
|
||||||
|
def _materialize_key(self) -> Path | None:
|
||||||
|
if self._s.ssh_key_b64 is None:
|
||||||
|
return None
|
||||||
|
key = base64.b64decode(self._s.ssh_key_b64.get_secret_value())
|
||||||
|
fd, name = tempfile.mkstemp(prefix="tgit-runner-")
|
||||||
|
with os.fdopen(fd, "wb") as f:
|
||||||
|
f.write(key if key.endswith(b"\n") else key + b"\n")
|
||||||
|
return Path(name)
|
||||||
|
|
||||||
|
def _ssh(self) -> list[str]:
|
||||||
|
cmd = ["ssh", "-F", "/dev/null", "-p", str(self._s.port), "-o", "BatchMode=yes", "-o", "ConnectTimeout=20"]
|
||||||
|
cmd += ["-o", "StrictHostKeyChecking=accept-new", "-o", f"UserKnownHostsFile={tempfile.gettempdir()}/tgit-kh"]
|
||||||
|
if self._key is not None:
|
||||||
|
cmd += ["-i", str(self._key), "-o", "IdentitiesOnly=yes"]
|
||||||
|
return [*cmd, f"{self._s.user}@{self._s.host}"]
|
||||||
|
|
||||||
|
async def run(self, *args: str, stdin: bytes = b"") -> RunnerResult:
|
||||||
|
proc = await asyncio.create_subprocess_exec(
|
||||||
|
*self._ssh(),
|
||||||
|
*args,
|
||||||
|
stdin=asyncio.subprocess.PIPE,
|
||||||
|
stdout=asyncio.subprocess.PIPE,
|
||||||
|
stderr=asyncio.subprocess.STDOUT,
|
||||||
|
)
|
||||||
|
try:
|
||||||
|
out, _ = await asyncio.wait_for(proc.communicate(stdin), timeout=self._s.timeout)
|
||||||
|
except TimeoutError:
|
||||||
|
proc.kill()
|
||||||
|
await proc.wait()
|
||||||
|
return RunnerResult(False, -1, "timeout", "")
|
||||||
|
text = out.decode(errors="replace")
|
||||||
|
last = text.strip().splitlines()[-1] if text.strip() else ""
|
||||||
|
code = proc.returncode if proc.returncode is not None else -1
|
||||||
|
ok = code == 0 and last.startswith("OK")
|
||||||
|
summary = last.removeprefix("OK ").removeprefix("FAIL ") if last.startswith(("OK", "FAIL")) else last[:200]
|
||||||
|
if not ok:
|
||||||
|
log.warning("runner %s: exit %s: %s", args[0] if args else "", code, summary)
|
||||||
|
return RunnerResult(ok, code, summary, text)
|
||||||
|
|
||||||
|
async def deploy(self, service: str, sha: str, archive: bytes) -> RunnerResult:
|
||||||
|
if not SERVICE.match(service) or not SHA.match(sha):
|
||||||
|
return RunnerResult(False, 2, "bad service name or sha", "")
|
||||||
|
return await self.run("deploy", service, sha, stdin=archive)
|
||||||
|
|
||||||
|
async def restart(self, service: str) -> RunnerResult:
|
||||||
|
if not SERVICE.match(service):
|
||||||
|
return RunnerResult(False, 2, "bad service name", "")
|
||||||
|
return await self.run("restart", service)
|
||||||
@@ -0,0 +1,75 @@
|
|||||||
|
"""After a push passes CI: the services it touched are (re)deployed through tgit-runner (PipelineWatcher, #11).
|
||||||
|
|
||||||
|
A service = `services/<name>/`. Gone from the commit — removed on the host. A refused deploy keeps the old version
|
||||||
|
running (the runner switches only to a healthy one) and raises ⛔ for the whole team.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import asyncio
|
||||||
|
import logging
|
||||||
|
from collections.abc import Awaitable, Callable, Iterable
|
||||||
|
from dataclasses import dataclass
|
||||||
|
|
||||||
|
from app.alerts.detectors import Condition
|
||||||
|
from app.alerts.service import AlertService
|
||||||
|
from app.db.models import AlertKind
|
||||||
|
from app.repo.git import GitError, UploadsRepo
|
||||||
|
from app.runner.client import SERVICE, RunnerClient, RunnerResult
|
||||||
|
|
||||||
|
log = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
|
def changed_services(paths: Iterable[str]) -> list[str]:
|
||||||
|
found = set()
|
||||||
|
for path in paths:
|
||||||
|
parts = path.split("/")
|
||||||
|
if len(parts) >= 3 and parts[0] == "services" and SERVICE.match(parts[1]):
|
||||||
|
found.add(parts[1])
|
||||||
|
return sorted(found)
|
||||||
|
|
||||||
|
|
||||||
|
@dataclass(frozen=True)
|
||||||
|
class Outcome:
|
||||||
|
service: str
|
||||||
|
result: RunnerResult
|
||||||
|
removed: bool = False
|
||||||
|
|
||||||
|
|
||||||
|
Report = Callable[[Outcome], Awaitable[None]]
|
||||||
|
|
||||||
|
|
||||||
|
class Deployer:
|
||||||
|
def __init__(self, repo: UploadsRepo, runner: RunnerClient, alerts: AlertService | None = None) -> None:
|
||||||
|
self._repo = repo
|
||||||
|
self._runner = runner
|
||||||
|
self._alerts = alerts
|
||||||
|
self._lock = asyncio.Lock() # one deploy at a time: builds are heavy for the shared host
|
||||||
|
|
||||||
|
async def deploy(self, sha: str, services: list[str], report: Report) -> None:
|
||||||
|
async with self._lock:
|
||||||
|
for service in services:
|
||||||
|
try:
|
||||||
|
outcome = await self._one(sha, service)
|
||||||
|
except Exception: # one broken service must not stop the others
|
||||||
|
log.exception("deploy %s @ %s", service, sha[:7])
|
||||||
|
outcome = Outcome(service, RunnerResult(False, -1, "internal error", ""))
|
||||||
|
if not outcome.result.ok and self._alerts is not None:
|
||||||
|
await self._alerts.event(
|
||||||
|
Condition(
|
||||||
|
AlertKind.DEPLOY_REFUSED,
|
||||||
|
f"tgit-svc-{service}",
|
||||||
|
{"detail": outcome.result.summary, "sha": sha[:7], "log": outcome.result.tail},
|
||||||
|
)
|
||||||
|
)
|
||||||
|
try:
|
||||||
|
await report(outcome)
|
||||||
|
except Exception:
|
||||||
|
log.exception("deploy report for %s", service)
|
||||||
|
|
||||||
|
async def _one(self, sha: str, service: str) -> Outcome:
|
||||||
|
tree = f"{sha}:services/{service}"
|
||||||
|
try:
|
||||||
|
await self._repo.git("cat-file", "-e", tree)
|
||||||
|
except GitError:
|
||||||
|
return Outcome(service, await self._runner.run("remove", service), removed=True)
|
||||||
|
archive = await self._repo.git("archive", "--format=tar", tree)
|
||||||
|
return Outcome(service, await self._runner.deploy(service, sha, archive))
|
||||||
+25
-1
@@ -113,7 +113,6 @@ class CommitTexts(_Strict):
|
|||||||
push_go: str
|
push_go: str
|
||||||
pushing: str
|
pushing: str
|
||||||
pushed: str
|
pushed: str
|
||||||
ci_pending: str
|
|
||||||
conflict: str
|
conflict: str
|
||||||
overwrite: str
|
overwrite: str
|
||||||
push_empty: str
|
push_empty: str
|
||||||
@@ -121,6 +120,27 @@ class CommitTexts(_Strict):
|
|||||||
push_failed: str
|
push_failed: str
|
||||||
|
|
||||||
|
|
||||||
|
class PipelineTexts(_Strict):
|
||||||
|
ci_pending: str
|
||||||
|
ci_running: str
|
||||||
|
ci_success: str
|
||||||
|
ci_failed: str
|
||||||
|
ci_timeout: str
|
||||||
|
cd_waiting: str
|
||||||
|
cd_running: str
|
||||||
|
cd_success: str
|
||||||
|
cd_failed: str
|
||||||
|
cd_not_run: str
|
||||||
|
cd_superseded: str
|
||||||
|
notify_ci_ok: str
|
||||||
|
notify_ci_failed: str
|
||||||
|
notify_ci_timeout: str
|
||||||
|
notify_cd_ok: str
|
||||||
|
deploy_failed: str
|
||||||
|
removed: str
|
||||||
|
log_file: str
|
||||||
|
|
||||||
|
|
||||||
class StatusTexts(_Strict):
|
class StatusTexts(_Strict):
|
||||||
title: str
|
title: str
|
||||||
line: str
|
line: str
|
||||||
@@ -148,6 +168,9 @@ class StatusTexts(_Strict):
|
|||||||
hours: str
|
hours: str
|
||||||
minutes: str
|
minutes: str
|
||||||
seconds: str
|
seconds: str
|
||||||
|
restart: str
|
||||||
|
restarted: str
|
||||||
|
restart_failed: str
|
||||||
|
|
||||||
|
|
||||||
class LogsTexts(_Strict):
|
class LogsTexts(_Strict):
|
||||||
@@ -227,6 +250,7 @@ class Texts(_Strict):
|
|||||||
section: SectionTexts
|
section: SectionTexts
|
||||||
members: MembersTexts
|
members: MembersTexts
|
||||||
commit: CommitTexts
|
commit: CommitTexts
|
||||||
|
pipeline: PipelineTexts
|
||||||
status: StatusTexts
|
status: StatusTexts
|
||||||
logs: LogsTexts
|
logs: LogsTexts
|
||||||
alerts: AlertsTexts
|
alerts: AlertsTexts
|
||||||
|
|||||||
+29
-1
@@ -119,7 +119,6 @@ commit:
|
|||||||
push_go: "✅ Запушить"
|
push_go: "✅ Запушить"
|
||||||
pushing: "⏳ Пушу…"
|
pushing: "⏳ Пушу…"
|
||||||
pushed: "✅ Запушено: {sha} — {message}"
|
pushed: "✅ Запушено: {sha} — {message}"
|
||||||
ci_pending: "⏳ CI…"
|
|
||||||
conflict: |-
|
conflict: |-
|
||||||
⚠️ Пока вы готовили черновик, в main изменились те же файлы:
|
⚠️ Пока вы готовили черновик, в main изменились те же файлы:
|
||||||
{paths}
|
{paths}
|
||||||
@@ -130,6 +129,32 @@ commit:
|
|||||||
push_limit: "Лимит — 20 пушей в час. Попробуйте позже."
|
push_limit: "Лимит — 20 пушей в час. Попробуйте позже."
|
||||||
push_failed: "Не удалось запушить: репозиторий недоступен. Черновик сохранён — попробуйте позже."
|
push_failed: "Не удалось запушить: репозиторий недоступен. Черновик сохранён — попробуйте позже."
|
||||||
|
|
||||||
|
pipeline:
|
||||||
|
ci_pending: "CI: ⏳ ждёт раннер"
|
||||||
|
ci_running: "CI: ⏳ идёт проверка"
|
||||||
|
ci_success: "CI: ✅ пройден"
|
||||||
|
ci_failed: "CI: ❌ упал: {step}"
|
||||||
|
ci_timeout: "CI: ⌛ не завершился за {minutes} мин"
|
||||||
|
cd_waiting: "CD: ⏸ {services} — после CI"
|
||||||
|
cd_running: "CD: 🚀 разворачиваю {services}…"
|
||||||
|
cd_success: "CD: 🚀 {services} развёрнуто"
|
||||||
|
cd_failed: "CD: ⛔ не развёрнуто: {services}"
|
||||||
|
cd_not_run: "CD: ⏭ не запускался — CI не прошёл"
|
||||||
|
cd_superseded: "CD: ⏭ эти сервисы развернёт более новый пуш"
|
||||||
|
notify_ci_ok: |-
|
||||||
|
✅ CI пройден · {sha}
|
||||||
|
{url}
|
||||||
|
notify_ci_failed: |-
|
||||||
|
❌ CI упал на шаге <b>{step}</b> · {sha}
|
||||||
|
Сервисы не развёрнуты. {url}
|
||||||
|
notify_ci_timeout: "⌛ CI не завершился за {minutes} мин · {sha} — сервисы не развёрнуты."
|
||||||
|
notify_cd_ok: "🚀 CD: сервисы {services} развёрнуты, ревизия {sha}"
|
||||||
|
deploy_failed: |-
|
||||||
|
⛔ <b>{service}</b> не развёрнут: {reason}
|
||||||
|
Работает прежняя версия, если она была.
|
||||||
|
removed: "🗑 <b>{service}</b> удалён из репозитория — остановлен на хосте"
|
||||||
|
log_file: "Полный лог CI"
|
||||||
|
|
||||||
status:
|
status:
|
||||||
title: "🩺 Статус · {count} конт. · {time} UTC"
|
title: "🩺 Статус · {count} конт. · {time} UTC"
|
||||||
line: "{light} {name} · {state}"
|
line: "{light} {name} · {state}"
|
||||||
@@ -163,6 +188,9 @@ status:
|
|||||||
hours: "ч"
|
hours: "ч"
|
||||||
minutes: "мин"
|
minutes: "мин"
|
||||||
seconds: "с"
|
seconds: "с"
|
||||||
|
restart: "♻️ Перезапустить"
|
||||||
|
restarted: "♻️ Перезапущен"
|
||||||
|
restart_failed: "Не удалось перезапустить: {reason}"
|
||||||
|
|
||||||
logs:
|
logs:
|
||||||
title: "📜 Логи · выберите контейнер"
|
title: "📜 Логи · выберите контейнер"
|
||||||
|
|||||||
@@ -7,6 +7,9 @@ services:
|
|||||||
DATABASE_URL: postgresql+asyncpg://tgit:${POSTGRES_PASSWORD:-tgit}@postgres:5432/tgit
|
DATABASE_URL: postgresql+asyncpg://tgit:${POSTGRES_PASSWORD:-tgit}@postgres:5432/tgit
|
||||||
GIT_CLONE_DIR: /data/uploads
|
GIT_CLONE_DIR: /data/uploads
|
||||||
DOCKER_URL: http://docker-proxy:2375
|
DOCKER_URL: http://docker-proxy:2375
|
||||||
|
# tgit-runner is reached over SSH on the host itself (RUNNER_HOST=host.docker.internal)
|
||||||
|
extra_hosts:
|
||||||
|
- host.docker.internal:host-gateway
|
||||||
volumes:
|
volumes:
|
||||||
- uploads:/data
|
- uploads:/data
|
||||||
depends_on:
|
depends_on:
|
||||||
|
|||||||
+43
-19
@@ -11,8 +11,8 @@
|
|||||||
│ tgit-bot (один процесс, asyncio) │
|
│ tgit-bot (один процесс, asyncio) │
|
||||||
│ ├─ bot/ меню, FSM, роутеры разделов, middleware доступа │
|
│ ├─ bot/ меню, FSM, роутеры разделов, middleware доступа │
|
||||||
│ ├─ access/ приглашения (QR), участники, антиперебор │
|
│ ├─ access/ приглашения (QR), участники, антиперебор │
|
||||||
│ ├─ repo/ дерево репозитория, черновики, Publisher (git + SSH) ────────┼──► SourceCraft
|
│ ├─ repo/ дерево репозитория, черновики, Publisher (git + SSH) ────────┼──► tgit-uploads (Gitea)
|
||||||
│ ├─ pipeline/ PipelineWatcher: CI и CD по sha, уведомления автору ◄────────┼─── REST API
|
│ ├─ pipeline/ PipelineWatcher: CI по sha → CD, уведомления автору ◄────────┼─── Gitea Actions API
|
||||||
│ ├─ runtime/ состояние контейнеров, проверки, логи │
|
│ ├─ runtime/ состояние контейнеров, проверки, логи │
|
||||||
│ ├─ logs/ grep-движок (Python, без shell) │
|
│ ├─ logs/ grep-движок (Python, без shell) │
|
||||||
│ ├─ agent/ фильтр → LLM-страж → LLM → LogQuery ─────────────────────────┼──► LLM API
|
│ ├─ agent/ фильтр → LLM-страж → LLM → LogQuery ─────────────────────────┼──► LLM API
|
||||||
@@ -28,7 +28,7 @@
|
|||||||
│ │ tgit-svc-<service> контейнеры (песочница) │
|
│ │ tgit-svc-<service> контейнеры (песочница) │
|
||||||
└─────┼──────────────────────────────────────────────────────────────────────── ┘
|
└─────┼──────────────────────────────────────────────────────────────────────── ┘
|
||||||
│
|
│
|
||||||
SourceCraft CI (tgit-uploads): check → deploy ──┘
|
бот (pipeline/), после зелёного CI tgit-uploads ──┘
|
||||||
```
|
```
|
||||||
|
|
||||||
**Репозитории:**
|
**Репозитории:**
|
||||||
@@ -36,7 +36,7 @@
|
|||||||
| Репозиторий | Что внутри | CI/CD |
|
| Репозиторий | Что внутри | CI/CD |
|
||||||
|---|---|---|
|
|---|---|---|
|
||||||
| `tgit` (Gitea → зеркало в SourceCraft) | код бота | push в любую ветку: lint + types + tests; push `main`: + деплой бота на хост |
|
| `tgit` (Gitea → зеркало в SourceCraft) | код бота | push в любую ветку: lint + types + tests; push `main`: + деплой бота на хост |
|
||||||
| `tgit-uploads` (SourceCraft) | общий репозиторий команды: `services/<name>/…` и любые файлы | push `main`: `check` → при успехе `deploy` изменённых сервисов через `tgit-runner` |
|
| `tgit-uploads` (Gitea, ADR-015/016) | общий репозиторий команды: `services/<name>/…` и любые файлы | push `main`: `check` в Gitea Actions → при успехе бот разворачивает изменённые сервисы через `tgit-runner` |
|
||||||
|
|
||||||
## 2. Компоненты
|
## 2. Компоненты
|
||||||
|
|
||||||
@@ -73,10 +73,26 @@ aiogram 3. Главное меню — reply-клавиатура (📝 Комм
|
|||||||
pydantic-модели ответов. Методы: `list_runs`, `get_run`, `get_cube_logs` (постранично), `run_commit_sha` — sha из лога куба `check/meta` (у прогона в API нет коммита, ADR-014). Подробности — [`dev/SOURCECRAFT_API.md`](dev/SOURCECRAFT_API.md).
|
pydantic-модели ответов. Методы: `list_runs`, `get_run`, `get_cube_logs` (постранично), `run_commit_sha` — sha из лога куба `check/meta` (у прогона в API нет коммита, ADR-014). Подробности — [`dev/SOURCECRAFT_API.md`](dev/SOURCECRAFT_API.md).
|
||||||
|
|
||||||
### 2.5 `app/pipeline` — наблюдатель CI/CD
|
### 2.5 `app/pipeline` — наблюдатель CI/CD
|
||||||
Фоновая задача. Пока есть незавершённые пуши — опрашивает `/cicd/runs` (10 с, бэкофф до 60 с), сопоставляет прогон с
|
CI репозитория загрузок — Gitea Actions (`.gitea/workflows/check.yml`), CD — сам бот через `tgit-runner` (ADR-016).
|
||||||
пушем по sha (из куба `meta`, ADR-014). Отслеживает задачи `check` (CI) и `deploy` (CD) workflow `check-and-deploy` отдельно; на смене статуса — редактирует сообщение
|
|
||||||
о пуше и шлёт автору уведомление (успех / падение + лог упавшего куба). Падение `deploy` → событие для `alerts`.
|
```
|
||||||
Таймаут — 30 мин. Webhooks у SourceCraft нет (ADR-003).
|
push ─► pushes(ci=pending, cd=pending|skipped) ─► wake()
|
||||||
|
PipelineWatcher: CIProvider.find(sha) каждые 10 с (бэкофф до 60 с при ошибках API)
|
||||||
|
├ pending/running ─► правка сообщения о push
|
||||||
|
├ success/skipped ─► «✅ CI пройден» ─► Deployer (tgit-runner) ─► 🚀 / ⛔ + алерт ⛔ всем
|
||||||
|
├ failed ─► «❌ CI упал на шаге …» + хвост лога (длинный — ещё и файлом), CD не запускается
|
||||||
|
└ нет вердикта 30 мин ─► ⌛ timeout, CD не запускается
|
||||||
|
```
|
||||||
|
- **Провайдер за протоколом** `CIProvider` (`find(sha)`, `failure_log(run)`): `gitea.py` — у прогона Gitea есть
|
||||||
|
`head_sha`, сопоставление прямое; SourceCraft (ADR-014, куб `meta`) подключится второй реализацией. `CI_PROVIDER=none` —
|
||||||
|
CI не ждём, сервисы разворачиваются сразу после push.
|
||||||
|
- **Состояние — в БД**: `pushes.ci_status/cd_status` и `pipeline_runs` (`ci` — id прогона, ссылка, упавший шаг;
|
||||||
|
`cd` — `tgit-runner`, не развёрнутые сервисы). Сообщение о push каждый раз рендерится из БД. После рестарта бот
|
||||||
|
продолжает: ждёт CI дальше, а прерванный деплой (`cd=running`) повторяет.
|
||||||
|
- **Опрос только по делу**: нет незавершённых push — цикл спит до `wake()` от нового push.
|
||||||
|
- **Порядок версий**: прогоны CI могут закончиться не по порядку; старый push не разворачивает сервис, который уже
|
||||||
|
развернёт более новый прошедший CI push — иначе откатил бы его.
|
||||||
|
- Правка сообщения не уведомляет, поэтому исходы — отдельными сообщениями автору ответом на сообщение о push (C-30…C-32).
|
||||||
|
|
||||||
### 2.6 `app/runtime` — контейнеры и проверки
|
### 2.6 `app/runtime` — контейнеры и проверки
|
||||||
Через `docker-socket-proxy` (только `GET`):
|
Через `docker-socket-proxy` (только `GET`):
|
||||||
@@ -134,7 +150,7 @@ pydantic-модели ответов. Методы: `list_runs`, `get_run`, `get
|
|||||||
закрывает. Пропущенное событие не теряет алерт, а бесконечный рестарт остаётся одним алертом.
|
закрывает. Пропущенное событие не теряет алерт, а бесконечный рестарт остаётся одним алертом.
|
||||||
- **Дедупликация** (`service.py`): ключ `kind + service`. Условие, вернувшееся в течение 10 мин после закрытия, снова
|
- **Дедупликация** (`service.py`): ключ `kind + service`. Условие, вернувшееся в течение 10 мин после закрытия, снова
|
||||||
открывает тот же алерт с `count += 1`: исходные сообщения редактируются, новые не шлются.
|
открывает тот же алерт с `count += 1`: исходные сообщения редактируются, новые не шлются.
|
||||||
- **Разовые события** (🛡 атака через агента; 📦 CD и ⛔ отказ после деплоя — после #11/#13) открываются сразу
|
- **Разовые события** (🛡 атака через агента; ⛔ отказ при деплое — от `Deployer`) открываются сразу
|
||||||
закрытыми; повтор в окне 10 мин — тоже счётчик.
|
закрытыми; повтор в окне 10 мин — тоже счётчик.
|
||||||
- **Рассылка** (`broadcast.py`) — всем активным участникам, 🛡 — только администраторам; ~25 сообщений/с,
|
- **Рассылка** (`broadcast.py`) — всем активным участникам, 🛡 — только администраторам; ~25 сообщений/с,
|
||||||
`RetryAfter` пережидается, заблокировавший бота участник пропускается. `alert_deliveries` хранит
|
`RetryAfter` пережидается, заблокировавший бота участник пропускается. `alert_deliveries` хранит
|
||||||
@@ -186,9 +202,9 @@ SQLAlchemy 2 (async) + Alembic. PostgreSQL в Compose; в тестах — SQLit
|
|||||||
│
|
│
|
||||||
Publisher (очередь) ─ fetch, конфликт?, commit, push ──► tgit-uploads@main
|
Publisher (очередь) ─ fetch, конфликт?, commit, push ──► tgit-uploads@main
|
||||||
│ SourceCraft CI
|
│ SourceCraft CI
|
||||||
«a1b2c3d ⏳ CI» ◄──────────────────────────────────────────────────────────────────────────┘ check: py_compile · ruff · pytest · bandit
|
«a1b2c3d ⏳ CI» ◄──────────────────────────────────────────────────────────────────────────┘ Gitea Actions check: compile · ruff · bandit · pytest
|
||||||
PipelineWatcher ─ poll runs(sha) ─► «✅ CI» ─► «⏳ CD» ─► «🚀 CD: weather» deploy: ssh → tgit-runner deploy <svc> <sha>
|
PipelineWatcher ─ poll runs(sha) ─► «✅ CI» ─► «🚀 CD: weather…» ─► «🚀 CD: weather развёрнут» CD: бот → ssh tgit-runner deploy <svc> <sha>
|
||||||
└► ❌ + лог шага автору; ❌ CD → алерт всем
|
└► ❌ + лог шага автору; ⛔ CD → алерт всем
|
||||||
```
|
```
|
||||||
|
|
||||||
### 3.3 Логи через агента
|
### 3.3 Логи через агента
|
||||||
@@ -203,7 +219,9 @@ SQLAlchemy 2 (async) + Alembic. PostgreSQL в Compose; в тестах — SQLit
|
|||||||
`authorized_keys` пользователя `tgit-deploy`). Подкоманды: `deploy <service> <sha>`, `restart <service>`,
|
`authorized_keys` пользователя `tgit-deploy`). Подкоманды: `deploy <service> <sha>`, `restart <service>`,
|
||||||
`probe <service> <path>` (HTTP-проба изнутри сети сервиса). Аргументы валидируются: имя `^[a-z][a-z0-9-]{1,30}$`, sha — hex.
|
`probe <service> <path>` (HTTP-проба изнутри сети сервиса). Аргументы валидируются: имя `^[a-z][a-z0-9-]{1,30}$`, sha — hex.
|
||||||
|
|
||||||
`deploy`: забирает архив `services/<service>` по sha из `tgit-uploads` → собирает образ `tgit-svc-<service>:<sha>` из
|
`deploy`: получает архив `services/<service>` в stdin — его делает бот (`git archive <sha>:services/<service>`)
|
||||||
|
после зелёного CI (§2.5), так что у хоста нет доступа к репозиторию → проверяет архив (только файлы и папки, без `..`,
|
||||||
|
≤ 20 МБ, есть `main.py`) → собирает образ `tgit-svc-<service>:<sha>` из
|
||||||
шаблона (`python:3.12-slim` + `requirements.txt`, `HEALTHCHECK` из `tgit.yaml` сервиса, если задан) → запускает:
|
шаблона (`python:3.12-slim` + `requirements.txt`, `HEALTHCHECK` из `tgit.yaml` сервиса, если задан) → запускает:
|
||||||
|
|
||||||
```
|
```
|
||||||
@@ -216,7 +234,11 @@ docker run -d --name tgit-svc-<service> --label tgit.managed=true --label tgit.s
|
|||||||
--restart on-failure:3
|
--restart on-failure:3
|
||||||
--log-opt max-size=5m --log-opt max-file=2
|
--log-opt max-size=5m --log-opt max-file=2
|
||||||
```
|
```
|
||||||
Если новый контейнер не стал healthy за 60 с — откат на предыдущий образ и ненулевой код выхода (CD падает → алерт ⛔).
|
Новая версия стартует рядом со старой как `tgit-svc-<service>-next` и заменяет её, только став healthy (с
|
||||||
|
`health_url` — по `HEALTHCHECK`, без него — 10 с без рестартов). Иначе новый контейнер удаляется, старый продолжает
|
||||||
|
работать, код выхода 3 → бот пишет автору push «⛔ не развёрнут» с хвостом лога и поднимает алерт ⛔ всей команде.
|
||||||
|
Сервис, папка которого исчезла из коммита, удаляется (`remove`). Перезапуск — `restart` из «Статуса» (админ).
|
||||||
|
Установка и обновление — [`infra/runner/README.md`](../infra/runner/README.md).
|
||||||
|
|
||||||
## 5. CI/CD
|
## 5. CI/CD
|
||||||
|
|
||||||
@@ -229,13 +251,15 @@ docker run -d --name tgit-svc-<service> --label tgit.managed=true --label tgit.s
|
|||||||
| push в любую ветку (через зеркало Gitea; PR живут в Gitea — ADR-010) | `check` | `uv sync --frozen` → `ruff check` + `ruff format --check` → `mypy app` → `pytest --cov=app --cov-fail-under=70` |
|
| push в любую ветку (через зеркало Gitea; PR живут в Gitea — ADR-010) | `check` | `uv sync --frozen` → `ruff check` + `ruff format --check` → `mypy app` → `pytest --cov=app --cov-fail-under=70` |
|
||||||
| push в `main` | `check` + `deploy` | → `infra/deploy/deploy.sh` по SSH (секреты `DEPLOY_SSH_KEY`, `DEPLOY_HOST`) → smoke: healthcheck контейнера |
|
| push в `main` | `check` + `deploy` | → `infra/deploy/deploy.sh` по SSH (секреты `DEPLOY_SSH_KEY`, `DEPLOY_HOST`) → smoke: healthcheck контейнера |
|
||||||
|
|
||||||
### 5.2 Репозиторий загрузок (`tgit-uploads/.sourcecraft/ci.yaml`)
|
### 5.2 Репозиторий загрузок (`HSE/tgit-uploads`, `.gitea/workflows/check.yml`)
|
||||||
Шаблон хранится в `tgit/infra/uploads-template/` и ставится в `tgit-uploads` один раз.
|
Шаблон хранится в `tgit/infra/uploads-template/` и ставится в `tgit-uploads` один раз (ADR-016).
|
||||||
|
|
||||||
| Workflow | Шаги |
|
| Что | Шаги |
|
||||||
|---|---|
|
|---|---|
|
||||||
| `check` | изменённые файлы (`git diff --name-only HEAD~1`) → `py_compile` · `ruff check` (мягкий набор правил) · `pytest` для затронутых сервисов · `bandit` |
|
| `check` (Gitea Actions, push в `main`) | `compileall` → `ruff check` (мягкий набор правил) → `bandit -ll` → `pytest` для сервисов с тестами; первый упавший шаг бот показывает автору |
|
||||||
| `deploy` (после `check`) | изменённые `services/<name>/` → `ssh tgit-deploy@host deploy <name> <sha>` для каждого |
|
| CD (бот, после зелёного `check`) | изменённые `services/<name>/` → `git archive` → `ssh tgit-deploy@host deploy <name> <sha>` для каждого |
|
||||||
|
|
||||||
|
`.sourcecraft/ci.yaml` в шаблоне остаётся для возврата к SourceCraft.
|
||||||
|
|
||||||
## 6. Деплой
|
## 6. Деплой
|
||||||
|
|
||||||
|
|||||||
@@ -137,3 +137,21 @@
|
|||||||
**Решение:** CI (`.gitea/workflows/ci.yml`: ruff, mypy, миграции на Postgres, pytest) и CD (`.gitea/workflows/deploy.yml` → `infra/deploy/deploy.sh`) — на Gitea Actions, как у Vobot и meditron, на общих раннерах Gitea. Деплой — на хост Vobot, `dev`/`prod` по веткам, через пользователя `tgit-deploy`. Репозиторий загрузок на сервере — временно `HSE/tgit-uploads` в Gitea.
|
**Решение:** CI (`.gitea/workflows/ci.yml`: ruff, mypy, миграции на Postgres, pytest) и CD (`.gitea/workflows/deploy.yml` → `infra/deploy/deploy.sh`) — на Gitea Actions, как у Vobot и meditron, на общих раннерах Gitea. Деплой — на хост Vobot, `dev`/`prod` по веткам, через пользователя `tgit-deploy`. Репозиторий загрузок на сервере — временно `HSE/tgit-uploads` в Gitea.
|
||||||
**Альтернативы:** ждать SourceCraft — блокирует проверку и демо.
|
**Альтернативы:** ждать SourceCraft — блокирует проверку и демо.
|
||||||
**Последствия:** `.sourcecraft/ci.yaml` бота остаётся для будущего возврата; при возврате к SourceCraft меняются `GIT_UPLOADS_URL` и ключ, код не трогаем. На RF-сервере Telegram — только через прокси (`TELEGRAM_PROXY_URL`).
|
**Последствия:** `.sourcecraft/ci.yaml` бота остаётся для будущего возврата; при возврате к SourceCraft меняются `GIT_UPLOADS_URL` и ключ, код не трогаем. На RF-сервере Telegram — только через прокси (`TELEGRAM_PROXY_URL`).
|
||||||
|
|
||||||
|
## ADR-016: CI репозитория загрузок — Gitea Actions; CD — бот после зелёного CI
|
||||||
|
**Дата:** 2026-10-19 · **Статус:** принято (меняет ADR-003/014 в части репозитория загрузок; SourceCraft — при возврате)
|
||||||
|
|
||||||
|
**Контекст:** после ADR-015 репозиторий загрузок живёт в Gitea (`HSE/tgit-uploads`), а CI у него не было: бот
|
||||||
|
разворачивал сервис сразу после push, проверив только синтаксис, и «⏳ CI» в сообщении не обновлялся (#11). SourceCraft
|
||||||
|
по-прежнему не принимает ключ бота (#7).
|
||||||
|
**Решение:** CI загрузок — Gitea Actions (`.gitea/workflows/check.yml`: compile · ruff · bandit · pytest). Бот
|
||||||
|
опрашивает Gitea API (`/actions/runs?head_sha=…`) — у прогона есть sha, куб `meta` не нужен. CD остаётся у бота: после
|
||||||
|
зелёного `check` он отправляет изменённые сервисы в `tgit-runner`, как в #13. Наблюдатель зависит от протокола
|
||||||
|
`CIProvider`, вендорский код — в `app/pipeline/gitea.py`.
|
||||||
|
**Альтернативы:** SourceCraft — блокирован ключом бота; деплой из самого workflow (ssh на хост из раннера Gitea) — ключ
|
||||||
|
к `tgit-runner` пришлось бы отдать общим раннерам Gitea, а бот всё равно должен знать исход деплоя для автора и алертов;
|
||||||
|
не ждать CI — ruff/bandit/тесты из CI ничего бы не решали.
|
||||||
|
**Последствия:** боту нужен токен с чтением Actions репозитория загрузок (`CI_TOKEN`); без `CI_PROVIDER` поведение
|
||||||
|
прежнее (деплой сразу). Деплой ждёт CI: на общих раннерах это от ~30 с до минут при очереди, таймаут — 30 мин.
|
||||||
|
Возврат к SourceCraft — вторая реализация `CIProvider` поверх готового `SourceCraftClient` (ADR-014), код наблюдателя
|
||||||
|
не меняется.
|
||||||
|
|||||||
+2
-1
@@ -12,6 +12,7 @@ CD — `.gitea/workflows/deploy.yml` (Gitea Actions, ADR-015): push в `dev` →
|
|||||||
| Контейнеры | Compose-проект `tgit-<env>`: `tgit-<env>-bot-1`, `tgit-<env>-postgres-1`; volumes `pgdata`, `uploads` |
|
| Контейнеры | Compose-проект `tgit-<env>`: `tgit-<env>-bot-1`, `tgit-<env>-postgres-1`; volumes `pgdata`, `uploads` |
|
||||||
| Telegram | с сервера напрямую недоступен → `TELEGRAM_PROXY_URL` (тот же прокси, что у meditron) |
|
| Telegram | с сервера напрямую недоступен → `TELEGRAM_PROXY_URL` (тот же прокси, что у meditron) |
|
||||||
| Репозиторий загрузок | временно `ssh://git@gitea.24studbank.ru:2222/HSE/tgit-uploads.git`, deploy key бота с записью |
|
| Репозиторий загрузок | временно `ssh://git@gitea.24studbank.ru:2222/HSE/tgit-uploads.git`, deploy key бота с записью |
|
||||||
|
| CI загрузок | Gitea Actions `HSE/tgit-uploads` (`check.yml`); боту в `.env` — `CI_PROVIDER=gitea`, `CI_TOKEN` (чтение репозитория загрузок). Без них деплой сервисов идёт сразу после push, без CI |
|
||||||
|
|
||||||
Секреты репозитория в Gitea (Settings → Actions → Secrets): `DEPLOY_SSH_KEY` (приватный ключ `tgit-cd@gitea`), `DEPLOY_HOST` (`tgit-deploy@72.56.247.166`).
|
Секреты репозитория в Gitea (Settings → Actions → Secrets): `DEPLOY_SSH_KEY` (приватный ключ `tgit-cd@gitea`), `DEPLOY_HOST` (`tgit-deploy@72.56.247.166`).
|
||||||
|
|
||||||
@@ -42,4 +43,4 @@ cd /srv/tgit/dev && rm -rf app && mv app.old app && cd app && docker compose -p
|
|||||||
|
|
||||||
Нужен **отдельный токен бота** (один токен — один процесс polling): создать бота у @BotFather →
|
Нужен **отдельный токен бота** (один токен — один процесс polling): создать бота у @BotFather →
|
||||||
`/srv/tgit/prod/.env` по образцу dev (`TELEGRAM_BOT_TOKEN`, `ADMIN_IDS`, `POSTGRES_PASSWORD`, `GIT_UPLOADS_URL`,
|
`/srv/tgit/prod/.env` по образцу dev (`TELEGRAM_BOT_TOKEN`, `ADMIN_IDS`, `POSTGRES_PASSWORD`, `GIT_UPLOADS_URL`,
|
||||||
`GIT_SSH_KEY_B64`, `TELEGRAM_PROXY_URL`, `LLM_PROVIDER`). Пока файла нет, деплой `main` пропускается.
|
`GIT_SSH_KEY_B64`, `TELEGRAM_PROXY_URL`, `LLM_PROVIDER`, `CI_PROVIDER`, `CI_TOKEN`, `RUNNER_*`). Пока файла нет, деплой `main` пропускается.
|
||||||
|
|||||||
@@ -15,6 +15,12 @@
|
|||||||
|
|
||||||
После `make demo-down` открытые алерты закрываются сообщением «✅ Восстановлено».
|
После `make demo-down` открытые алерты закрываются сообщением «✅ Восстановлено».
|
||||||
|
|
||||||
|
## Песочница — `tgit-runner`
|
||||||
|
|
||||||
|
[`escape_attempts/`](escape_attempts/) — сервис, который пробует выбраться: запись в код и `/etc`, сеть, DNS,
|
||||||
|
docker.sock, `setuid(0)`, raw-сокет, 200 процессов, запуск из `/tmp`. Положите его в `services/escape/` репозитория
|
||||||
|
загрузок — в «📜 Логи» каждая строка должна быть `BLOCKED`.
|
||||||
|
|
||||||
## Коммит — «📝 Коммит»
|
## Коммит — «📝 Коммит»
|
||||||
|
|
||||||
| Файл | Что показывает |
|
| Файл | Что показывает |
|
||||||
|
|||||||
@@ -0,0 +1,97 @@
|
|||||||
|
"""Tries to break out of the tgit-runner sandbox and reports each attempt. Every line should say BLOCKED.
|
||||||
|
|
||||||
|
Deploy it like any service (put it into services/escape/ of the uploads repo) and read its log in «📜 Логи».
|
||||||
|
Memory is not probed here: past 128 MB the kernel OOM-kills the process — the limit works, but the report would die.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import os
|
||||||
|
import socket
|
||||||
|
import subprocess
|
||||||
|
import time
|
||||||
|
|
||||||
|
|
||||||
|
def attempt(name: str, action) -> None: # type: ignore[no-untyped-def]
|
||||||
|
try:
|
||||||
|
detail = action()
|
||||||
|
except Exception as e: # each failure is the expected outcome
|
||||||
|
print(f"BLOCKED {name}: {type(e).__name__}: {e}", flush=True)
|
||||||
|
return
|
||||||
|
print(f"ALLOWED {name}: {detail}", flush=True)
|
||||||
|
|
||||||
|
|
||||||
|
def write_code() -> str:
|
||||||
|
with open("/app/pwned.txt", "w") as f:
|
||||||
|
f.write("x")
|
||||||
|
return "wrote into /app"
|
||||||
|
|
||||||
|
|
||||||
|
def write_root() -> str:
|
||||||
|
with open("/etc/pwned", "w") as f:
|
||||||
|
f.write("x")
|
||||||
|
return "wrote into /etc"
|
||||||
|
|
||||||
|
|
||||||
|
def internet() -> str:
|
||||||
|
socket.create_connection(("1.1.1.1", 443), timeout=3).close()
|
||||||
|
return "reached 1.1.1.1:443"
|
||||||
|
|
||||||
|
|
||||||
|
def dns() -> str:
|
||||||
|
return socket.gethostbyname("example.com")
|
||||||
|
|
||||||
|
|
||||||
|
def docker_socket() -> str:
|
||||||
|
s = socket.socket(socket.AF_UNIX)
|
||||||
|
s.connect("/var/run/docker.sock")
|
||||||
|
return "connected to docker.sock"
|
||||||
|
|
||||||
|
|
||||||
|
def become_root() -> str:
|
||||||
|
os.setuid(0)
|
||||||
|
return "setuid(0) worked"
|
||||||
|
|
||||||
|
|
||||||
|
def raw_socket() -> str:
|
||||||
|
socket.socket(socket.AF_INET, socket.SOCK_RAW, socket.IPPROTO_ICMP)
|
||||||
|
return "opened a raw socket"
|
||||||
|
|
||||||
|
|
||||||
|
def fork_bomb() -> str:
|
||||||
|
procs = []
|
||||||
|
try:
|
||||||
|
for _ in range(200):
|
||||||
|
procs.append(subprocess.Popen(["sleep", "30"]))
|
||||||
|
finally:
|
||||||
|
started = len(procs)
|
||||||
|
for p in procs:
|
||||||
|
p.kill()
|
||||||
|
if started >= 200:
|
||||||
|
return "started 200 processes"
|
||||||
|
raise RuntimeError(f"stopped after {started} processes")
|
||||||
|
|
||||||
|
|
||||||
|
def exec_tmp() -> str:
|
||||||
|
path = "/tmp/run.sh"
|
||||||
|
with open(path, "w") as f:
|
||||||
|
f.write("#!/bin/sh\necho hi\n")
|
||||||
|
os.chmod(path, 0o755)
|
||||||
|
return subprocess.run([path], capture_output=True, check=True, text=True).stdout.strip()
|
||||||
|
|
||||||
|
|
||||||
|
if __name__ == "__main__":
|
||||||
|
print(f"uid={os.getuid()} gid={os.getgid()}", flush=True)
|
||||||
|
for name, fn in [
|
||||||
|
("write into the code dir", write_code),
|
||||||
|
("write into /etc", write_root),
|
||||||
|
("connect to the internet", internet),
|
||||||
|
("resolve DNS", dns),
|
||||||
|
("talk to docker.sock", docker_socket),
|
||||||
|
("setuid(0)", become_root),
|
||||||
|
("raw socket", raw_socket),
|
||||||
|
("200 processes", fork_bomb),
|
||||||
|
("execute from /tmp", exec_tmp),
|
||||||
|
]:
|
||||||
|
attempt(name, fn)
|
||||||
|
print("done", flush=True)
|
||||||
|
while True:
|
||||||
|
time.sleep(3600)
|
||||||
+1
-1
@@ -3,5 +3,5 @@
|
|||||||
| Каталог | Что будет | Задача |
|
| Каталог | Что будет | Задача |
|
||||||
|---|---|---|
|
|---|---|---|
|
||||||
| [`deploy/`](deploy/) | `deploy.sh` + `remote.sh` — деплой бота на сервер (CD и руками), см. [`docs/dev/DEPLOY.md`](../docs/dev/DEPLOY.md) | #12 |
|
| [`deploy/`](deploy/) | `deploy.sh` + `remote.sh` — деплой бота на сервер (CD и руками), см. [`docs/dev/DEPLOY.md`](../docs/dev/DEPLOY.md) | #12 |
|
||||||
| `runner/` | `tgit-runner` — песочница и запуск сервисов на хосте | #13 |
|
| [`runner/`](runner/) | `tgit-runner` — песочница и запуск сервисов на хосте, установка | #13 |
|
||||||
| [`uploads-template/`](uploads-template/) | шаблон [`tgit-uploads`](https://sourcecraft.dev/rogachevegorushka/tgit-uploads): CI/CD, служебные скрипты `.tgit/`, `ruff.toml`, демо-сервис `hello` — правки вносить здесь и переносить в репозиторий загрузок | #7 |
|
| [`uploads-template/`](uploads-template/) | шаблон [`tgit-uploads`](https://sourcecraft.dev/rogachevegorushka/tgit-uploads): CI/CD, служебные скрипты `.tgit/`, `ruff.toml`, демо-сервис `hello` — правки вносить здесь и переносить в репозиторий загрузок | #7 |
|
||||||
|
|||||||
@@ -35,5 +35,9 @@ if [ "$status" != healthy ]; then
|
|||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
echo "$SHA" > ../REVISION
|
echo "$SHA" > ../REVISION
|
||||||
|
# Keep tgit-runner in step with the bot (installed once by infra/runner/install.sh; absent — nothing to update)
|
||||||
|
if [ -d /srv/tgit/runner ] && [ -w /srv/tgit/runner ]; then
|
||||||
|
install -m 755 infra/runner/tgit-runner /srv/tgit/runner/tgit-runner
|
||||||
|
fi
|
||||||
docker image prune -f >/dev/null
|
docker image prune -f >/dev/null
|
||||||
echo "bot healthy @ $SHA ($ENV)"
|
echo "bot healthy @ $SHA ($ENV)"
|
||||||
|
|||||||
@@ -0,0 +1,38 @@
|
|||||||
|
# infra/runner — `tgit-runner`
|
||||||
|
|
||||||
|
Запуск сервисов из репозитория загрузок (`services/<name>/`) на хосте в песочнице — ADR-005, ADR-006,
|
||||||
|
[ARCHITECTURE §4](../../docs/ARCHITECTURE.md).
|
||||||
|
|
||||||
|
```
|
||||||
|
бот: push ─► git archive <sha>:services/<name> ─► ssh tgit-deploy@host deploy <name> <sha> (архив — в stdin)
|
||||||
|
│ forced command: только tgit-runner
|
||||||
|
host: tgit-runner ─► проверка аргументов и архива ─► docker build ─► docker run …-next (песочница)
|
||||||
|
─► healthy за 60 с? ─► да: заменить старый контейнер · нет: удалить новый, старый работает, exit 3
|
||||||
|
```
|
||||||
|
|
||||||
|
| Команда | Что делает |
|
||||||
|
|---|---|
|
||||||
|
| `deploy <service> <sha>` | собрать образ `tgit-svc-<service>:<sha>` и запустить, старый заменяется только здоровым новым |
|
||||||
|
| `restart <service>` | перезапуск (кнопка «♻️ Перезапустить» в «Статусе», только админ) |
|
||||||
|
| `remove <service>` | сервис удалён из репозитория — остановить и удалить образы |
|
||||||
|
| `list` | контейнеры сервисов |
|
||||||
|
|
||||||
|
**Песочница:** uid 10001, read-only FS, `/tmp` — tmpfs 16 МБ с `noexec`, `--cap-drop ALL`, `no-new-privileges`,
|
||||||
|
128 МБ без swap, 0.25 CPU, 64 процесса, `--network none` (интернет — только если `network: true` в `tgit.yaml`
|
||||||
|
**и** имя сервиса есть в `/srv/tgit/runner/egress-allow`), без Docker-сокета, `--restart on-failure:3`, ротация логов.
|
||||||
|
Не больше 10 сервисов на хост; хранятся два последних образа сервиса.
|
||||||
|
|
||||||
|
**Проверено** [`examples/escape_attempts`](../../examples/escape_attempts/): запись в код и `/etc`, сеть, DNS,
|
||||||
|
docker.sock, `setuid(0)`, raw-сокет, 200 процессов, запуск из `/tmp` — всё `BLOCKED`.
|
||||||
|
|
||||||
|
**Установка** (один раз, root на хосте):
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh-keygen -t ed25519 -N '' -C tgit-runner -f tgit-runner-key # ключ бота для раннера
|
||||||
|
sudo infra/runner/install.sh tgit-runner-key.pub tgit-deploy
|
||||||
|
# в .env бота на сервере:
|
||||||
|
# RUNNER_HOST=host.docker.internal RUNNER_SSH_KEY_B64=$(base64 -w0 tgit-runner-key)
|
||||||
|
```
|
||||||
|
|
||||||
|
Дальше скрипт обновляется сам: деплой бота (`infra/deploy/remote.sh`) копирует свежий `tgit-runner`, если раннер
|
||||||
|
установлен. Тесты — `tests/runner/` (фейковые `docker` и `ssh`, сам скрипт настоящий).
|
||||||
@@ -0,0 +1,6 @@
|
|||||||
|
# ~tgit-deploy/.ssh/authorized_keys on the host. `restrict` = no-pty, no port/agent/X11 forwarding;
|
||||||
|
# command= makes the key run tgit-runner and nothing else, the requested command arrives as SSH_ORIGINAL_COMMAND.
|
||||||
|
command="/srv/tgit/runner/tgit-runner",restrict ssh-ed25519 AAAA…bot-runner-key… tgit-runner
|
||||||
|
|
||||||
|
# The CD key of the bot itself stays unrestricted (it deploys the bot; Gitea secret DEPLOY_SSH_KEY):
|
||||||
|
ssh-ed25519 AAAA…cd-key… tgit-cd@gitea
|
||||||
Executable
+28
@@ -0,0 +1,28 @@
|
|||||||
|
#!/usr/bin/env bash
|
||||||
|
# One-time setup of tgit-runner on the host (as root). Safe to re-run.
|
||||||
|
# sudo infra/runner/install.sh /path/to/tgit-runner-key.pub [deploy-user]
|
||||||
|
# The key is the bot's runner key (RUNNER_SSH_KEY_B64 holds its private half); it is pinned to the runner only.
|
||||||
|
set -euo pipefail
|
||||||
|
PUB=${1:?usage: install.sh <runner-key.pub> [user]}
|
||||||
|
USER_NAME=${2:-tgit-deploy}
|
||||||
|
DIR=/srv/tgit/runner
|
||||||
|
HERE=$(cd "$(dirname "$0")" && pwd)
|
||||||
|
|
||||||
|
id "$USER_NAME" >/dev/null
|
||||||
|
install -d -o "$USER_NAME" -g "$USER_NAME" -m 700 "$DIR"
|
||||||
|
install -o "$USER_NAME" -g "$USER_NAME" -m 755 "$HERE/tgit-runner" "$DIR/tgit-runner"
|
||||||
|
touch "$DIR/egress-allow" && chown "$USER_NAME:" "$DIR/egress-allow" && chmod 600 "$DIR/egress-allow"
|
||||||
|
|
||||||
|
HOME_DIR=$(getent passwd "$USER_NAME" | cut -d: -f6)
|
||||||
|
install -d -o "$USER_NAME" -g "$USER_NAME" -m 700 "$HOME_DIR/.ssh"
|
||||||
|
AK=$HOME_DIR/.ssh/authorized_keys
|
||||||
|
touch "$AK" && chown "$USER_NAME:" "$AK" && chmod 600 "$AK"
|
||||||
|
KEY=$(awk '{print $1" "$2}' "$PUB")
|
||||||
|
LINE="command=\"$DIR/tgit-runner\",restrict $KEY tgit-runner"
|
||||||
|
if grep -qF "$KEY" "$AK"; then
|
||||||
|
echo "key already in $AK — leaving it"
|
||||||
|
else
|
||||||
|
echo "$LINE" >>"$AK"
|
||||||
|
echo "added the runner key to $AK (forced command)"
|
||||||
|
fi
|
||||||
|
echo "runner installed in $DIR; allow internet for a service: echo <service> >> $DIR/egress-allow"
|
||||||
Executable
+184
@@ -0,0 +1,184 @@
|
|||||||
|
#!/usr/bin/env bash
|
||||||
|
# tgit-runner — the only command the bot's SSH key may run on the host (ADR-006, ARCHITECTURE §4).
|
||||||
|
# authorized_keys: command="/srv/tgit/runner/tgit-runner",restrict ssh-ed25519 AAAA… tgit-runner
|
||||||
|
#
|
||||||
|
# deploy <service> <sha> stdin: tar of services/<service>/ (files at the archive root)
|
||||||
|
# restart <service>
|
||||||
|
# remove <service>
|
||||||
|
# list
|
||||||
|
#
|
||||||
|
# Exit codes: 0 ok · 2 bad request · 3 not healthy (old container kept) · 4 limit · 5 build failed.
|
||||||
|
# The last line of output is machine-readable: «OK …» or «FAIL <reason>».
|
||||||
|
set -euo pipefail
|
||||||
|
umask 077
|
||||||
|
export LC_ALL=C
|
||||||
|
|
||||||
|
STATE=${TGIT_RUNNER_STATE:-/srv/tgit/runner}
|
||||||
|
MAX_SERVICES=${TGIT_MAX_SERVICES:-10}
|
||||||
|
HEALTH_TIMEOUT=${TGIT_HEALTH_TIMEOUT:-60}
|
||||||
|
SETTLE=${TGIT_SETTLE:-10} # no healthcheck: must stay up this long without restarts
|
||||||
|
MAX_TAR=${TGIT_MAX_TAR:-$((20 * 1024 * 1024))}
|
||||||
|
BASE_IMAGE=${TGIT_BASE_IMAGE:-python:3.12-slim}
|
||||||
|
EGRESS_NET=tgit-svc-egress
|
||||||
|
NAME_RE='^[a-z][a-z0-9-]{1,30}$'
|
||||||
|
SHA_RE='^[0-9a-f]{7,40}$'
|
||||||
|
HEALTH_RE='^http://(localhost|127\.0\.0\.1):[0-9]{2,5}(/[A-Za-z0-9._/-]*)?$'
|
||||||
|
|
||||||
|
fail() { echo "FAIL $2"; exit "$1"; }
|
||||||
|
|
||||||
|
# Called through SSH the arguments come in SSH_ORIGINAL_COMMAND: split on spaces, never evaluated, never globbed
|
||||||
|
if [ -n "${SSH_ORIGINAL_COMMAND+x}" ]; then
|
||||||
|
set -f
|
||||||
|
read -r -a ARGS <<<"$SSH_ORIGINAL_COMMAND"
|
||||||
|
set +f
|
||||||
|
else
|
||||||
|
ARGS=("$@")
|
||||||
|
fi
|
||||||
|
[ "${#ARGS[@]}" -ge 1 ] || fail 2 "usage: deploy|restart|remove|list"
|
||||||
|
CMD=${ARGS[0]}
|
||||||
|
|
||||||
|
need_service() {
|
||||||
|
[ "${#ARGS[@]}" -eq "$1" ] || fail 2 "wrong number of arguments for $CMD"
|
||||||
|
SVC=${ARGS[1]}
|
||||||
|
[[ $SVC =~ $NAME_RE ]] || fail 2 "bad service name"
|
||||||
|
NAME=tgit-svc-$SVC
|
||||||
|
}
|
||||||
|
|
||||||
|
exists() { docker container inspect "$1" >/dev/null 2>&1; }
|
||||||
|
|
||||||
|
# tgit.yaml is read with grep, not executed or parsed by anything clever
|
||||||
|
yaml_value() { grep -E "^$1:" "$2" 2>/dev/null | head -1 | sed -E "s/^$1:[[:space:]]*//; s/[[:space:]]+#.*$//; s/^[\"']//; s/[\"']$//" || true; }
|
||||||
|
|
||||||
|
wait_ready() { # $1 container, $2 has healthcheck (yes/no)
|
||||||
|
local deadline=$((SECONDS + HEALTH_TIMEOUT)) status restarts
|
||||||
|
if [ "$2" = yes ]; then
|
||||||
|
while [ $SECONDS -lt $deadline ]; do
|
||||||
|
status=$(docker inspect -f '{{.State.Health.Status}}' "$1" 2>/dev/null || echo gone)
|
||||||
|
case $status in healthy) return 0 ;; unhealthy | gone) return 1 ;; esac
|
||||||
|
sleep 2
|
||||||
|
done
|
||||||
|
return 1
|
||||||
|
fi
|
||||||
|
sleep "$SETTLE"
|
||||||
|
status=$(docker inspect -f '{{.State.Status}}' "$1" 2>/dev/null || echo gone)
|
||||||
|
restarts=$(docker inspect -f '{{.RestartCount}}' "$1" 2>/dev/null || echo 1)
|
||||||
|
[ "$status" = running ] && [ "$restarts" = 0 ]
|
||||||
|
}
|
||||||
|
|
||||||
|
deploy() {
|
||||||
|
need_service 3
|
||||||
|
SHA=${ARGS[2]}
|
||||||
|
[[ $SHA =~ $SHA_RE ]] || fail 2 "bad sha"
|
||||||
|
|
||||||
|
if ! exists "$NAME"; then
|
||||||
|
local count
|
||||||
|
count=$(docker ps -a --filter label=tgit.service --format '{{.Names}}' | grep -cv -- '-next$' || true)
|
||||||
|
[ "$count" -lt "$MAX_SERVICES" ] || fail 4 "service limit $MAX_SERVICES reached"
|
||||||
|
fi
|
||||||
|
|
||||||
|
mkdir -p "$STATE"
|
||||||
|
WORK=$(mktemp -d "$STATE/build.XXXXXX")
|
||||||
|
trap 'rm -rf "$WORK"' EXIT
|
||||||
|
head -c $((MAX_TAR + 1)) >"$WORK/src.tar"
|
||||||
|
[ "$(stat -c %s "$WORK/src.tar")" -le "$MAX_TAR" ] || fail 2 "archive larger than $MAX_TAR bytes"
|
||||||
|
# Only plain files and directories, relative paths without "..": no symlinks, devices or hard links
|
||||||
|
tar -tvf "$WORK/src.tar" >"$WORK/list" 2>/dev/null || fail 2 "not a tar archive"
|
||||||
|
if grep -qvE '^[-d]' "$WORK/list"; then fail 2 "archive may contain only files and directories"; fi
|
||||||
|
if tar -tf "$WORK/src.tar" | grep -qE '(^/|(^|/)\.\.(/|$))'; then fail 2 "unsafe path in archive"; fi
|
||||||
|
mkdir -p "$WORK/ctx/app"
|
||||||
|
tar -xf "$WORK/src.tar" -C "$WORK/ctx/app" --no-same-owner --no-same-permissions
|
||||||
|
[ -f "$WORK/ctx/app/main.py" ] || fail 2 "services/$SVC/main.py is missing"
|
||||||
|
# umask 077 above protects the build dir; inside the image the code must be readable (not writable) by uid 10001
|
||||||
|
chmod -R u=rwX,go=rX "$WORK/ctx"
|
||||||
|
|
||||||
|
local health network healthcheck=no net=none
|
||||||
|
health=$(yaml_value health_url "$WORK/ctx/app/tgit.yaml")
|
||||||
|
network=$(yaml_value network "$WORK/ctx/app/tgit.yaml")
|
||||||
|
if [ -n "$health" ]; then
|
||||||
|
[[ $health =~ $HEALTH_RE ]] || fail 2 "health_url must be http://localhost:<port>/<path>"
|
||||||
|
healthcheck=yes
|
||||||
|
fi
|
||||||
|
# Internet only if tgit.yaml asks AND the admin allowed this service on the host
|
||||||
|
if [ "$network" = true ] && grep -qxF "$SVC" "$STATE/egress-allow" 2>/dev/null; then
|
||||||
|
docker network inspect "$EGRESS_NET" >/dev/null 2>&1 || docker network create "$EGRESS_NET" >/dev/null
|
||||||
|
net=$EGRESS_NET
|
||||||
|
fi
|
||||||
|
|
||||||
|
{
|
||||||
|
echo "FROM $BASE_IMAGE"
|
||||||
|
echo "RUN useradd --uid 10001 --no-create-home --home-dir /tmp svc"
|
||||||
|
echo "WORKDIR /app"
|
||||||
|
echo "COPY app/ /app/"
|
||||||
|
echo "RUN if [ -f requirements.txt ]; then pip install --no-cache-dir --disable-pip-version-check -r requirements.txt; fi"
|
||||||
|
echo "ENV PYTHONUNBUFFERED=1 PYTHONDONTWRITEBYTECODE=1 HOME=/tmp"
|
||||||
|
if [ "$healthcheck" = yes ]; then
|
||||||
|
# exec form, the URL is an argument — nothing here goes through a shell
|
||||||
|
printf 'HEALTHCHECK --interval=5s --timeout=3s --start-period=10s --retries=3 CMD ["python", "-c", "import sys, urllib.request; urllib.request.urlopen(sys.argv[1], timeout=2)", "%s"]\n' "$health"
|
||||||
|
fi
|
||||||
|
echo "USER 10001:10001"
|
||||||
|
echo 'CMD ["python", "main.py"]'
|
||||||
|
} >"$WORK/ctx/Dockerfile"
|
||||||
|
|
||||||
|
IMAGE=$NAME:$SHA
|
||||||
|
echo "building $IMAGE"
|
||||||
|
if ! docker build -q -t "$IMAGE" --label tgit.service="$SVC" "$WORK/ctx" >"$WORK/build.log" 2>&1; then
|
||||||
|
tail -20 "$WORK/build.log"
|
||||||
|
fail 5 "build failed"
|
||||||
|
fi
|
||||||
|
|
||||||
|
NEXT=$NAME-next
|
||||||
|
docker rm -f "$NEXT" >/dev/null 2>&1 || true
|
||||||
|
docker run -d --name "$NEXT" \
|
||||||
|
--label tgit.managed=true --label tgit.service="$SVC" \
|
||||||
|
--label tgit.sha="$SHA" --label tgit.deployed_at="$(date -u +%Y-%m-%dT%H:%M:%SZ)" \
|
||||||
|
--user 10001:10001 --read-only --tmpfs /tmp:rw,noexec,nosuid,size=16m \
|
||||||
|
--cap-drop ALL --security-opt no-new-privileges \
|
||||||
|
--memory 128m --memory-swap 128m --cpus 0.25 --pids-limit 64 \
|
||||||
|
--network "$net" --restart on-failure:3 \
|
||||||
|
--log-opt max-size=5m --log-opt max-file=2 \
|
||||||
|
"$IMAGE" >/dev/null
|
||||||
|
|
||||||
|
if ! wait_ready "$NEXT" "$healthcheck"; then
|
||||||
|
echo "--- last log lines of the new version ---"
|
||||||
|
docker logs --tail 20 "$NEXT" 2>&1 || true
|
||||||
|
docker rm -f "$NEXT" >/dev/null 2>&1 || true
|
||||||
|
if exists "$NAME"; then fail 3 "not healthy within ${HEALTH_TIMEOUT}s, previous version kept"; fi
|
||||||
|
fail 3 "not healthy within ${HEALTH_TIMEOUT}s"
|
||||||
|
fi
|
||||||
|
docker rm -f "$NAME" >/dev/null 2>&1 || true
|
||||||
|
docker rename "$NEXT" "$NAME"
|
||||||
|
|
||||||
|
# Keep the current image and one previous; older ones go
|
||||||
|
{ docker image ls "$NAME" --format '{{.Tag}}' | grep -vxF "$SHA" || true; } | tail -n +2 | while read -r old; do
|
||||||
|
docker image rm "$NAME:$old" >/dev/null 2>&1 || true
|
||||||
|
done
|
||||||
|
echo "OK deployed $SVC $SHA network=$net healthcheck=$healthcheck"
|
||||||
|
}
|
||||||
|
|
||||||
|
restart() {
|
||||||
|
need_service 2
|
||||||
|
exists "$NAME" || fail 2 "no such service"
|
||||||
|
docker restart -t 10 "$NAME" >/dev/null
|
||||||
|
echo "OK restarted $SVC"
|
||||||
|
}
|
||||||
|
|
||||||
|
remove() {
|
||||||
|
need_service 2
|
||||||
|
docker rm -f "$NAME" "$NAME-next" >/dev/null 2>&1 || true
|
||||||
|
docker image ls "$NAME" --format '{{.Repository}}:{{.Tag}}' | while read -r img; do
|
||||||
|
docker image rm "$img" >/dev/null 2>&1 || true
|
||||||
|
done
|
||||||
|
echo "OK removed $SVC"
|
||||||
|
}
|
||||||
|
|
||||||
|
case $CMD in
|
||||||
|
deploy) deploy ;;
|
||||||
|
restart) restart ;;
|
||||||
|
remove) remove ;;
|
||||||
|
list)
|
||||||
|
[ "${#ARGS[@]}" -eq 1 ] || fail 2 "list takes no arguments"
|
||||||
|
docker ps -a --filter label=tgit.service --format '{{.Names}} {{.Status}} {{.Label "tgit.sha"}}'
|
||||||
|
echo "OK list"
|
||||||
|
;;
|
||||||
|
*) fail 2 "unknown command" ;;
|
||||||
|
esac
|
||||||
@@ -0,0 +1,39 @@
|
|||||||
|
# CI репозитория загрузок tgit на Gitea Actions (ADR-016). Шаблон живёт в HSE/tgit: infra/uploads-template/ — правки делать там.
|
||||||
|
# Бот находит этот прогон по sha своего push и разворачивает изменённые сервисы только после зелёного check (#11).
|
||||||
|
# Имя файла — CI_WORKFLOW бота. Из Telegram не редактируется (SPEC C-16).
|
||||||
|
name: check
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
|
||||||
|
# Без cancel-in-progress: каждый push ждёт свой вердикт — иначе бот не дождётся отменённого прогона.
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
check:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 25
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Setup Python 3.12
|
||||||
|
uses: actions/setup-python@v5
|
||||||
|
with:
|
||||||
|
python-version: "3.12"
|
||||||
|
|
||||||
|
- name: Install uv
|
||||||
|
# pip, а не astral-sh/setup-uv: на self-hosted Gitea экшен ходит в GitHub API и падает (опыт Vobot)
|
||||||
|
run: pip install uv
|
||||||
|
|
||||||
|
# Шаги последовательно: имя первого упавшего бот покажет автору push вместе с логом.
|
||||||
|
- name: compile
|
||||||
|
run: python -m compileall -q -x '(^|/)\.' .
|
||||||
|
|
||||||
|
- name: lint
|
||||||
|
run: uvx ruff@0.13 check .
|
||||||
|
|
||||||
|
- name: security
|
||||||
|
run: uvx bandit@1.8 -q -r services -ll
|
||||||
|
|
||||||
|
- name: tests
|
||||||
|
run: python .tgit/run_tests.py
|
||||||
@@ -47,5 +47,6 @@ workflows:
|
|||||||
script:
|
script:
|
||||||
- python .tgit/changed_services.py > changed.txt
|
- python .tgit/changed_services.py > changed.txt
|
||||||
- cat changed.txt
|
- cat changed.txt
|
||||||
# Запуск через tgit-runner по SSH — #13. До этого — только список сервисов к развёртыванию.
|
# Разворачивает бот после зелёного check (tgit-runner, #13); здесь — только список для лога.
|
||||||
- 'echo "deploy stub: $(wc -l < changed.txt) service(s) to deploy"'
|
# Сейчас CI загрузок — Gitea Actions (.gitea/workflows/check.yml, ADR-016); этот файл — для возврата.
|
||||||
|
- 'echo "$(wc -l < changed.txt) service(s) changed — deployed by the bot via tgit-runner"'
|
||||||
|
|||||||
@@ -11,12 +11,15 @@ services/<name>/ сервис = папка; имя — ^[a-z][a-z0-9-]{1,
|
|||||||
requirements.txt зависимости (необязательно)
|
requirements.txt зависимости (необязательно)
|
||||||
test_*.py тесты pytest (необязательно, запускаются в CI)
|
test_*.py тесты pytest (необязательно, запускаются в CI)
|
||||||
tgit.yaml настройки запуска: health_url, network (необязательно)
|
tgit.yaml настройки запуска: health_url, network (необязательно)
|
||||||
.sourcecraft/ci.yaml CI/CD — из Telegram не редактируется
|
.gitea/workflows/ CI (Gitea Actions) — из Telegram не редактируется
|
||||||
|
.sourcecraft/ci.yaml CI для SourceCraft — на будущее возвращение, сейчас не используется
|
||||||
.tgit/ служебные скрипты CI — из Telegram не редактируются
|
.tgit/ служебные скрипты CI — из Telegram не редактируются
|
||||||
ruff.toml мягкий набор правил линтера
|
ruff.toml мягкий набор правил линтера
|
||||||
```
|
```
|
||||||
|
|
||||||
## CI/CD (`check-and-deploy`, push в `main`)
|
## CI/CD (push в `main`)
|
||||||
|
|
||||||
|
CI — workflow `check` в Gitea Actions; CD делает бот, когда `check` зелёный.
|
||||||
|
|
||||||
| Шаг | Что делает | Падает, если |
|
| Шаг | Что делает | Падает, если |
|
||||||
|---|---|---|
|
|---|---|---|
|
||||||
@@ -24,7 +27,7 @@ ruff.toml мягкий набор правил линтера
|
|||||||
| `lint` | `ruff check` (мягкие правила) | неопределённые имена, неиспользуемые импорты, опасные значения по умолчанию |
|
| `lint` | `ruff check` (мягкие правила) | неопределённые имена, неиспользуемые импорты, опасные значения по умолчанию |
|
||||||
| `security` | `bandit -ll` по `services/` | найдена уязвимость уровня medium+ |
|
| `security` | `bandit -ll` по `services/` | найдена уязвимость уровня medium+ |
|
||||||
| `tests` | `pytest` для каждого сервиса с тестами, со своими `requirements.txt` | упал тест |
|
| `tests` | `pytest` для каждого сервиса с тестами, со своими `requirements.txt` | упал тест |
|
||||||
| `deploy` | список изменённых сервисов → `tgit-runner` на хосте | сервис не поднялся (healthy) за 60 с |
|
| CD (бот) | изменённые сервисы → `tgit-runner` на хосте | сервис не поднялся (healthy) за 60 с — работает прежняя версия |
|
||||||
|
|
||||||
## Развернуть заново
|
## Развернуть заново
|
||||||
|
|
||||||
|
|||||||
@@ -53,7 +53,7 @@ async def test_message_then_push_lands_in_remote(tg: TgClient, texts: Texts, rem
|
|||||||
pushing, done = tg.session.sent(EditMessageText)[-2:]
|
pushing, done = tg.session.sent(EditMessageText)[-2:]
|
||||||
assert pushing.text == t.pushing
|
assert pushing.text == t.pushing
|
||||||
assert done.text.startswith("✅ Запушено: ") and "docs: заметки" in done.text
|
assert done.text.startswith("✅ Запушено: ") and "docs: заметки" in done.text
|
||||||
assert "https://example.test/commit/" in done.text and done.text.endswith(t.ci_pending)
|
assert "https://example.test/commit/" in done.text and "CI" not in done.text # no CI configured
|
||||||
|
|
||||||
clone = tmp_path / "verify"
|
clone = tmp_path / "verify"
|
||||||
git(tmp_path, "clone", "-q", str(remote), str(clone))
|
git(tmp_path, "clone", "-q", str(remote), str(clone))
|
||||||
|
|||||||
@@ -0,0 +1,130 @@
|
|||||||
|
from typing import Any
|
||||||
|
|
||||||
|
import httpx
|
||||||
|
import pytest
|
||||||
|
import respx
|
||||||
|
|
||||||
|
from app.config import CISettings
|
||||||
|
from app.db.models import PipelineStatus
|
||||||
|
from app.pipeline.gitea import GiteaCI, GiteaError
|
||||||
|
from app.pipeline.provider import CIRun
|
||||||
|
|
||||||
|
API = "https://gitea.example.test"
|
||||||
|
RUNS = f"{API}/api/v1/repos/HSE/up/actions/runs"
|
||||||
|
SHA = "eda569427093acb3b39ad5b301216cb5b7400ab8"
|
||||||
|
|
||||||
|
|
||||||
|
def run(
|
||||||
|
id: int, path: str = "check.yml@refs/heads/main", status: str = "completed", conclusion: Any = "success"
|
||||||
|
) -> Any:
|
||||||
|
"""Shape of a real Gitea 1.27 run (fields the client doesn't read trimmed)."""
|
||||||
|
run = {"id": id, "path": path, "status": status, "head_sha": SHA, "html_url": f"{API}/HSE/up/actions/runs/{id}"}
|
||||||
|
return run | ({"conclusion": conclusion} if conclusion else {}) # in-progress runs have no conclusion key
|
||||||
|
|
||||||
|
|
||||||
|
def job(id: int, conclusion: str, steps: list[tuple[str, str]]) -> Any:
|
||||||
|
return {
|
||||||
|
"id": id,
|
||||||
|
"name": "check",
|
||||||
|
"status": "completed",
|
||||||
|
"conclusion": conclusion,
|
||||||
|
"steps": [{"name": n, "number": i, "status": "completed", "conclusion": c} for i, (n, c) in enumerate(steps)],
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
async def no_sleep(_: float) -> None:
|
||||||
|
pass
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
async def ci() -> Any:
|
||||||
|
c = GiteaCI(CISettings(token="secret", api_url=API, repo="HSE/up", max_attempts=3), sleep=no_sleep) # type: ignore[arg-type]
|
||||||
|
yield c
|
||||||
|
await c.aclose()
|
||||||
|
|
||||||
|
|
||||||
|
@respx.mock
|
||||||
|
async def test_finds_newest_run_of_the_workflow(ci: GiteaCI) -> None:
|
||||||
|
other = run(9, path="other.yml@refs/heads/main")
|
||||||
|
route = respx.get(RUNS).respond(
|
||||||
|
json={"workflow_runs": [run(5), other, run(7, status="in_progress", conclusion=None)]}
|
||||||
|
)
|
||||||
|
|
||||||
|
found = await ci.find(SHA)
|
||||||
|
|
||||||
|
assert found == CIRun("7", PipelineStatus.RUNNING, f"{API}/HSE/up/actions/runs/7")
|
||||||
|
assert route.calls.last.request.url.params["head_sha"] == SHA
|
||||||
|
assert route.calls.last.request.headers["Authorization"] == "token secret"
|
||||||
|
|
||||||
|
|
||||||
|
@respx.mock
|
||||||
|
async def test_no_run_yet(ci: GiteaCI) -> None:
|
||||||
|
respx.get(RUNS).respond(json={"workflow_runs": [], "total_count": 0})
|
||||||
|
|
||||||
|
assert await ci.find(SHA) is None
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.parametrize(
|
||||||
|
("status", "conclusion", "expected"),
|
||||||
|
[
|
||||||
|
("queued", None, PipelineStatus.PENDING),
|
||||||
|
("waiting", None, PipelineStatus.PENDING),
|
||||||
|
("in_progress", None, PipelineStatus.RUNNING),
|
||||||
|
("completed", "success", PipelineStatus.SUCCESS),
|
||||||
|
("completed", "skipped", PipelineStatus.SKIPPED),
|
||||||
|
],
|
||||||
|
)
|
||||||
|
@respx.mock
|
||||||
|
async def test_status_mapping(ci: GiteaCI, status: str, conclusion: str | None, expected: PipelineStatus) -> None:
|
||||||
|
respx.get(RUNS).respond(json={"workflow_runs": [run(1, status=status, conclusion=conclusion)]})
|
||||||
|
|
||||||
|
found = await ci.find(SHA)
|
||||||
|
|
||||||
|
assert found is not None and found.status is expected
|
||||||
|
|
||||||
|
|
||||||
|
@respx.mock
|
||||||
|
async def test_failure_names_the_step_and_reads_its_log(ci: GiteaCI) -> None:
|
||||||
|
respx.get(RUNS).respond(json={"workflow_runs": [run(3, conclusion="failure")]})
|
||||||
|
steps = [("Run actions/checkout@v4", "success"), ("lint", "failure"), ("tests", "skipped")]
|
||||||
|
respx.get(f"{RUNS}/3/jobs").respond(json={"jobs": [job(31, "failure", steps)], "total_count": 1})
|
||||||
|
log = "2026-10-18T20:44:56.7721431Z ruff check .\n2026-10-18T20:44:57.1Z F821 undefined name 'x'\n"
|
||||||
|
respx.get(f"{API}/api/v1/repos/HSE/up/actions/jobs/31/logs").respond(text=log)
|
||||||
|
|
||||||
|
found = await ci.find(SHA)
|
||||||
|
assert found is not None
|
||||||
|
text = await ci.failure_log(found)
|
||||||
|
|
||||||
|
assert (found.status, found.failed_step) == (PipelineStatus.FAILED, "check / lint")
|
||||||
|
assert text == "ruff check .\nF821 undefined name 'x'\n"
|
||||||
|
|
||||||
|
|
||||||
|
@respx.mock
|
||||||
|
async def test_cancelled_run_is_a_failure(ci: GiteaCI) -> None:
|
||||||
|
respx.get(RUNS).respond(json={"workflow_runs": [run(4, conclusion="cancelled")]})
|
||||||
|
respx.get(f"{RUNS}/4/jobs").respond(json={"jobs": [job(41, "cancelled", [("lint", "success")])]})
|
||||||
|
|
||||||
|
found = await ci.find(SHA)
|
||||||
|
|
||||||
|
assert found is not None and (found.status, found.failed_step) == (PipelineStatus.FAILED, "check / cancelled")
|
||||||
|
|
||||||
|
|
||||||
|
@respx.mock
|
||||||
|
async def test_retries_transport_errors_and_5xx(ci: GiteaCI) -> None:
|
||||||
|
route = respx.get(RUNS)
|
||||||
|
route.side_effect = [
|
||||||
|
httpx.ConnectError("reset"),
|
||||||
|
httpx.Response(502),
|
||||||
|
httpx.Response(200, json={"workflow_runs": []}),
|
||||||
|
]
|
||||||
|
|
||||||
|
assert await ci.find(SHA) is None
|
||||||
|
assert route.call_count == 3
|
||||||
|
|
||||||
|
|
||||||
|
@respx.mock
|
||||||
|
async def test_gives_up_with_a_typed_error(ci: GiteaCI) -> None:
|
||||||
|
respx.get(RUNS).respond(404, json={"message": "not found"})
|
||||||
|
|
||||||
|
with pytest.raises(GiteaError, match="404"):
|
||||||
|
await ci.find(SHA)
|
||||||
@@ -0,0 +1,337 @@
|
|||||||
|
"""PipelineWatcher with a scripted CI, a fake deployer, an offline Telegram and a hand-turned clock."""
|
||||||
|
|
||||||
|
import asyncio
|
||||||
|
from collections.abc import Callable
|
||||||
|
from datetime import datetime, timedelta
|
||||||
|
from typing import Any
|
||||||
|
|
||||||
|
import pytest
|
||||||
|
from aiogram import Bot
|
||||||
|
from aiogram.methods import EditMessageText, SendDocument, SendMessage
|
||||||
|
from sqlalchemy import select
|
||||||
|
|
||||||
|
from app.access.service import AccessService
|
||||||
|
from app.db.models import PipelineRun, PipelineStatus, Push, Stage, utcnow
|
||||||
|
from app.db.session import SessionMaker
|
||||||
|
from app.pipeline.gitea import GiteaError
|
||||||
|
from app.pipeline.provider import CIRun
|
||||||
|
from app.pipeline.watcher import PipelineWatcher, initial_status
|
||||||
|
from app.runner.client import RunnerResult
|
||||||
|
from app.runner.deployer import Deployer, Outcome, Report
|
||||||
|
from app.texts import Texts
|
||||||
|
from tests.conftest import ADMIN
|
||||||
|
from tests.fakes import BOT_TOKEN, FakeSession
|
||||||
|
|
||||||
|
S = PipelineStatus
|
||||||
|
URL = "https://ci.example/runs/1"
|
||||||
|
|
||||||
|
|
||||||
|
class FakeCI:
|
||||||
|
def __init__(self) -> None:
|
||||||
|
self.runs: dict[str, CIRun | None] = {}
|
||||||
|
self.log = "ruff check .\nF821 undefined name 'x'"
|
||||||
|
self.errors = 0 # the next N calls fail like a flaky API
|
||||||
|
|
||||||
|
def set(self, sha: str, status: PipelineStatus, step: str | None = None) -> None:
|
||||||
|
self.runs[sha] = CIRun("1", status, URL, step)
|
||||||
|
|
||||||
|
async def find(self, sha: str) -> CIRun | None:
|
||||||
|
if self.errors:
|
||||||
|
self.errors -= 1
|
||||||
|
raise GiteaError("HTTP 502")
|
||||||
|
return self.runs.get(sha)
|
||||||
|
|
||||||
|
async def failure_log(self, run: CIRun) -> str:
|
||||||
|
return self.log
|
||||||
|
|
||||||
|
async def aclose(self) -> None:
|
||||||
|
pass
|
||||||
|
|
||||||
|
|
||||||
|
class FakeDeployer(Deployer):
|
||||||
|
def __init__(self, fail: frozenset[str] = frozenset()) -> None: # no repo, no runner
|
||||||
|
self.calls: list[tuple[str, list[str]]] = []
|
||||||
|
self.fail = fail
|
||||||
|
|
||||||
|
async def deploy(self, sha: str, services: list[str], report: Report) -> None:
|
||||||
|
self.calls.append((sha, services))
|
||||||
|
for svc in services:
|
||||||
|
ok = svc not in self.fail
|
||||||
|
await report(Outcome(svc, RunnerResult(ok, 0 if ok else 3, "ok" if ok else "not healthy", "boom\nFAIL")))
|
||||||
|
|
||||||
|
|
||||||
|
class Clock:
|
||||||
|
def __init__(self) -> None:
|
||||||
|
self.now = utcnow()
|
||||||
|
|
||||||
|
def __call__(self) -> datetime:
|
||||||
|
return self.now
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
def tg() -> tuple[Bot, FakeSession]:
|
||||||
|
session = FakeSession()
|
||||||
|
return Bot(BOT_TOKEN, session=session), session
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
def ci() -> FakeCI:
|
||||||
|
return FakeCI()
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
def clock() -> Clock:
|
||||||
|
return Clock()
|
||||||
|
|
||||||
|
|
||||||
|
Make = Callable[..., PipelineWatcher]
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
def make(sessions: SessionMaker, texts: Texts, tg: tuple[Bot, FakeSession], ci: FakeCI, clock: Clock) -> Make:
|
||||||
|
def build(deployer: Deployer | None = None, with_ci: bool = True) -> PipelineWatcher:
|
||||||
|
return PipelineWatcher(
|
||||||
|
sessions, tg[0], texts, ci if with_ci else None, deployer, poll_interval=0.01, clock=clock
|
||||||
|
)
|
||||||
|
|
||||||
|
return build
|
||||||
|
|
||||||
|
|
||||||
|
async def add_push(
|
||||||
|
sessions: SessionMaker, sha: str, paths: list[str], ci: PipelineStatus = S.PENDING, cd: PipelineStatus = S.PENDING
|
||||||
|
) -> Push:
|
||||||
|
push = Push(
|
||||||
|
user_id=ADMIN, sha=sha, message="feat: x", paths=paths, chat_id=ADMIN, message_id=7, ci_status=ci, cd_status=cd
|
||||||
|
)
|
||||||
|
async with sessions.begin() as s:
|
||||||
|
s.add(push)
|
||||||
|
return push
|
||||||
|
|
||||||
|
|
||||||
|
async def state(sessions: SessionMaker, sha: str) -> tuple[PipelineStatus, PipelineStatus]:
|
||||||
|
async with sessions() as s:
|
||||||
|
push = await s.scalar(select(Push).where(Push.sha == sha))
|
||||||
|
assert push is not None
|
||||||
|
return push.ci_status, push.cd_status
|
||||||
|
|
||||||
|
|
||||||
|
async def settle(watcher: PipelineWatcher, sessions: SessionMaker, sha: str) -> tuple[PipelineStatus, PipelineStatus]:
|
||||||
|
"""CD runs in a background task. Wait for it, not for the DB: in tests SQLite is one shared connection, and a
|
||||||
|
read racing the watcher's write could roll that write back."""
|
||||||
|
async with asyncio.timeout(5):
|
||||||
|
await watcher.drain()
|
||||||
|
return await state(sessions, sha)
|
||||||
|
|
||||||
|
|
||||||
|
def texts_of(session: FakeSession, method: Any) -> list[str]:
|
||||||
|
return [m.text if hasattr(m, "text") else m.caption for m in session.sent(method)]
|
||||||
|
|
||||||
|
|
||||||
|
SHA_A = "a" * 40
|
||||||
|
SHA_B = "b" * 40
|
||||||
|
HELLO = ["services/hello/main.py"]
|
||||||
|
|
||||||
|
|
||||||
|
def test_initial_status() -> None:
|
||||||
|
assert initial_status(True, True, HELLO) == (S.PENDING, S.PENDING)
|
||||||
|
assert initial_status(True, True, ["README.md"]) == (S.PENDING, S.SKIPPED) # no service touched
|
||||||
|
assert initial_status(False, True, HELLO) == (S.SKIPPED, S.PENDING)
|
||||||
|
assert initial_status(True, False, HELLO) == (S.PENDING, S.SKIPPED)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_ci_green_then_deploy(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make, ci: FakeCI, tg: tuple[Bot, FakeSession], texts: Texts
|
||||||
|
) -> None:
|
||||||
|
t, session = texts.pipeline, tg[1]
|
||||||
|
deployer = FakeDeployer()
|
||||||
|
watcher = make(deployer)
|
||||||
|
await add_push(sessions, SHA_A, HELLO)
|
||||||
|
|
||||||
|
assert await watcher.tick() is True # no run yet: nothing to say, keep polling
|
||||||
|
assert session.requests == []
|
||||||
|
|
||||||
|
ci.set(SHA_A, S.RUNNING)
|
||||||
|
await watcher.tick()
|
||||||
|
assert texts_of(session, EditMessageText)[-1].endswith(
|
||||||
|
f"{t.ci_running}\n{URL}\n{t.cd_waiting.format(services='hello')}"
|
||||||
|
)
|
||||||
|
await watcher.tick() # same status — no edit
|
||||||
|
assert len(session.sent(EditMessageText)) == 1
|
||||||
|
|
||||||
|
ci.set(SHA_A, S.SUCCESS)
|
||||||
|
assert await watcher.tick() is False
|
||||||
|
assert await settle(watcher, sessions, SHA_A) == (S.SUCCESS, S.SUCCESS)
|
||||||
|
|
||||||
|
assert deployer.calls == [(SHA_A, ["hello"])]
|
||||||
|
edits = texts_of(session, EditMessageText)
|
||||||
|
assert edits[-1].endswith(f"{t.ci_success}\n{URL}\n{t.cd_success.format(services='hello')}")
|
||||||
|
assert any(t.cd_running.format(services="hello") in e for e in edits)
|
||||||
|
sent = texts_of(session, SendMessage)
|
||||||
|
assert sent == [
|
||||||
|
t.notify_ci_ok.format(sha="aaaaaaa", url=URL),
|
||||||
|
t.notify_cd_ok.format(services="hello", sha="aaaaaaa"),
|
||||||
|
]
|
||||||
|
assert all(m.reply_parameters.message_id == 7 for m in session.sent(SendMessage))
|
||||||
|
assert await watcher.tick() is False # nothing left: the loop would sleep until wake()
|
||||||
|
|
||||||
|
|
||||||
|
async def test_ci_red_reports_step_and_log_and_skips_deploy(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make, ci: FakeCI, tg: tuple[Bot, FakeSession], texts: Texts
|
||||||
|
) -> None:
|
||||||
|
t, session = texts.pipeline, tg[1]
|
||||||
|
deployer = FakeDeployer()
|
||||||
|
watcher = make(deployer)
|
||||||
|
await add_push(sessions, SHA_A, HELLO)
|
||||||
|
ci.set(SHA_A, S.FAILED, "check / lint")
|
||||||
|
|
||||||
|
await watcher.tick()
|
||||||
|
|
||||||
|
assert await state(sessions, SHA_A) == (S.FAILED, S.SKIPPED)
|
||||||
|
assert deployer.calls == []
|
||||||
|
assert texts_of(session, EditMessageText)[-1].endswith(
|
||||||
|
f"{t.ci_failed.format(step='check / lint')}\n{URL}\n{t.cd_not_run}"
|
||||||
|
)
|
||||||
|
[note] = texts_of(session, SendMessage)
|
||||||
|
assert note.startswith("❌ CI упал на шаге <b>check / lint</b> · aaaaaaa")
|
||||||
|
assert "<pre>ruff check .\nF821 undefined name 'x'</pre>" in note
|
||||||
|
assert session.sent(SendDocument) == [] # short log: inline only
|
||||||
|
async with sessions() as s:
|
||||||
|
row = await s.scalar(select(PipelineRun).where(PipelineRun.stage == Stage.CI))
|
||||||
|
assert row is not None and (row.run_slug, row.status, row.failed_cube, row.url) == (
|
||||||
|
"1",
|
||||||
|
S.FAILED,
|
||||||
|
"check / lint",
|
||||||
|
URL,
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_long_log_also_goes_as_a_file(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make, ci: FakeCI, tg: tuple[Bot, FakeSession]
|
||||||
|
) -> None:
|
||||||
|
ci.log = "\n".join(f"line {i}" for i in range(2000))
|
||||||
|
await add_push(sessions, SHA_A, HELLO)
|
||||||
|
ci.set(SHA_A, S.FAILED, "check / tests")
|
||||||
|
|
||||||
|
await make(FakeDeployer()).tick()
|
||||||
|
|
||||||
|
[note] = texts_of(tg[1], SendMessage)
|
||||||
|
assert "line 1999" in note and "line 0\n" not in note and len(note) < 4096
|
||||||
|
[doc] = tg[1].sent(SendDocument)
|
||||||
|
assert doc.document.filename == "ci-aaaaaaa.log"
|
||||||
|
|
||||||
|
|
||||||
|
async def test_no_run_for_too_long_times_out(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make, clock: Clock, tg: tuple[Bot, FakeSession], texts: Texts
|
||||||
|
) -> None:
|
||||||
|
deployer = FakeDeployer()
|
||||||
|
watcher = make(deployer)
|
||||||
|
await add_push(sessions, SHA_A, HELLO)
|
||||||
|
clock.now += timedelta(minutes=29)
|
||||||
|
await watcher.tick()
|
||||||
|
assert await state(sessions, SHA_A) == (S.PENDING, S.PENDING)
|
||||||
|
|
||||||
|
clock.now += timedelta(minutes=2)
|
||||||
|
await watcher.tick()
|
||||||
|
|
||||||
|
assert await state(sessions, SHA_A) == (S.TIMEOUT, S.SKIPPED)
|
||||||
|
assert deployer.calls == []
|
||||||
|
assert texts_of(tg[1], SendMessage) == [texts.pipeline.notify_ci_timeout.format(sha="aaaaaaa", minutes=30)]
|
||||||
|
|
||||||
|
|
||||||
|
async def test_failed_deploy(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make, ci: FakeCI, tg: tuple[Bot, FakeSession], texts: Texts
|
||||||
|
) -> None:
|
||||||
|
t = texts.pipeline
|
||||||
|
paths = ["services/hello/main.py", "services/weather/main.py"]
|
||||||
|
await add_push(sessions, SHA_A, paths)
|
||||||
|
ci.set(SHA_A, S.SUCCESS)
|
||||||
|
|
||||||
|
watcher = make(FakeDeployer(fail=frozenset({"weather"})))
|
||||||
|
await watcher.tick()
|
||||||
|
|
||||||
|
assert await settle(watcher, sessions, SHA_A) == (S.SUCCESS, S.FAILED)
|
||||||
|
assert texts_of(tg[1], EditMessageText)[-1].endswith(t.cd_failed.format(services="weather"))
|
||||||
|
sent = texts_of(tg[1], SendMessage)
|
||||||
|
assert sent[1].startswith("⛔ <b>weather</b> не развёрнут: not healthy") and "<pre>boom" in sent[1]
|
||||||
|
assert sent[2] == t.notify_cd_ok.format(services="hello", sha="aaaaaaa")
|
||||||
|
|
||||||
|
|
||||||
|
async def test_newer_push_wins_when_ci_finishes_out_of_order(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make, ci: FakeCI, tg: tuple[Bot, FakeSession], texts: Texts
|
||||||
|
) -> None:
|
||||||
|
deployer = FakeDeployer()
|
||||||
|
watcher = make(deployer)
|
||||||
|
await add_push(sessions, SHA_A, HELLO)
|
||||||
|
await add_push(sessions, SHA_B, [*HELLO, "services/weather/main.py"])
|
||||||
|
ci.set(SHA_B, S.SUCCESS)
|
||||||
|
await watcher.tick()
|
||||||
|
await settle(watcher, sessions, SHA_B)
|
||||||
|
|
||||||
|
ci.set(SHA_A, S.SUCCESS) # the older push turns green later: deploying it would roll hello back
|
||||||
|
await watcher.tick()
|
||||||
|
|
||||||
|
assert await settle(watcher, sessions, SHA_A) == (S.SUCCESS, S.SKIPPED)
|
||||||
|
assert deployer.calls == [(SHA_B, ["hello", "weather"])]
|
||||||
|
assert any(texts.pipeline.cd_superseded in e for e in texts_of(tg[1], EditMessageText))
|
||||||
|
|
||||||
|
|
||||||
|
async def test_restart_picks_up_an_interrupted_deploy(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make
|
||||||
|
) -> None:
|
||||||
|
await add_push(sessions, SHA_A, HELLO, ci=S.SUCCESS, cd=S.RUNNING) # the bot died mid-deploy
|
||||||
|
deployer = FakeDeployer()
|
||||||
|
watcher = make(deployer)
|
||||||
|
|
||||||
|
await watcher.tick()
|
||||||
|
|
||||||
|
assert await settle(watcher, sessions, SHA_A) == (S.SUCCESS, S.SUCCESS)
|
||||||
|
assert deployer.calls == [(SHA_A, ["hello"])]
|
||||||
|
|
||||||
|
|
||||||
|
async def test_without_ci_deploys_right_away(access: AccessService, sessions: SessionMaker, make: Make) -> None:
|
||||||
|
await add_push(sessions, SHA_A, HELLO, ci=S.SKIPPED)
|
||||||
|
deployer = FakeDeployer()
|
||||||
|
|
||||||
|
watcher = make(deployer, with_ci=False)
|
||||||
|
|
||||||
|
assert await watcher.tick() is False
|
||||||
|
assert await settle(watcher, sessions, SHA_A) == (S.SKIPPED, S.SUCCESS)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_ci_switched_off_does_not_leave_pushes_waiting(
|
||||||
|
access: AccessService, sessions: SessionMaker, make: Make
|
||||||
|
) -> None:
|
||||||
|
await add_push(sessions, SHA_A, ["README.md"], cd=S.SKIPPED)
|
||||||
|
|
||||||
|
await make(FakeDeployer(), with_ci=False).tick()
|
||||||
|
|
||||||
|
assert await state(sessions, SHA_A) == (S.SKIPPED, S.SKIPPED)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_loop_survives_api_errors_and_wakes_on_push(
|
||||||
|
access: AccessService,
|
||||||
|
sessions: SessionMaker,
|
||||||
|
make: Make,
|
||||||
|
ci: FakeCI,
|
||||||
|
tg: tuple[Bot, FakeSession],
|
||||||
|
texts: Texts,
|
||||||
|
) -> None:
|
||||||
|
watcher = make(FakeDeployer())
|
||||||
|
done = texts.pipeline.notify_cd_ok.format(services="hello", sha="aaaaaaa")
|
||||||
|
await add_push(sessions, SHA_B, ["README.md"], ci=S.SKIPPED, cd=S.SKIPPED) # finished: nothing to watch
|
||||||
|
loop = asyncio.create_task(watcher.run())
|
||||||
|
try:
|
||||||
|
await asyncio.sleep(0.05)
|
||||||
|
assert tg[1].requests == [] # idle until woken
|
||||||
|
ci.errors = 2
|
||||||
|
ci.set(SHA_A, S.SUCCESS)
|
||||||
|
await add_push(sessions, SHA_A, HELLO)
|
||||||
|
watcher.wake()
|
||||||
|
async with asyncio.timeout(5):
|
||||||
|
while done not in texts_of(tg[1], SendMessage): # noqa: ASYNC110 - fake Telegram, no event
|
||||||
|
await asyncio.sleep(0.01)
|
||||||
|
await watcher.drain()
|
||||||
|
assert ci.errors == 0
|
||||||
|
finally:
|
||||||
|
loop.cancel()
|
||||||
|
assert await state(sessions, SHA_A) == (S.SUCCESS, S.SUCCESS)
|
||||||
@@ -5,7 +5,7 @@ import pytest
|
|||||||
from sqlalchemy import select
|
from sqlalchemy import select
|
||||||
|
|
||||||
from app.access.service import AccessService
|
from app.access.service import AccessService
|
||||||
from app.db.models import ChangeOp, Push, User, utcnow
|
from app.db.models import ChangeOp, PipelineStatus, Push, User, utcnow
|
||||||
from app.db.session import SessionMaker
|
from app.db.session import SessionMaker
|
||||||
from app.repo.drafts import Change, DraftView
|
from app.repo.drafts import Change, DraftView
|
||||||
from app.repo.git import UploadsRepo
|
from app.repo.git import UploadsRepo
|
||||||
@@ -129,11 +129,13 @@ async def test_rate_limit_and_record(repo: UploadsRepo, sessions: SessionMaker,
|
|||||||
publisher = Publisher(repo, sessions)
|
publisher = Publisher(repo, sessions)
|
||||||
draft = await draft_on(repo, Change("x.txt", ChangeOp.ADD, b"x"))
|
draft = await draft_on(repo, Change("x.txt", ChangeOp.ADD, b"x"))
|
||||||
result = await publisher.push(member, draft, "x")
|
result = await publisher.push(member, draft, "x")
|
||||||
await publisher.record(member, result, "x", draft, chat_id=ADMIN, message_id=77)
|
status = (PipelineStatus.PENDING, PipelineStatus.SKIPPED)
|
||||||
|
await publisher.record(member, result, "x", draft, chat_id=ADMIN, message_id=77, status=status)
|
||||||
|
|
||||||
async with sessions() as s:
|
async with sessions() as s:
|
||||||
push = await s.scalar(select(Push))
|
push = await s.scalar(select(Push))
|
||||||
assert push is not None and push.sha == result.sha and push.paths == ["x.txt"] and push.message_id == 77
|
assert push is not None and push.sha == result.sha and push.paths == ["x.txt"] and push.message_id == 77
|
||||||
|
assert (push.ci_status, push.cd_status) == status
|
||||||
for i in range(PUSHES_PER_HOUR):
|
for i in range(PUSHES_PER_HOUR):
|
||||||
s.add(
|
s.add(
|
||||||
Push(user_id=ADMIN, sha=f"{i:040d}", message="m", chat_id=1, created_at=utcnow() - timedelta(minutes=1))
|
Push(user_id=ADMIN, sha=f"{i:040d}", message="m", chat_id=1, created_at=utcnow() - timedelta(minutes=1))
|
||||||
|
|||||||
@@ -27,6 +27,7 @@ def test_bad_names(name: str) -> None:
|
|||||||
("path", "protected"),
|
("path", "protected"),
|
||||||
[
|
[
|
||||||
(".sourcecraft/ci.yaml", True),
|
(".sourcecraft/ci.yaml", True),
|
||||||
|
(".gitea/workflows/check.yml", True),
|
||||||
(".tgit/x.py", True),
|
(".tgit/x.py", True),
|
||||||
("ruff.toml", True),
|
("ruff.toml", True),
|
||||||
(".sourcecraft", True),
|
(".sourcecraft", True),
|
||||||
|
|||||||
Executable
+83
@@ -0,0 +1,83 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""A fake `docker` CLI for tgit-runner tests: records every call, keeps containers in a JSON file.
|
||||||
|
|
||||||
|
Behaviour knobs (env): FAKE_HEALTH (healthy|unhealthy|none), FAKE_STATUS (running|exited), FAKE_RESTARTS,
|
||||||
|
FAKE_BUILD_FAIL=1. State: $FAKE_DOCKER_STATE (json), calls: $FAKE_DOCKER_LOG (one json array per line).
|
||||||
|
"""
|
||||||
|
|
||||||
|
import json
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
|
||||||
|
state_path = os.environ["FAKE_DOCKER_STATE"]
|
||||||
|
args = sys.argv[1:]
|
||||||
|
with open(os.environ["FAKE_DOCKER_LOG"], "a") as log:
|
||||||
|
log.write(json.dumps(args) + "\n")
|
||||||
|
try:
|
||||||
|
with open(state_path) as f:
|
||||||
|
state = json.load(f)
|
||||||
|
except FileNotFoundError:
|
||||||
|
state = {"containers": {}, "images": []}
|
||||||
|
|
||||||
|
|
||||||
|
def save() -> None:
|
||||||
|
with open(state_path, "w") as f:
|
||||||
|
json.dump(state, f)
|
||||||
|
|
||||||
|
|
||||||
|
def done(code: int = 0, out: str = "") -> None:
|
||||||
|
if out:
|
||||||
|
print(out)
|
||||||
|
save()
|
||||||
|
sys.exit(code)
|
||||||
|
|
||||||
|
|
||||||
|
cmd = args[0] if args else ""
|
||||||
|
if cmd == "container" and args[1] == "inspect":
|
||||||
|
done(0 if args[2] in state["containers"] else 1)
|
||||||
|
if cmd == "inspect":
|
||||||
|
fmt, name = args[2], args[3]
|
||||||
|
if name not in state["containers"]:
|
||||||
|
done(1)
|
||||||
|
if "Health" in fmt:
|
||||||
|
done(0, os.environ.get("FAKE_HEALTH", "healthy"))
|
||||||
|
if "RestartCount" in fmt:
|
||||||
|
done(0, os.environ.get("FAKE_RESTARTS", "0"))
|
||||||
|
done(0, os.environ.get("FAKE_STATUS", "running"))
|
||||||
|
if cmd == "ps":
|
||||||
|
done(0, "\n".join(n for n in state["containers"]))
|
||||||
|
if cmd == "build":
|
||||||
|
if os.environ.get("FAKE_BUILD_FAIL"):
|
||||||
|
done(1, "ERROR: could not install requirements")
|
||||||
|
tag = args[args.index("-t") + 1]
|
||||||
|
ctx = args[-1]
|
||||||
|
with open(os.path.join(ctx, "Dockerfile")) as f:
|
||||||
|
state.setdefault("dockerfiles", {})[tag] = f.read()
|
||||||
|
state["images"].append(tag)
|
||||||
|
done(0, "sha256:feed")
|
||||||
|
if cmd == "run":
|
||||||
|
name = args[args.index("--name") + 1]
|
||||||
|
state["containers"][name] = {"image": args[-1], "args": args}
|
||||||
|
done(0, "cid")
|
||||||
|
if cmd == "rm":
|
||||||
|
for name in args[2:]:
|
||||||
|
state["containers"].pop(name, None)
|
||||||
|
done(0)
|
||||||
|
if cmd == "rename":
|
||||||
|
state["containers"][args[2]] = state["containers"].pop(args[1])
|
||||||
|
done(0)
|
||||||
|
if cmd == "restart":
|
||||||
|
done(0)
|
||||||
|
if cmd == "logs":
|
||||||
|
done(0, "Traceback: boom")
|
||||||
|
if cmd == "image" and args[1] == "ls":
|
||||||
|
repo = args[2]
|
||||||
|
tags = [t.split(":", 1)[1] for t in reversed(state["images"]) if t.split(":", 1)[0] == repo]
|
||||||
|
full = "{{.Repository}}" in args[-1]
|
||||||
|
done(0, "\n".join(f"{repo}:{t}" if full else t for t in tags))
|
||||||
|
if cmd == "image" and args[1] == "rm":
|
||||||
|
state["images"] = [t for t in state["images"] if t != args[2]]
|
||||||
|
done(0)
|
||||||
|
if cmd == "network":
|
||||||
|
done(0)
|
||||||
|
done(2, f"fake docker: unsupported {args}")
|
||||||
Executable
+14
@@ -0,0 +1,14 @@
|
|||||||
|
#!/usr/bin/env python3
|
||||||
|
"""A fake `ssh`: like sshd with a forced command — the remote command becomes SSH_ORIGINAL_COMMAND of tgit-runner."""
|
||||||
|
|
||||||
|
import os
|
||||||
|
import sys
|
||||||
|
|
||||||
|
args = sys.argv[1:]
|
||||||
|
with_value = {"-F", "-p", "-o", "-i"}
|
||||||
|
i = 0
|
||||||
|
while i < len(args) and args[i].startswith("-"):
|
||||||
|
i += 2 if args[i] in with_value else 1
|
||||||
|
remote = " ".join(args[i + 1 :])
|
||||||
|
os.environ["SSH_ORIGINAL_COMMAND"] = remote
|
||||||
|
os.execvp("bash", ["bash", os.environ["FAKE_RUNNER"]])
|
||||||
@@ -0,0 +1,247 @@
|
|||||||
|
"""Bot side of #13: the SSH client, the deployer and the push → deploy flow, down to the real tgit-runner script."""
|
||||||
|
|
||||||
|
import asyncio
|
||||||
|
import base64
|
||||||
|
import json
|
||||||
|
import subprocess
|
||||||
|
from collections.abc import AsyncIterator, Callable
|
||||||
|
from dataclasses import replace
|
||||||
|
from datetime import UTC, datetime
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
import pytest
|
||||||
|
from aiogram import Bot, Dispatcher
|
||||||
|
from aiogram.methods import AnswerCallbackQuery, EditMessageText, SendMessage
|
||||||
|
from aiogram.types import InlineKeyboardMarkup
|
||||||
|
|
||||||
|
from app.access.service import AccessService
|
||||||
|
from app.alerts.broadcast import TelegramNotifier
|
||||||
|
from app.alerts.detectors import load_alerts_config
|
||||||
|
from app.alerts.service import AlertService
|
||||||
|
from app.bot.routers.commit import Act, CommitCb
|
||||||
|
from app.bot.routers.status import StatusAct, StatusCb, render_card, runner_service
|
||||||
|
from app.config import RunnerSettings
|
||||||
|
from app.db.models import Alert, AlertKind
|
||||||
|
from app.db.session import SessionMaker
|
||||||
|
from app.main import build_dispatcher
|
||||||
|
from app.pipeline.watcher import PipelineWatcher
|
||||||
|
from app.repo.git import UploadsRepo
|
||||||
|
from app.runner.client import RunnerClient, RunnerResult
|
||||||
|
from app.runner.deployer import Deployer, Outcome, changed_services
|
||||||
|
from app.texts import Texts
|
||||||
|
from tests.alerts.test_detectors import RUNNING
|
||||||
|
from tests.conftest import ADMIN, ROOT
|
||||||
|
from tests.fakes import TgClient
|
||||||
|
from tests.runner.test_tgit_runner import FAKE, RUNNER
|
||||||
|
|
||||||
|
HERE = Path(__file__).parent
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
def fake_host(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> Path:
|
||||||
|
"""ssh → tgit-runner → fake docker, all on PATH."""
|
||||||
|
bin_dir = tmp_path / "bin"
|
||||||
|
bin_dir.mkdir()
|
||||||
|
(bin_dir / "ssh").symlink_to(HERE / "fake_ssh.py")
|
||||||
|
(bin_dir / "docker").symlink_to(FAKE)
|
||||||
|
monkeypatch.setenv("PATH", f"{bin_dir}:/usr/bin:/bin")
|
||||||
|
monkeypatch.setenv("FAKE_RUNNER", str(RUNNER))
|
||||||
|
monkeypatch.setenv("FAKE_DOCKER_STATE", str(tmp_path / "state.json"))
|
||||||
|
monkeypatch.setenv("FAKE_DOCKER_LOG", str(tmp_path / "calls.log"))
|
||||||
|
monkeypatch.setenv("TGIT_RUNNER_STATE", str(tmp_path / "runner"))
|
||||||
|
monkeypatch.setenv("TGIT_HEALTH_TIMEOUT", "2")
|
||||||
|
monkeypatch.setenv("TGIT_SETTLE", "0")
|
||||||
|
return tmp_path
|
||||||
|
|
||||||
|
|
||||||
|
def docker_state(host: Path) -> dict[str, object]:
|
||||||
|
return json.loads((host / "state.json").read_text())
|
||||||
|
|
||||||
|
|
||||||
|
def client(key: bytes | None = None) -> RunnerClient:
|
||||||
|
b64 = base64.b64encode(key).decode() if key else None
|
||||||
|
return RunnerClient(RunnerSettings(host="host.example", ssh_key_b64=b64, timeout=30)) # type: ignore[arg-type]
|
||||||
|
|
||||||
|
|
||||||
|
def test_changed_services() -> None:
|
||||||
|
paths = ["services/hello/main.py", "services/hello/x/y.py", "services/Bad/main.py", "README.md", "services/a.py"]
|
||||||
|
assert changed_services(paths) == ["hello"]
|
||||||
|
|
||||||
|
|
||||||
|
async def test_client_validates_before_ssh(fake_host: Path) -> None:
|
||||||
|
result = await client().deploy("../etc", "abc1234", b"")
|
||||||
|
|
||||||
|
assert (result.ok, result.code) == (False, 2)
|
||||||
|
assert not (fake_host / "calls.log").exists()
|
||||||
|
|
||||||
|
|
||||||
|
def archive_of(remote: Path) -> bytes:
|
||||||
|
return subprocess.run(
|
||||||
|
["git", "archive", "--format=tar", "HEAD:services/hello"], cwd=remote, capture_output=True, check=True
|
||||||
|
).stdout
|
||||||
|
|
||||||
|
|
||||||
|
async def until(condition: Callable[[], bool]) -> None:
|
||||||
|
"""The deploy runs in a background task: wait for its visible effect."""
|
||||||
|
async with asyncio.timeout(15):
|
||||||
|
while not condition(): # noqa: ASYNC110 - polling a fake Telegram session, no event to wait on
|
||||||
|
await asyncio.sleep(0.05)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_client_through_forced_command(fake_host: Path, remote: Path) -> None:
|
||||||
|
c = client(b"-----BEGIN KEY-----")
|
||||||
|
archive = archive_of(remote)
|
||||||
|
|
||||||
|
ok = await c.deploy("hello", "abc1234", archive)
|
||||||
|
restart = await c.restart("hello")
|
||||||
|
bad = await c.run("deploy", "hello;", "rm")
|
||||||
|
|
||||||
|
assert ok.ok and ok.summary.startswith("deployed hello abc1234")
|
||||||
|
assert restart.ok
|
||||||
|
assert (bad.ok, bad.code) == (False, 2)
|
||||||
|
assert "tgit-svc-hello" in docker_state(fake_host)["containers"] # type: ignore[operator]
|
||||||
|
|
||||||
|
|
||||||
|
class Recorder:
|
||||||
|
def __init__(self) -> None:
|
||||||
|
self.outcomes: list[Outcome] = []
|
||||||
|
|
||||||
|
async def __call__(self, o: Outcome) -> None:
|
||||||
|
self.outcomes.append(o)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_deployer_deploys_and_removes(fake_host: Path, repo: UploadsRepo, remote: Path, tmp_path: Path) -> None:
|
||||||
|
await repo.ensure()
|
||||||
|
sha = await repo.head()
|
||||||
|
rec = Recorder()
|
||||||
|
deployer = Deployer(repo, client())
|
||||||
|
|
||||||
|
await deployer.deploy(sha, ["hello", "gone"], rec)
|
||||||
|
|
||||||
|
hello, gone = rec.outcomes
|
||||||
|
assert hello.result.ok and not hello.removed
|
||||||
|
assert gone.removed and gone.result.ok
|
||||||
|
calls = [json.loads(line) for line in (fake_host / "calls.log").read_text().splitlines()]
|
||||||
|
assert any(c[0] == "run" for c in calls)
|
||||||
|
|
||||||
|
|
||||||
|
async def test_refused_deploy_raises_alert(
|
||||||
|
fake_host: Path, repo: UploadsRepo, sessions: SessionMaker, monkeypatch: pytest.MonkeyPatch
|
||||||
|
) -> None:
|
||||||
|
await repo.ensure()
|
||||||
|
monkeypatch.setenv("FAKE_BUILD_FAIL", "1")
|
||||||
|
events: list[Alert] = []
|
||||||
|
|
||||||
|
class Notes:
|
||||||
|
async def opened(self, alert: Alert, admins_only: bool) -> None:
|
||||||
|
events.append(alert)
|
||||||
|
|
||||||
|
async def repeated(self, alert: Alert) -> None: ...
|
||||||
|
|
||||||
|
async def resolved(self, alert: Alert) -> None: ...
|
||||||
|
|
||||||
|
alerts = AlertService(sessions, Notes(), load_alerts_config(ROOT / "config"))
|
||||||
|
rec = Recorder()
|
||||||
|
await Deployer(repo, client(), alerts).deploy(await repo.head(), ["hello"], rec)
|
||||||
|
|
||||||
|
[o] = rec.outcomes
|
||||||
|
assert (o.result.ok, o.result.code, o.result.summary) == (False, 5, "build failed")
|
||||||
|
[alert] = events
|
||||||
|
assert alert.kind is AlertKind.DEPLOY_REFUSED and alert.service == "tgit-svc-hello"
|
||||||
|
assert "could not install" in alert.payload["log"]
|
||||||
|
|
||||||
|
|
||||||
|
async def test_one_broken_service_does_not_stop_others(repo: UploadsRepo) -> None:
|
||||||
|
class Flaky(RunnerClient):
|
||||||
|
async def run(self, *args: str, stdin: bytes = b"") -> RunnerResult:
|
||||||
|
if args[1] == "aaa":
|
||||||
|
raise RuntimeError("ssh exploded")
|
||||||
|
return RunnerResult(True, 0, f"deployed {args[1]}", "OK")
|
||||||
|
|
||||||
|
await repo.ensure()
|
||||||
|
rec = Recorder()
|
||||||
|
await Deployer(repo, Flaky(RunnerSettings(host="h"))).deploy(await repo.head(), ["aaa", "hello"], rec)
|
||||||
|
|
||||||
|
assert [(o.service, o.result.ok) for o in rec.outcomes] == [("aaa", False), ("hello", True)]
|
||||||
|
|
||||||
|
|
||||||
|
# ---- in the bot -------------------------------------------------------------------------------------------------
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
async def tg(
|
||||||
|
texts: Texts, sessions: SessionMaker, access: AccessService, repo: UploadsRepo, fake_host: Path
|
||||||
|
) -> AsyncIterator[TgClient]:
|
||||||
|
loops: list[asyncio.Task[None]] = []
|
||||||
|
|
||||||
|
def dispatcher(bot: Bot) -> Dispatcher:
|
||||||
|
alerts = AlertService(
|
||||||
|
sessions, TelegramNotifier(bot, access, sessions, texts), load_alerts_config(ROOT / "config")
|
||||||
|
)
|
||||||
|
# no CI configured: the watcher deploys right after push
|
||||||
|
watcher = PipelineWatcher(sessions, bot, texts, None, Deployer(repo, client(), alerts), poll_interval=0.05)
|
||||||
|
loops.append(asyncio.create_task(watcher.run()))
|
||||||
|
return build_dispatcher(texts, sessions, access, repo, alerts=alerts, runner=client(), pipeline=watcher)
|
||||||
|
|
||||||
|
yield TgClient(dispatcher)
|
||||||
|
for loop in loops:
|
||||||
|
loop.cancel()
|
||||||
|
|
||||||
|
|
||||||
|
async def test_push_deploys_touched_service(tg: TgClient, texts: Texts, fake_host: Path, remote: Path) -> None:
|
||||||
|
await tg.send(ADMIN, texts.menu.commit)
|
||||||
|
await tg.press(ADMIN, CommitCb(a=Act.OPEN, i=2).pack()) # services
|
||||||
|
await tg.press(ADMIN, CommitCb(a=Act.OPEN, i=0).pack()) # services/hello
|
||||||
|
await tg.press(ADMIN, CommitCb(a=Act.NEW_FILE).pack())
|
||||||
|
await tg.send(ADMIN, "util.py")
|
||||||
|
await tg.send(ADMIN, "X = 1\n")
|
||||||
|
await tg.press(ADMIN, CommitCb(a=Act.MESSAGE).pack())
|
||||||
|
await tg.send(ADMIN, "add util")
|
||||||
|
await tg.press(ADMIN, CommitCb(a=Act.PUSH).pack())
|
||||||
|
await tg.press(ADMIN, CommitCb(a=Act.PUSH_GO).pack())
|
||||||
|
await until(lambda: any("развёрнуты" in (m.text or "") for m in tg.session.sent(SendMessage)))
|
||||||
|
|
||||||
|
pushed = [e.text for e in tg.session.sent(EditMessageText) if "Запушено" in (e.text or "")]
|
||||||
|
assert "CI" not in pushed[0] and pushed[0].endswith(texts.pipeline.cd_running.format(services="hello"))
|
||||||
|
await until(
|
||||||
|
lambda: tg.session.sent(EditMessageText)[-1].text.endswith(texts.pipeline.cd_success.format(services="hello"))
|
||||||
|
)
|
||||||
|
[done] = [m for m in tg.session.sent(SendMessage) if "развёрнуты" in (m.text or "")]
|
||||||
|
assert done.text.startswith("🚀 CD: сервисы hello развёрнуты, ревизия ")
|
||||||
|
assert "tgit-svc-hello" in docker_state(fake_host)["containers"] # type: ignore[operator]
|
||||||
|
|
||||||
|
|
||||||
|
def buttons(markup: InlineKeyboardMarkup) -> list[str]:
|
||||||
|
return [b.text for row in markup.inline_keyboard for b in row]
|
||||||
|
|
||||||
|
|
||||||
|
def test_restart_button_only_for_runner_services(texts: Texts) -> None:
|
||||||
|
service = replace(RUNNING, labels={"tgit.service": "hello"})
|
||||||
|
now = datetime.now(UTC)
|
||||||
|
|
||||||
|
assert texts.status.restart in buttons(render_card(texts, service, None, now, can_restart=True)[1])
|
||||||
|
assert texts.status.restart not in buttons(render_card(texts, service, None, now)[1]) # not an admin
|
||||||
|
assert texts.status.restart not in buttons(render_card(texts, RUNNING, None, now, can_restart=True)[1]) # the bot
|
||||||
|
assert runner_service(service) == "hello" and runner_service(RUNNING) is None
|
||||||
|
|
||||||
|
|
||||||
|
async def test_restart_through_status(
|
||||||
|
texts: Texts, sessions: SessionMaker, access: AccessService, repo: UploadsRepo, monkeypatch: pytest.MonkeyPatch
|
||||||
|
) -> None:
|
||||||
|
from tests.runtime.fake_docker import FakeDocker
|
||||||
|
|
||||||
|
fake = FakeDocker()
|
||||||
|
svc = fake.add("tgit-svc-hello", labels={"tgit.service": "hello"})
|
||||||
|
calls: list[tuple[str, ...]] = []
|
||||||
|
|
||||||
|
async def run(self: RunnerClient, *args: str, stdin: bytes = b"") -> RunnerResult:
|
||||||
|
calls.append(args)
|
||||||
|
return RunnerResult(True, 0, "restarted hello", "OK restarted hello")
|
||||||
|
|
||||||
|
monkeypatch.setattr(RunnerClient, "run", run)
|
||||||
|
async with fake.client() as docker:
|
||||||
|
tg = TgClient(build_dispatcher(texts, sessions, access, repo, docker=docker, runner=client()))
|
||||||
|
await tg.press(ADMIN, StatusCb(a=StatusAct.RESTART, id=svc.id[:12]).pack())
|
||||||
|
|
||||||
|
assert calls == [("restart", "hello")]
|
||||||
|
assert tg.session.sent(AnswerCallbackQuery)[0].text == texts.status.restarted
|
||||||
@@ -0,0 +1,250 @@
|
|||||||
|
"""tgit-runner against a fake docker: argument validation, archive checks, sandbox flags, rollback."""
|
||||||
|
|
||||||
|
import io
|
||||||
|
import json
|
||||||
|
import shutil
|
||||||
|
import subprocess
|
||||||
|
import tarfile
|
||||||
|
from collections.abc import Callable
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
import pytest
|
||||||
|
|
||||||
|
from tests.conftest import ROOT
|
||||||
|
|
||||||
|
RUNNER = ROOT / "infra" / "runner" / "tgit-runner"
|
||||||
|
FAKE = Path(__file__).with_name("fake_docker.py")
|
||||||
|
|
||||||
|
pytestmark = pytest.mark.skipif(shutil.which("bash") is None, reason="needs bash")
|
||||||
|
|
||||||
|
|
||||||
|
def tar_of(files: dict[str, str | bytes], extra: Callable[[tarfile.TarFile], None] | None = None) -> bytes:
|
||||||
|
buf = io.BytesIO()
|
||||||
|
with tarfile.open(fileobj=buf, mode="w") as tar:
|
||||||
|
for name, content in files.items():
|
||||||
|
data = content.encode() if isinstance(content, str) else content
|
||||||
|
info = tarfile.TarInfo(name)
|
||||||
|
info.size = len(data)
|
||||||
|
tar.addfile(info, io.BytesIO(data))
|
||||||
|
if extra:
|
||||||
|
extra(tar)
|
||||||
|
return buf.getvalue()
|
||||||
|
|
||||||
|
|
||||||
|
HELLO = tar_of({"main.py": "print('hi')\n"})
|
||||||
|
|
||||||
|
|
||||||
|
class Runner:
|
||||||
|
def __init__(self, tmp: Path) -> None:
|
||||||
|
self.tmp = tmp
|
||||||
|
bin_dir = tmp / "bin"
|
||||||
|
bin_dir.mkdir()
|
||||||
|
(bin_dir / "docker").symlink_to(FAKE)
|
||||||
|
self.env = {
|
||||||
|
"PATH": f"{bin_dir}:/usr/bin:/bin",
|
||||||
|
"FAKE_DOCKER_STATE": str(tmp / "state.json"),
|
||||||
|
"FAKE_DOCKER_LOG": str(tmp / "calls.log"),
|
||||||
|
"TGIT_RUNNER_STATE": str(tmp / "runner"),
|
||||||
|
"TGIT_HEALTH_TIMEOUT": "4",
|
||||||
|
"TGIT_SETTLE": "0",
|
||||||
|
}
|
||||||
|
|
||||||
|
def __call__(
|
||||||
|
self, *args: str, stdin: bytes = b"", ssh: str | None = None, **env: str
|
||||||
|
) -> subprocess.CompletedProcess[str]:
|
||||||
|
e = {**self.env, **env}
|
||||||
|
if ssh is not None:
|
||||||
|
e["SSH_ORIGINAL_COMMAND"] = ssh
|
||||||
|
return subprocess.run(
|
||||||
|
["bash", str(RUNNER), *args],
|
||||||
|
input=stdin.decode("latin-1"),
|
||||||
|
env=e,
|
||||||
|
capture_output=True,
|
||||||
|
text=True,
|
||||||
|
timeout=30,
|
||||||
|
encoding="latin-1",
|
||||||
|
)
|
||||||
|
|
||||||
|
def calls(self) -> list[list[str]]:
|
||||||
|
path = self.tmp / "calls.log"
|
||||||
|
return [json.loads(line) for line in path.read_text().splitlines()] if path.exists() else []
|
||||||
|
|
||||||
|
def state(self) -> dict[str, object]:
|
||||||
|
return json.loads((self.tmp / "state.json").read_text())
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.fixture
|
||||||
|
def runner(tmp_path: Path) -> Runner:
|
||||||
|
return Runner(tmp_path)
|
||||||
|
|
||||||
|
|
||||||
|
def last(out: subprocess.CompletedProcess[str]) -> str:
|
||||||
|
return out.stdout.strip().splitlines()[-1]
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.parametrize(
|
||||||
|
("ssh", "reason"),
|
||||||
|
[
|
||||||
|
("", "usage"),
|
||||||
|
("rm -rf /", "unknown command"),
|
||||||
|
("deploy hello; rm -rf /", "wrong number of arguments"),
|
||||||
|
("deploy Hello abc1234", "bad service name"),
|
||||||
|
("deploy ../etc abc1234", "bad service name"),
|
||||||
|
("deploy hello $(id)", "bad sha"),
|
||||||
|
("deploy hello abc1234 extra", "wrong number of arguments"),
|
||||||
|
("restart *", "bad service name"),
|
||||||
|
("list now", "list takes no arguments"),
|
||||||
|
],
|
||||||
|
)
|
||||||
|
def test_bad_requests_rejected_before_docker(runner: Runner, ssh: str, reason: str) -> None:
|
||||||
|
out = runner(ssh=ssh)
|
||||||
|
|
||||||
|
assert out.returncode == 2
|
||||||
|
assert reason in last(out)
|
||||||
|
assert runner.calls() == []
|
||||||
|
|
||||||
|
|
||||||
|
def test_deploy_runs_in_sandbox(runner: Runner) -> None:
|
||||||
|
out = runner(ssh="deploy hello abc1234", stdin=tar_of({"main.py": "x", "tgit.yaml": "network: true\n"}))
|
||||||
|
|
||||||
|
assert out.returncode == 0, out.stdout + out.stderr
|
||||||
|
assert last(out) == "OK deployed hello abc1234 network=none healthcheck=no" # network not allowed by admin
|
||||||
|
[run] = [c for c in runner.calls() if c[0] == "run"]
|
||||||
|
flags = " ".join(run)
|
||||||
|
for flag in [
|
||||||
|
"--user 10001:10001",
|
||||||
|
"--read-only",
|
||||||
|
"--cap-drop ALL",
|
||||||
|
"--security-opt no-new-privileges",
|
||||||
|
"--memory 128m --memory-swap 128m",
|
||||||
|
"--cpus 0.25",
|
||||||
|
"--pids-limit 64",
|
||||||
|
"--network none",
|
||||||
|
"--label tgit.managed=true",
|
||||||
|
"--label tgit.sha=abc1234",
|
||||||
|
]:
|
||||||
|
assert flag in flags
|
||||||
|
assert "/var/run/docker.sock" not in flags
|
||||||
|
assert list(runner.state()["containers"]) == ["tgit-svc-hello"] # -next renamed into place
|
||||||
|
|
||||||
|
|
||||||
|
def test_egress_only_when_admin_allows(runner: Runner) -> None:
|
||||||
|
(runner.tmp / "runner").mkdir()
|
||||||
|
(runner.tmp / "runner" / "egress-allow").write_text("weather\n")
|
||||||
|
|
||||||
|
out = runner(ssh="deploy weather abc1234", stdin=tar_of({"main.py": "x", "tgit.yaml": "network: true\n"}))
|
||||||
|
|
||||||
|
assert "network=tgit-svc-egress" in last(out)
|
||||||
|
|
||||||
|
|
||||||
|
def test_healthcheck_from_tgit_yaml_without_a_shell(runner: Runner) -> None:
|
||||||
|
yaml = "health_url: http://localhost:8080/health # probe\n"
|
||||||
|
out = runner(ssh="deploy web abc1234", stdin=tar_of({"main.py": "x", "tgit.yaml": yaml}))
|
||||||
|
|
||||||
|
assert out.returncode == 0
|
||||||
|
dockerfile = runner.state()["dockerfiles"]["tgit-svc-web:abc1234"] # type: ignore[index]
|
||||||
|
assert 'CMD ["python", "-c", "import sys, urllib.request' in dockerfile
|
||||||
|
assert '"http://localhost:8080/health"]' in dockerfile
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.parametrize(
|
||||||
|
"url", ["http://evil.com/x", "http://localhost:80/$(id)", 'http://localhost:80/"]; RUN id #', "file:///etc/passwd"]
|
||||||
|
)
|
||||||
|
def test_health_url_must_be_local(runner: Runner, url: str) -> None:
|
||||||
|
out = runner(ssh="deploy web abc1234", stdin=tar_of({"main.py": "x", "tgit.yaml": f"health_url: {url}\n"}))
|
||||||
|
|
||||||
|
assert out.returncode == 2 and "health_url" in last(out)
|
||||||
|
assert not [c for c in runner.calls() if c[0] == "build"]
|
||||||
|
|
||||||
|
|
||||||
|
def symlink_to_etc(tar: tarfile.TarFile) -> None:
|
||||||
|
info = tarfile.TarInfo("link")
|
||||||
|
info.type = tarfile.SYMTYPE
|
||||||
|
info.linkname = "/etc/passwd"
|
||||||
|
tar.addfile(info)
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.parametrize(
|
||||||
|
("archive", "reason"),
|
||||||
|
[
|
||||||
|
(b"not a tar", "not a tar archive"),
|
||||||
|
(tar_of({"main.py": "x"}, symlink_to_etc), "only files and directories"),
|
||||||
|
(tar_of({"main.py": "x", "../escape.py": "x"}), "unsafe path"),
|
||||||
|
(tar_of({"lib.py": "x"}), "main.py is missing"),
|
||||||
|
],
|
||||||
|
)
|
||||||
|
def test_bad_archives(runner: Runner, archive: bytes, reason: str) -> None:
|
||||||
|
out = runner(ssh="deploy hello abc1234", stdin=archive)
|
||||||
|
|
||||||
|
assert out.returncode == 2 and reason in last(out)
|
||||||
|
|
||||||
|
|
||||||
|
def test_archive_size_limit(runner: Runner) -> None:
|
||||||
|
out = runner(ssh="deploy hello abc1234", stdin=HELLO, TGIT_MAX_TAR="100")
|
||||||
|
|
||||||
|
assert out.returncode == 2 and "larger than" in last(out)
|
||||||
|
|
||||||
|
|
||||||
|
def test_unhealthy_new_version_keeps_the_old_one(runner: Runner) -> None:
|
||||||
|
assert runner(ssh="deploy hello abc1234", stdin=HELLO).returncode == 0
|
||||||
|
|
||||||
|
out = runner(ssh="deploy hello def5678", stdin=HELLO, FAKE_HEALTH="unhealthy", FAKE_STATUS="exited")
|
||||||
|
|
||||||
|
assert out.returncode == 3
|
||||||
|
assert last(out) == "FAIL not healthy within 4s, previous version kept"
|
||||||
|
assert "Traceback: boom" in out.stdout # the new version's log tail for the user
|
||||||
|
assert list(runner.state()["containers"]) == ["tgit-svc-hello"]
|
||||||
|
assert runner.state()["containers"]["tgit-svc-hello"]["image"] == "tgit-svc-hello:abc1234" # type: ignore[index]
|
||||||
|
|
||||||
|
|
||||||
|
def test_restarting_without_healthcheck_is_not_ready(runner: Runner) -> None:
|
||||||
|
out = runner(ssh="deploy hello abc1234", stdin=HELLO, FAKE_RESTARTS="2")
|
||||||
|
|
||||||
|
assert out.returncode == 3 and last(out) == "FAIL not healthy within 4s"
|
||||||
|
|
||||||
|
|
||||||
|
def test_build_failure(runner: Runner) -> None:
|
||||||
|
out = runner(ssh="deploy hello abc1234", stdin=HELLO, FAKE_BUILD_FAIL="1")
|
||||||
|
|
||||||
|
assert out.returncode == 5 and "could not install requirements" in out.stdout
|
||||||
|
|
||||||
|
|
||||||
|
def test_service_limit(runner: Runner) -> None:
|
||||||
|
for i in range(3):
|
||||||
|
assert runner(ssh=f"deploy s{i}x abc1234", stdin=HELLO, TGIT_MAX_SERVICES="3").returncode == 0
|
||||||
|
|
||||||
|
out = runner(ssh="deploy extra abc1234", stdin=HELLO, TGIT_MAX_SERVICES="3")
|
||||||
|
redeploy = runner(ssh="deploy s0x def5678", stdin=HELLO, TGIT_MAX_SERVICES="3")
|
||||||
|
|
||||||
|
assert out.returncode == 4 and "limit 3" in last(out)
|
||||||
|
assert redeploy.returncode == 0 # an existing service may always be updated
|
||||||
|
|
||||||
|
|
||||||
|
def test_old_images_pruned(runner: Runner) -> None:
|
||||||
|
for sha in ("aaa1111", "bbb2222", "ccc3333"):
|
||||||
|
assert runner(ssh=f"deploy hello {sha}", stdin=HELLO).returncode == 0
|
||||||
|
|
||||||
|
assert sorted(runner.state()["images"]) == ["tgit-svc-hello:bbb2222", "tgit-svc-hello:ccc3333"] # type: ignore[type-var]
|
||||||
|
|
||||||
|
|
||||||
|
def test_restart_remove_list(runner: Runner) -> None:
|
||||||
|
assert last(runner(ssh="restart hello")) == "FAIL no such service"
|
||||||
|
runner(ssh="deploy hello abc1234", stdin=HELLO)
|
||||||
|
|
||||||
|
assert last(runner(ssh="restart hello")) == "OK restarted hello"
|
||||||
|
assert last(runner(ssh="list")) == "OK list"
|
||||||
|
assert last(runner(ssh="remove hello")) == "OK removed hello"
|
||||||
|
assert runner.state()["containers"] == {}
|
||||||
|
assert runner.state()["images"] == []
|
||||||
|
|
||||||
|
|
||||||
|
def test_direct_invocation_without_ssh(runner: Runner) -> None:
|
||||||
|
out = runner("list")
|
||||||
|
assert out.returncode == 0
|
||||||
|
|
||||||
|
|
||||||
|
def test_glob_in_ssh_command_is_not_expanded(runner: Runner, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
||||||
|
monkeypatch.chdir(tmp_path)
|
||||||
|
(tmp_path / "abc1234").write_text("")
|
||||||
|
out = runner(ssh="deploy hello abc12*")
|
||||||
|
assert out.returncode == 2 and "bad sha" in last(out)
|
||||||
@@ -3,7 +3,7 @@ from pathlib import Path
|
|||||||
import pytest
|
import pytest
|
||||||
from pydantic import ValidationError
|
from pydantic import ValidationError
|
||||||
|
|
||||||
from app.config import AccessSettings
|
from app.config import AccessSettings, Settings
|
||||||
from app.texts import load_texts
|
from app.texts import load_texts
|
||||||
|
|
||||||
|
|
||||||
@@ -21,3 +21,11 @@ def test_texts_with_missing_key_fail_fast(tmp_path: Path) -> None:
|
|||||||
(tmp_path / "texts.yaml").write_text("menu: {commit: x}\n", encoding="utf-8")
|
(tmp_path / "texts.yaml").write_text("menu: {commit: x}\n", encoding="utf-8")
|
||||||
with pytest.raises(ValidationError):
|
with pytest.raises(ValidationError):
|
||||||
load_texts(tmp_path)
|
load_texts(tmp_path)
|
||||||
|
|
||||||
|
|
||||||
|
def test_ci_runner_env_does_not_break_settings(monkeypatch: pytest.MonkeyPatch) -> None:
|
||||||
|
# Gitea/GitHub runners export CI=true; it must not collide with a settings field
|
||||||
|
monkeypatch.setenv("CI", "true")
|
||||||
|
monkeypatch.setenv("TELEGRAM_BOT_TOKEN", "42:x")
|
||||||
|
|
||||||
|
assert Settings().uploads_ci.provider == "none"
|
||||||
|
|||||||
Reference in New Issue
Block a user