# PR #28810 完整报告

- 仓库：`sgl-project/sglang`
- 标题：[CI] Remove deprecated test/srt legacy CI setup
- 合并时间：2026-06-21 06:09
- 原文链接：http://prhub.com.cn/sgl-project/sglang/pull/28810

---

# 执行摘要

- 一句话：删除遗留 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 孤立未运行。

# 实现拆解

1. **删除目录**：移除整个 `test/srt/` 文件夹，包括 `run_suite.py`、`cpu/`、`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；类别 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 脚本默认目录变更

# 关联脉络

- 暂无明显关联 PR