fix(main): add ci-canary.yml + disable pr-auto-scan schedule on main #1638

Merged

xiaoxia merged 1 commits from infra/fix-main-workflows into main 2026-09-03 10:57:31 +08:00

Conversation 3 Commits 1 Files Changed 2

+52 -1

xiaoxia commented 2026-09-03 10:47:29 +08:00

Owner

Copy Link

Copy Source

Summary

Sync two critical workflow fixes from develop to main:

1. Add ci-canary.yml (from PR #1636): 30-minute schedule health check for runner→Docker→network→Gitea chain
2. Disable pr-auto-scan.yml schedule (PR #1635 already disabled on develop, but main still had */15 running)

Changes

• • New file copied from develop
• • Comment out schedule cron on main

## Summary Sync two critical workflow fixes from develop to main: 1. **Add ci-canary.yml** (from PR #1636): 30-minute schedule health check for runner→Docker→network→Gitea chain 2. **Disable pr-auto-scan.yml schedule** (PR #1635 already disabled on develop, but main still had */15 running) ## Changes - - New file copied from develop - - Comment out schedule cron on main

xiaoxia added 1 commit 2026-09-03 10:47:30 +08:00

fix(main): add ci-canary.yml + disable pr-auto-scan schedule on main ... 1cd592c957

- Copy ci-canary.yml from develop (PR #1636) to enable 30-min health checks on main
- Disable pr-auto-scan.yml schedule on main (was still running */15, causing failure spam)
- Relates: PR #1635 (develop already had schedule disabled)

gitea-actions bot commented 2026-09-03 10:48:51 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-09-03 10:51:31 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1638

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

1. .gitea/workflows/ci-canary.yml 第15行：错误处理逻辑失效，由于脚本开头设置了 set -e，当 curl 命令失败（如网络不通、DNS解析失败）时，脚本会立即退出，导致后续第16行的自定义错误信息 ::error::Gitea API unreachable 无法打印，降低了故障排查效率。
2. .gitea/workflows/ci-canary.yml 第22行：错误处理逻辑失效，同理，当 getent hosts 命令失败时，脚本会立即退出，导致第24-25行的自定义 DNS 解析失败提示无法打印。

💡 建议（2个可选）

1. .gitea/workflows/ci-canary.yml 第22行：域名 git.xiaoxiajianji.com 被硬编码在脚本中，建议使用环境变量（如 ${{ env.GITHUB_SERVER_URL }} 提取域名）或通过 Workflow 输入传入，以提高配置的灵活性和可移植性。
2. .gitea/workflows/pr-auto-scan.yml 第6行：注释内容“每10分钟扫描一次”与实际 Cron 表达式 */15（每15分钟）不一致，建议修正注释以保持代码与文档的同步。

---

❌ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 02:51:31 | 模型:}

## 代码审查结果 - PR #1638 ### ⚠️ 问题（2个需要修改） 1. **.gitea/workflows/ci-canary.yml 第15行**：错误处理逻辑失效，由于脚本开头设置了 `set -e`，当 `curl` 命令失败（如网络不通、DNS解析失败）时，脚本会立即退出，导致后续第16行的自定义错误信息 `::error::Gitea API unreachable` 无法打印，降低了故障排查效率。 2. **.gitea/workflows/ci-canary.yml 第22行**：错误处理逻辑失效，同理，当 `getent hosts` 命令失败时，脚本会立即退出，导致第24-25行的自定义 DNS 解析失败提示无法打印。 ### 💡 建议（2个可选） 1. **.gitea/workflows/ci-canary.yml 第22行**：域名 `git.xiaoxiajianji.com` 被硬编码在脚本中，建议使用环境变量（如 `${{ env.GITHUB_SERVER_URL }}` 提取域名）或通过 Workflow 输入传入，以提高配置的灵活性和可移植性。 2. **.gitea/workflows/pr-auto-scan.yml 第6行**：注释内容“每10分钟扫描一次”与实际 Cron 表达式 `*/15`（每15分钟）不一致，建议修正注释以保持代码与文档的同步。 --- ❌ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 02:51:31 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 83542cab86 into main 2026-09-03 10:57:31 +08:00

gitea-actions bot commented 2026-09-03 10:57:45 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1638 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#1638