chore(ci): 删除 synthetic-monitor (monitoring theater + 长期红)#155
Merged
Conversation
根因 1 (表面): pip install -e runtime/ 不装 utils/ 依赖 (openpyxl/factory-boy) → 每跑必 ModuleNotFoundError。 根因 2 (设计): 4 region matrix 全跑同一 GitHub-hosted Ubuntu,假分发,无真实多地域监控价值。 根因 3 (重复): selftest-weekly.yml 已覆盖 smoke + 真 LLM e2e + artifact 保留, synthetic-monitor 是其劣化版。 每 6 小时 × 4 假 region = 16 红 run/天,长期吃 Actions minutes + 噪音。 与 V1.43.0 防 mock / 诚实化主线相悖 (monitoring theater = mock 变体)。 未来若真需多 region 监控,走真实跨地域分发 (Vercel/CF Workers/外部监控),不再 matrix 假装。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
目标
删除
.github/workflows/synthetic-monitor.yml—— monitoring theater,长期红,无真实价值。诊断
1. 表面问题(依赖缺失)
pip install -e runtime/只装 runtime 自身依赖,跑 utils/ 时缺openpyxl/factory-boy→ 每次跑都 ModuleNotFoundError。2. 设计层问题(核心)
matrix.region只是 label,没有真分发到 us-east/us-west/eu-west/ap-southeast)3. 已被 selftest-weekly 完整覆盖
处置
直接删除
synthetic-monitor.yml。selftest-weekly.yml 保留 smoke + e2e 覆盖。不在本 PR 范围