Prhub

#21978 [CI] Remove stale Ascend suite entries from test/srt/run_suite.py

原始 PR 作者 Fridge003 合并时间 2026-04-03 07:47 文件变更 1 提交数 1 评论 3 代码增减 +3 / -37

执行摘要

清理 CI 测试套件中过时的 Ascend NPU 条目,修复 XPU CI 检查失败。

根据PR body描述,Ascend NPU测试已迁移到新的注册制CI系统(test/registered/ascend/,由test/run_suite.py管理),但旧条目未清理。这导致XPU CI的sanity check失败,因为_sanity_check_suites()断言所有缺失的Ascend测试文件都存在,即使XPU从不运行这些测试。具体失败见https://github.com/sgl-project/sglang/actions/runs/23919335318/job/69761428239。

该PR变更简单直接,适合快速浏览以了解CI套件清理模式。值得关注的是如何通过__not_in_ci__处理遗留文件,以及sanity check机制对文件存在性的依赖。

讨论亮点

无review评论,但关联Issue评论显示:

  1. gemini-code-assist[bot]提示配额限制。
  2. Fridge003执行/tag-and-rerun-ci。
  3. Godmook确认此PR已修复问题,并提及#21916(可能为相关PR)。

实现拆解

  1. 删除test/srt/run_suite.py中整个suite_ascend字典定义(37行删除),包括per-commit-1-npu-a2、per-commit-2-npu-a2、per-commit-4-npu-a3、per-commit-16-npu-a3四个子套件。
  2. 将唯一保留的本地文件ascend/test_embed_interpolate_unittest.py移至__not_in_ci__列表(3行添加)。
  3. 移除suites.update(suite_ascend)调用。
文件 模块 状态 重要度
test/srt/run_suite.py CI/ 测试基础设施 modified 8.0

关键符号

_sanity_check_suites()(间接影响)

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

修复 CI 失败确认 other

Godmook 在关联 Issue 评论中确认此 PR 已修复问题,并提及 #21916。

结论:PR 有效解决了 XPU CI 的 sanity check 失败。 · 已解决

风险与影响

  1. 回归风险:删除suite_ascend可能意外影响依赖旧路径的脚本或文档,但新注册系统已就绪,风险较低。
  2. 兼容性风险:__not_in_ci__列表添加文件可能被误用,但该文件本就不在CI中,影响有限。
  3. 逻辑风险:sanity check逻辑假设所有套件条目对应真实文件,此次清理正修复此假设,无新增风险。
  1. 对系统:修复XPU CI稳定性,避免因过时条目导致失败;清理代码库,减少技术债务。
  2. 对团队:CI运行更可靠,减少调试时间;明确测试套件管理边界。
  3. 对用户:无直接影响,属内部基础设施改进。
清理过时条目 CI 依赖假设

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论