Prhub

#21187 ci: unify PR test suite naming

原始 PR 作者 merrymercy 合并时间 2026-03-23 15:18 文件变更 283 提交数 1 评论 2 代码增减 +554 / -554

执行摘要

统一 CI 测试套件命名,提升命名一致性和可读性。

根据 PR body,变更旨在“rename CI job IDs and registered-test suite= strings to a single, predictable pattern so stage / GPU count / size are ordered consistently”,以改进 CI 的可维护性和可预测性。

对于 CI 维护者和测试开发者,值得浏览以了解新的命名规范;对于一般开发者,无需深入,但可参考 .claude/skills/write-sglang-test/SKILL.md 更新后的测试编写指南。

讨论亮点

没有 review 评论,issue 评论中仅有作者发出的 '/tag-and-rerun-ci' 指令,无技术讨论或争议。

实现拆解

实现方案包括:

1) 更新 GitHub Actions 工作流文件(如 pr-test.yml, pr-test-amd.yml)中的 job IDs;
2) 修改 test/run_suite.py 中的 PER_COMMIT_SUITES 和 suite 列表;
3) 调整所有注册测试文件中的 suite= 参数;
4) 更新文档文件如 .claude/skills/write-sglang-test/SKILL.md 和 test/README.md;
5) 修改 scripts/ci/utils/slash_command_handler.py 中的 rerun-stage 相关映射。

文件 模块 状态 重要度
.github/workflows/pr-test.yml CI/Workflows modified 8.0
.github/workflows/pr-test-amd.yml CI/Workflows modified 7.0
test/run_suite.py Testing modified 6.0
.claude/skills/write-sglang-test/SKILL.md Documentation modified 5.0

关键符号

PER_COMMIT_SUITES (in test/run_suite.py) handle_rerun_stage (in scripts/ci/utils/slash_command_handler.py) register_cuda_ci/register_amd_ci/register_cpu_ci decorators in test files

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

评论区精华

无讨论或争议 other

没有 review 评论,issue 评论中仅有 '/tag-and-rerun-ci' 指令。

结论:PR 被顺利合并,无异议。 · 已解决

风险与影响

主要风险是命名更新可能遗漏某些文件或配置,导致 CI 运行失败(例如,脚本硬编码旧名称)。但 PR 覆盖 283 个文件且声称无逻辑变更,风险较低;需在合并后监控 CI 作业是否正常触发。

对用户无直接影响;对团队,CI 测试套件命名更规范,便于管理和调试;系统层面,提高 CI 配置的一致性和可维护性,减少因命名混乱导致的错误。

命名更新遗漏风险 CI 配置兼容性

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论