grok
bc9b29dec8
Зачем: прежний CI запускал миграции и тесты под POSTGRES_USER сервиса (суперпользователь), не проверял откат миграций, не запускал e2e на реальной БД, держал плавающие теги actions (F13) и выполнял docker build в контейнере без Docker daemon. - verify: bootstrap-админ сервиса только создаёт роли (provision-pg.mjs, пароли случайные); миграции up -> откат всех -> up -> up (no-op) ролью krona_migrator без SUPERUSER; `npm run check` с DATABASE_TEST_URL (app-роль) и DATABASE_TEST_MIGRATION_URL (очистка, см. N5); npm audit. - format: prettier вынесен в отдельную задачу — он красный по 25 файлам (F14) и не должен маскировать статус verify/e2e. - e2e: реальный сервер + PostgreSQL 17 через scripts/test/run-e2e.mjs. Выбран «храповик» known-defects.json, а не отдельная задача с continue-on-error: так CI зелёный, только если стенд работает и каждый красный сценарий — известный открытый дефект; новый красный сценарий (регрессия) или исправленный, но не вычеркнутый дефект делают задачу красной. continue-on-error скрыл бы и поломку самого стенда. - docker-build: отдельная задача на метке `docker-host` только для push, после verify; без такого раннера она ждёт в очереди, не блокируя остальные. - actions/checkout v4.4.0 и actions/setup-node v4.4.0 закреплены SHA (git ls-remote; одинаковы на github.com и code.forgejo.org). Проверено: actionlint 1.7.12 (метки docker/docker-host объявлены как self-hosted) — 0 замечаний; scripts/test/ci-local.sh повторяет шаги verify/format/e2e на локальной PostgreSQL 17.11 (Forgejo-раннер реально не запускался — не проверено).
104 lines
4.9 KiB
YAML
104 lines
4.9 KiB
YAML
name: ci
|
||
on:
|
||
push:
|
||
branches: [main, codex/bootstrap-platform-foundation]
|
||
pull_request:
|
||
|
||
# Все actions закреплены полным SHA коммита (F13); SHA получены `git ls-remote`
|
||
# и совпадают на github.com и code.forgejo.org (зеркало), поэтому закрепление
|
||
# работает при любом DEFAULT_ACTIONS_URL раннера.
|
||
#
|
||
# Роли БД как в pilot, но с минимальными привилегиями: POSTGRES_USER сервиса —
|
||
# только bootstrap-админ (роль DBA/entrypoint образа), им scripts/test/provision-pg.mjs
|
||
# создаёт krona_migrator (NOSUPERUSER, владелец БД) и krona_staff_app (NOBYPASSRLS)
|
||
# со случайными паролями. Ни миграции, ни тесты, ни сервер под bootstrap-админом не работают.
|
||
|
||
jobs:
|
||
verify:
|
||
runs-on: docker
|
||
services:
|
||
postgres:
|
||
image: postgres:17-alpine@sha256:18cfe3ef5e6815560c98237d6216d1e5119702fb0f3894c8785dd58b8bbe5d73
|
||
env:
|
||
POSTGRES_DB: postgres
|
||
POSTGRES_USER: ci_bootstrap
|
||
POSTGRES_PASSWORD: ci-bootstrap-only
|
||
options: >-
|
||
--health-cmd "pg_isready -U ci_bootstrap -d postgres"
|
||
--health-interval 5s
|
||
--health-timeout 3s
|
||
--health-retries 20
|
||
env:
|
||
E2E_PG_ADMIN_URL: postgres://ci_bootstrap:ci-bootstrap-only@postgres:5432/postgres
|
||
steps:
|
||
- uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4.4.0
|
||
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
|
||
with: { node-version: 22, cache: npm }
|
||
- run: npm ci
|
||
- name: provision roles (migrator NOSUPERUSER, app NOBYPASSRLS)
|
||
run: node scripts/test/provision-pg.mjs --out "$RUNNER_TEMP/db.env" --db krona_staff
|
||
- name: migrations up -> rollback all -> up -> up (no-op)
|
||
run: |
|
||
set -a; . "$RUNNER_TEMP/db.env"; set +a
|
||
node scripts/test/migrate-roundtrip.mjs roundtrip
|
||
- name: npm run check (tenant-isolation on real PostgreSQL as app role)
|
||
run: |
|
||
set -a; . "$RUNNER_TEMP/db.env"; set +a
|
||
unset DATABASE_URL POSTGRES_APP_PASSWORD
|
||
npm run check
|
||
- run: npm audit --audit-level=high
|
||
|
||
# Отдельная задача: prettier сейчас красный по 25 файлам (F14), и это не должно
|
||
# маскировать статус verify/e2e.
|
||
format:
|
||
runs-on: docker
|
||
steps:
|
||
- uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4.4.0
|
||
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
|
||
with: { node-version: 22, cache: npm }
|
||
- run: npm ci
|
||
- run: npm run format:check
|
||
|
||
# Приёмочные сценарии на реальном сервере и PostgreSQL 17. Красные сценарии
|
||
# известных открытых дефектов перечислены в tests/e2e/known-defects.json:
|
||
# задача зелёная, если стенд работает ([HARNESS]/[CTRL] зелёные) и каждый
|
||
# красный сценарий есть в списке; новый красный сценарий или исправленный,
|
||
# но не вычеркнутый дефект делают задачу красной.
|
||
e2e:
|
||
runs-on: docker
|
||
services:
|
||
postgres:
|
||
image: postgres:17-alpine@sha256:18cfe3ef5e6815560c98237d6216d1e5119702fb0f3894c8785dd58b8bbe5d73
|
||
env:
|
||
POSTGRES_DB: postgres
|
||
POSTGRES_USER: ci_bootstrap
|
||
POSTGRES_PASSWORD: ci-bootstrap-only
|
||
options: >-
|
||
--health-cmd "pg_isready -U ci_bootstrap -d postgres"
|
||
--health-interval 5s
|
||
--health-timeout 3s
|
||
--health-retries 20
|
||
env:
|
||
E2E_PG_ADMIN_URL: postgres://ci_bootstrap:ci-bootstrap-only@postgres:5432/postgres
|
||
E2E_MIGRATIONS: roundtrip
|
||
steps:
|
||
- uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4.4.0
|
||
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
|
||
with: { node-version: 22, cache: npm }
|
||
- run: npm ci
|
||
- name: e2e (known-defects ratchet)
|
||
run: node scripts/test/run-e2e.mjs
|
||
- name: e2e table
|
||
if: always()
|
||
run: cat tests/e2e/evidence/e2e-table.md || true
|
||
|
||
# docker build требует Docker daemon, которого нет в контейнере метки `docker`.
|
||
# Нужен раннер с меткой `docker-host` (хост с доступом к Docker). Пока такого
|
||
# раннера нет, задача ждёт в очереди и не блокирует verify/format/e2e.
|
||
docker-build:
|
||
runs-on: docker-host
|
||
needs: [verify]
|
||
if: github.event_name == 'push'
|
||
steps:
|
||
- uses: actions/checkout@11d5960a326750d5838078e36cf38b85af677262 # v4.4.0
|
||
- run: docker build -t "krona-staff:${GITHUB_SHA}" .
|