ci(e2e): 落地「活栈供给方式」决策 — Compose in CI,并修掉 6 处真实缺陷
决策:E2E 活栈在 runner 内用 docker compose 起全栈(postgres + redis + minio + backend), 与本地验证路径完全一致;未采纳「service containers + runner 上 mvn package 后 java -jar」—— 那需要为后端另写一套 host 网络配置,等于制造第二条与本地不一致的路径。 本机实测(与 CI 同一序列):中间件三项 healthcheck 全 Healthy(pg_isready / redis-cli ping / mc ready local)→ backend /api/v1/health/live 返回 200 → seed 幂等可执行(INSERT 0 0) → pytest tests/e2e → 49 passed。 (本机限制:Docker Hub 不可达且无 temurin 基础镜像,未复现 --build;改用本地既有镜像为底座、 挂载当日 16:24 构建的 jar,使被验证的代码版本与 CI 一致 —— 否则是拿 09-09 的旧代码验 09-18 的用例。) 修掉的 6 处缺陷: 1. minio/minio:latest 是滚动 tag → pin RELEASE.2025-09-07T16-13-09Z(否则 CI 拉到的镜像 与本地验证时不同,表现为不可复现的偶发失败) 2. up 未显式列出 minio(靠 depends_on 隐式拉起),而 minio 的 healthcheck 正是 backend 能否启动的前置条件 3. up --build 把构建与起栈耦合,失败日志混杂 → 拆为 build / up 两步 4. 就绪窗口 5min 偏紧(首次构建后仍需 Spring 启动 + Flyway 46 个迁移)→ 7.5min 5. 缺 down -v 前置清卷:seed 用 ON CONFLICT (username) DO NOTHING,残留卷会让账号口令 停留在上一轮 → 登录 401,并伪装成业务失败 6. job 无 timeout(GH Actions 默认 360 分钟)→ timeout-minutes: 30 另:pytest -v → pytest tests/e2e -v。原以为从仓库根收集会误收 tests/*_test.py (其在 import 期依赖 psycopg2 与 E2E_DB_PASSWORD),实测否定:pytest.ini 的 testpaths = tests/e2e 已限定范围,从根收集正是 49 个用例且无 error。显式传参作为兜底保留。 计划 Task 2.3 由「依赖基础设施决策」改为「已决策并落地」,仅余 Step 3(CI 上实测一次绿)。
This commit is contained in:
@@ -164,6 +164,8 @@ jobs:
|
||||
e2e:
|
||||
name: E2E (live stack)
|
||||
runs-on: ubuntu-latest
|
||||
# 防挂死:GH Actions 默认 360 分钟,起栈失败若无人超时会把流水线拖住数小时。
|
||||
timeout-minutes: 30
|
||||
# Phase 23 遗留 D-23-D:计划要求 CI 串联 E2E。此前 E2E 从未真正执行过——
|
||||
# 无 conftest、fixture 不存在、文件在 import 期就崩。本 job 起真实栈后执行。
|
||||
needs: [backend, migration]
|
||||
@@ -194,20 +196,43 @@ jobs:
|
||||
WECHAT_APP_SECRET=
|
||||
ENV
|
||||
|
||||
- name: 起真实栈(postgres + redis + backend)
|
||||
# 「活栈供给方式」决策(2026-09-18):**在 runner 内用 Compose 起全栈**。
|
||||
# 备选是 service containers + 在 runner 上 `mvn package` 后 `java -jar`,但那需要为后端
|
||||
# 另写一套 host 网络配置,等于制造第二条与本地不一致的路径;Compose 全栈则与本地验证
|
||||
# 路径完全相同(2026-09-18 本机按同一序列实测:中间件三项 healthcheck 全 Healthy、
|
||||
# backend /health/live 200、seed 幂等可执行、**pytest tests/e2e 49 passed**)。
|
||||
- name: 校验 compose 配置(秒级快速失败,早于任何构建)
|
||||
working-directory: infrastructure
|
||||
run: docker compose up -d --build postgres redis backend
|
||||
run: docker compose config --quiet
|
||||
|
||||
# 保证库为全新:seed 用 `ON CONFLICT (username) DO NOTHING`,若卷是上一轮残留,
|
||||
# 账号密码会停留在上一轮的 `E2E_OPERATOR_PASSWORD` 上 → 本轮登录 401(表现为各种业务失败)。
|
||||
- name: 清理可能的残留栈与数据卷
|
||||
working-directory: infrastructure
|
||||
run: docker compose down -v --remove-orphans || true
|
||||
|
||||
# 构建与起栈分两步:构建失败时的日志不混杂在 up 输出里,定位更快。
|
||||
- name: 构建 backend 镜像
|
||||
working-directory: infrastructure
|
||||
run: docker compose build backend
|
||||
|
||||
- name: 起真实栈(postgres + redis + minio + backend)
|
||||
# 显式列出 minio:backend 的 depends_on 虽会隐式拉起它,但隐式依赖不利于阅读,
|
||||
# 且 minio 的 healthcheck(`mc ready local`)是 backend 能否启动的前置条件。
|
||||
working-directory: infrastructure
|
||||
run: docker compose up -d postgres redis minio backend
|
||||
|
||||
- name: 等待 backend 就绪(超时即失败,不静默跳过)
|
||||
# 窗口 90×5s=7.5min:首次构建后还需 Spring 启动 + Flyway 46 个迁移,5min 偏紧。
|
||||
run: |
|
||||
for i in $(seq 1 60); do
|
||||
for i in $(seq 1 90); do
|
||||
if curl -sf "$E2E_BASE_URL/health/live" > /dev/null; then
|
||||
echo "backend READY after ${i} checks"; exit 0
|
||||
fi
|
||||
sleep 5
|
||||
done
|
||||
echo "::error::backend 未在 5 分钟内就绪"
|
||||
docker compose -f infrastructure/docker-compose.yml logs backend
|
||||
echo "::error::backend 未在 7.5 分钟内就绪"
|
||||
docker compose -f infrastructure/docker-compose.yml logs --tail=200 backend
|
||||
exit 1
|
||||
|
||||
- name: Seed E2E 运营账号(迁移不含 operator 种子,全新库无账号可登录)
|
||||
@@ -217,7 +242,10 @@ jobs:
|
||||
-v pwd="$E2E_OPERATOR_PASSWORD" -f - < ../tests/e2e/sql/seed_e2e_operators.sql
|
||||
|
||||
- name: Run E2E
|
||||
run: pytest -v
|
||||
# 显式给出目录:pytest.ini 的 `testpaths = tests/e2e` 已限定范围,但显式传参可防
|
||||
# 该配置被改动后悄悄扩大收集范围(仓库根另有 tests/*_test.py 不是 E2E 用例,
|
||||
# 且它们在 import 期即依赖 psycopg2 与 E2E_DB_PASSWORD,会被收集成 error)。
|
||||
run: pytest tests/e2e -v
|
||||
|
||||
- name: Teardown(无论成败都清理并留日志)
|
||||
if: always()
|
||||
|
||||
Reference in New Issue
Block a user