fix(ci): #909 #910 #911 fix monitor workflows - checkout + runner labels #927

Merged

xiaoxia merged 2 commits from fix/ci-monitor-workflows-909-910-911 into develop 2026-07-26 14:10:34 +08:00

Conversation 3 Commits 2 Files Changed 4

+18 -20

xiaoxia commented 2026-07-26 13:14:05 +08:00

Owner

Copy Link

Copy Source

#909 ci-trigger-monitor 全失败

根因：uses: actions/checkout@v3 不被 Gitea act runner 支持，且 runs-on: ubuntu-latest 标签不匹配
修复：

• 替换为 curl + tarball 的 shell 脚本 checkout（与 ci-pipeline 一致）
• runs-on: ubuntu-latest → runs-on: ci-l2

#910 code-review 全失败

根因：与 #909 完全相同
修复：同样的 checkout 和标签替换

#911 3个监控workflow从未运行

根因：daily-check.yml 和 ci-health-daily.yml 使用 runs-on: saas 标签，但没有 runner 注册此标签，导致调度后无法分配执行
修复：

• runs-on: saas → runs-on: ci-l2（共 6 处，daily-check 5个 job + ci-health-daily 1个 job）
• ci-failure-monitor.yml 标签本身正确（ci-l2），schedule 功能正常（ci-trigger-monitor 每5分钟触发证明可用），每6小时触发频率较低

关 #909 #910 #911

## #909 ci-trigger-monitor 全失败 **根因**：`uses: actions/checkout@v3` 不被 Gitea act runner 支持，且 `runs-on: ubuntu-latest` 标签不匹配 **修复**： - 替换为 curl + tarball 的 shell 脚本 checkout（与 ci-pipeline 一致） - `runs-on: ubuntu-latest` → `runs-on: ci-l2` ## #910 code-review 全失败 **根因**：与 #909 完全相同 **修复**：同样的 checkout 和标签替换 ## #911 3个监控workflow从未运行 **根因**：`daily-check.yml` 和 `ci-health-daily.yml` 使用 `runs-on: saas` 标签，但没有 runner 注册此标签，导致调度后无法分配执行 **修复**： - `runs-on: saas` → `runs-on: ci-l2`（共 6 处，daily-check 5个 job + ci-health-daily 1个 job） - `ci-failure-monitor.yml` 标签本身正确（ci-l2），schedule 功能正常（ci-trigger-monitor 每5分钟触发证明可用），每6小时触发频率较低 关 #909 #910 #911

xiaoxia added 1 commit 2026-07-26 13:14:05 +08:00

fix(ci): #909 #910 #911 fix monitor workflows - replace actions/checkout + fix runner labels 6a2f38c056

xiaoxia added 1 commit 2026-07-26 13:15:56 +08:00

Merge branch 'develop' of https://git.xiaoxiajianji.com/xiaoxia/xiaoxia-saas into fix/ci-monitor-workflows-909-910-911 93af88304a

auto-approve-bot commented 2026-07-26 13:30:18 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #927

⚠️ 问题（3个需要修改）

1. .gitea/workflows/code-review.yml 第22行：移除了 fetch-depth: 0 配置。代码审查任务通常需要完整的 Git 历史记录才能进行准确的代码比对（如与父 Commit 或 Base 分支对比），移除此配置将导致默认浅克隆（depth=1），极大概率导致后续审查步骤因无法获取历史上下文而报错或分析结果不完整。
2. .g.gitea/workflows/ci-trigger-monitor.yml 第20行：移除了原有的 retry 重试机制。原代码配置了网络波动自动重试，而新的 curl 命令未包含重试参数，在网络不稳定的环境下 CI 健壮性会显著降低。
3. .gitea/workflows/code-review.yml 第22行：存在安全与稳定性风险。使用 curl ... | bash 直接执行远程脚本是不安全的做法（Supply Chain 风险）。此外，若 curl 请求失败（如 404），标准错误输出可能被管道传入 bash 执行，导致难以排查的异常行为。

💡 建议（1个可选）

1. 如果必须使用 curl 进行 checkout，建议在命令中添加 --retry 3 --retry-delay 5 --fail 参数，以恢复网络容错能力并确保请求失败时立即中断。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-26 05:30:18 | 模型:}

## 代码审查结果 - PR #927 ### ⚠️ 问题（3个需要修改） 1. **.gitea/workflows/code-review.yml 第22行**：移除了 `fetch-depth: 0` 配置。代码审查任务通常需要完整的 Git 历史记录才能进行准确的代码比对（如与父 Commit 或 Base 分支对比），移除此配置将导致默认浅克隆（depth=1），极大概率导致后续审查步骤因无法获取历史上下文而报错或分析结果不完整。 2. **.g.gitea/workflows/ci-trigger-monitor.yml 第20行**：移除了原有的 `retry` 重试机制。原代码配置了网络波动自动重试，而新的 `curl` 命令未包含重试参数，在网络不稳定的环境下 CI 健壮性会显著降低。 3. **.gitea/workflows/code-review.yml 第22行**：存在安全与稳定性风险。使用 `curl ... | bash` 直接执行远程脚本是不安全的做法（Supply Chain 风险）。此外，若 `curl` 请求失败（如 404），标准错误输出可能被管道传入 `bash` 执行，导致难以排查的异常行为。 ### 💡 建议（1个可选） 1. 如果必须使用 `curl` 进行 checkout，建议在命令中添加 `--retry 3 --retry-delay 5 --fail` 参数，以恢复网络容错能力并确保请求失败时立即中断。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-26 05:30:18 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-07-26 13:33:44 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#927
预览链接	https://pr-927.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #927 | | 预览链接 | [https://pr-927.preview.xiaoxiajianji.com](https://pr-927.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

xiaoxia merged commit 23573a8209 into develop 2026-07-26 14:10:34 +08:00

gitea-actions bot commented 2026-07-26 14:11:50 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #927 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #927 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#927