Prhub

#28810 [CI] Remove deprecated test/srt legacy CI setup

原始 PR 作者 merrymercy 合并时间 2026-06-21 06:09 文件变更 29 提交数 1 评论 1 代码增减 +2 / -5773

执行摘要

删除遗留 test/srt CI 设置,迁移至注册制测试框架

PR body 指出 test/srt/run_suite.py 是遗留 CI runner,已被 registry-based test/run_suite.py 取代,并携带弃用通知。它不再被任何 GitHub Actions 工作流、脚本或导入使用。删除前已验证冗余覆盖:test/srt/cpu/ 完全冗余(已镜像在 test/registered/cpu/),test/srt/run_suite.py 无任何引用,test/srt/mem_cache/test_int8_checkpoint_store.py 孤立未运行。

值得快速合并,这是一项干净的基础设施清理。建议项目成员阅读 test/README.md 了解注册制测试框架(test/run_suite.py),未来添加测试时遵循新流程。

讨论亮点

唯一 review 来自 gemini-code-assist[bot] 的自动评论,确认 PR 内容并包含自身弃用通知,无人工讨论或质疑。

实现拆解

  1. 删除目录:移除整个 test/srt/ 文件夹,包括 run_suite.pycpu/mem_cache/ 等子目录及所有文件。
  2. 更新文档:在 test/README.md 中删除 srt/ 条目,并修复过时的文件引用(将 test/srt/nightly/test_vlms_mmmu_eval.py 改为 test/registered/eval/test_vlms_mmmu_eval.py)。
  3. 调整 CI 脚本scripts/ci/amd/amd_ci_exec.sh 中默认工作目录从 test/srt 改为 test,以对齐新测试布局。
  4. 同步白名单scripts/code_sync/utils.py 中移除 test/srt,防止同步工具尝试复制已删除文件。
    无额外测试或配置变更。
文件 模块 状态 重要度
test/srt/run_suite.py 历史 CI removed 4.0
test/README.md 文档 modified 2.0
scripts/ci/amd/amd_ci_exec.sh AMD 脚本 modified 3.0
scripts/code_sync/utils.py 同步配置 modified 2.0

关键符号

layernorm rotary_emb native_torch native_torch_int8 TestQKVProjWithROPE test_bf16_qkv_proj_with_rope test_int8_qkv_proj_with_rope test_fp8_qkv_proj_with_rope parametrize decorator

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

评论区精华

自动代码审查 other

gemini-code-assist[bot] 自动评论确认 PR 内容并包含自身弃用通知,无实际反馈。

结论:无人工讨论,自动审查无问题。 · 已解决

风险与影响

低风险。作者已通过 grep 确认无工作流、脚本或导入引用被删除的 runner。CPU 测试覆盖已由 test/registered/cpu/ 保留(且为超集)。但存在未发现间接引用的可能性,例如外部工具或遗留分支。AMD CI 脚本默认工作目录变更可能影响未显式指定 -w 参数的调用者,但 PR 已同步更新。建议合并后短期内监控 CI 稳定性。

对最终用户无影响。对开发者和 CI 维护者而言,消除了遗留代码债务,减少维护成本,简化了测试框架结构。AMD CI 调用者若依赖旧默认目录可能需调整,但新默认值已与注册制对齐。

无工作流引用 CPU 测试覆盖保留 AMD 脚本默认目录变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论