执行摘要
- 一句话:删除遗留 test/srt CI 设置,迁移至注册制测试框架
- 推荐动作:值得快速合并,这是一项干净的基础设施清理。建议项目成员阅读
test/README.md 了解注册制测试框架(test/run_suite.py),未来添加测试时遵循新流程。
功能与动机
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/srt/ 文件夹,包括 run_suite.py、cpu/、mem_cache/ 等子目录及所有文件。
- 更新文档:在
test/README.md 中删除 srt/ 条目,并修复过时的文件引用(将 test/srt/nightly/test_vlms_mmmu_eval.py 改为 test/registered/eval/test_vlms_mmmu_eval.py)。
- 调整 CI 脚本:
scripts/ci/amd/amd_ci_exec.sh 中默认工作目录从 test/srt 改为 test,以对齐新测试布局。
- 同步白名单:
scripts/code_sync/utils.py 中移除 test/srt,防止同步工具尝试复制已删除文件。
无额外测试或配置变更。
关键文件:
test/srt/run_suite.py(模块 历史CI;类别 test;类型 deletion): 被删除的入口 runner,表明整个 legacy 体系的移除。
test/README.md(模块 文档;类别 docs;类型 configuration): 文档更新以反映新目录结构和文件引用。
scripts/ci/amd/amd_ci_exec.sh(模块 AMD脚本;类别 infra;类型 configuration): AMD CI 脚本默认工作目录变更,影响未指定 -w 的调用。
scripts/code_sync/utils.py(模块 同步配置;类别 infra;类型 configuration): 从同步白名单中移除 test/srt,避免同步无效路径。
关键符号: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
评论区精华
唯一 review 来自 gemini-code-assist[bot] 的自动评论,确认 PR 内容并包含自身弃用通知,无人工讨论或质疑。
- 自动代码审查 (other): 无人工讨论,自动审查无问题。
风险与影响
- 风险:低风险。作者已通过 grep 确认无工作流、脚本或导入引用被删除的 runner。CPU 测试覆盖已由
test/registered/cpu/ 保留(且为超集)。但存在未发现间接引用的可能性,例如外部工具或遗留分支。AMD CI 脚本默认工作目录变更可能影响未显式指定 -w 参数的调用者,但 PR 已同步更新。建议合并后短期内监控 CI 稳定性。
- 影响:对最终用户无影响。对开发者和 CI 维护者而言,消除了遗留代码债务,减少维护成本,简化了测试框架结构。AMD CI 调用者若依赖旧默认目录可能需调整,但新默认值已与注册制对齐。
- 风险标记:无工作流引用, CPU测试覆盖保留, AMD脚本默认目录变更
关联脉络
参与讨论