fix(ci): Validate增加独立PG容器，修复alembic迁移检查全部失败 #405

Merged

xiaoxia merged 12 commits from fix/ci-validate-pg-container into develop 2026-07-16 20:06:09 +08:00

Conversation 0 Commits 12 Files Changed 4

+65 -22

xiaoxia commented 2026-07-16 15:13:16 +08:00

Owner

Copy Link

Copy Source

问题

所有open PR的Validate Code Quality And Tests都失败，根因是 Validate Alembic migrations步骤无法连接数据库。

根因分析

• alembic upgrade head --sql 需要连接数据库读取 alembic_version 表确定起始版本
• 但Validate job没有启动PostgreSQL容器，DATABASE_URL 指向 127.0.0.1:5432（端口关闭）
• Integration Tests有启动PG所以能过，Validate没有所以全挂

修复方案

在 Validate Alembic migrations 步骤前增加一步：启动独立PG容器（随机端口映射），通过 GITHUB_ENV 更新 DATABASE_URL。步骤结束后 always() 确保清理容器。

参考PR #399的DB Migrations子job实现（已验证通过），此为最小改动版本，不拆分job。

验收标准

• 本PR的Validate Alembic migrations + Check migration safety全部通过
• 其他PR rebase到此分支后Validate不再因数据库连接失败

## 问题 所有open PR的Validate Code Quality And Tests都失败，根因是 **Validate Alembic migrations步骤无法连接数据库**。 ## 根因分析 - `alembic upgrade head --sql` 需要连接数据库读取 `alembic_version` 表确定起始版本 - 但Validate job没有启动PostgreSQL容器，`DATABASE_URL` 指向 `127.0.0.1:5432`（端口关闭） - Integration Tests有启动PG所以能过，Validate没有所以全挂 ## 修复方案 在 `Validate Alembic migrations` 步骤前增加一步：启动独立PG容器（随机端口映射），通过 `GITHUB_ENV` 更新 `DATABASE_URL`。步骤结束后 `always()` 确保清理容器。 参考PR #399的DB Migrations子job实现（已验证通过），此为最小改动版本，不拆分job。 ## 验收标准 - [ ] 本PR的Validate Alembic migrations + Check migration safety全部通过 - [ ] 其他PR rebase到此分支后Validate不再因数据库连接失败

xiaoxia added 1 commit 2026-07-16 15:13:17 +08:00

fix(ci): Validate job增加独立PG容器，修复alembic迁移检查失败\n\n根因：alembic upgrade head --sql需要连接数据库读取alembic_version表，\n但Validate job没有启动PostgreSQL容器，DATABASE_URL指向不存在的端口。\n\n修复：在Validate Alembic migrations步骤前启动一个独立PG容器，\n用随机端口映射，通过GITHUB_ENV更新DATABASE_URL。\n步骤结束后自动清理容器（always()确保失败也清理）。\n\n参考PR #399的实现，最小改动版本，不拆分job。 7ca0014da2

xiaoxia added 1 commit 2026-07-16 15:36:00 +08:00

fix(ci): 修复develop分支遗留的isort import排序问题 ... 31aa3fbb86

修复edit_plans_clips.py和edit_plans_clips_batch.py的import排序，
确保全量isort扫描通过。

xiaoxia added 1 commit 2026-07-16 16:07:37 +08:00

fix(ci): PG启动+alembic合并为单步，修复GITHUB_ENV传参失效 f4b7bec0cb

xiaoxia added 1 commit 2026-07-16 16:08:38 +08:00

chore: trigger CI re-run 6f0bb8c25e

xiaoxia added 1 commit 2026-07-16 16:11:38 +08:00

fix(ci): 修复YAML语法，alembic步骤改用literal block scalar ee6034b5fd

xiaoxia added 1 commit 2026-07-16 16:45:59 +08:00

ci: reduce detect-secrets stdout output to avoid log truncation 715e663f8b

xiaoxia added 1 commit 2026-07-16 17:43:26 +08:00

ci: silence detect-secrets stdout to preserve log output 2a6c7d4086

xiaoxia added 1 commit 2026-07-16 17:49:43 +08:00

ci: revert detect-secrets args, use simple redirect 188fec52c6

xiaoxia added 1 commit 2026-07-16 18:03:37 +08:00

ci: init git repo before migration safety check for diff support cf6b625ffb

xiaoxia added 1 commit 2026-07-16 18:25:30 +08:00

ci: fix migration safety check for tar.gz checkout environment e8e4928062

xiaoxia added 1 commit 2026-07-16 18:43:58 +08:00

ci: use refspec to create origin/develop remote branch after fetch a3df4fb324

xiaoxia added 1 commit 2026-07-16 19:57:28 +08:00

test: trigger CI to validate runner log fix 71d1a8e78d

xiaoxia merged commit bfce6f48ce into develop 2026-07-16 20:06:09 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#405