Prhub

#26240 [Test] split test/registered/distributed/ into topic folders

原始 PR 作者 hnyls2002 合并时间 2026-05-25 08:02 文件变更 16 提交数 1 评论 1 代码增减 +1 / -1

执行摘要

拆分 distributed 测试目录为主题子文件夹

PR body 说明:'Eliminate the catch-all test/registered/distributed/ folder; route each test to the topic folder that matches its actual subject'。目的是消除通配文件夹,按主题组织测试,便于维护和导航。

此 PR 是一次测试基础设施的清理,值得了解测试组织方式,但技术洞察有限。主要价值在于维护测试目录的可管理性。

讨论亮点

只有一条 review comment 来自 gemini-code-assist[bot],内容为无反馈。无实质讨论。

实现拆解

  1. 创建目标文件夹:在 test/registered/ 下新增 disaggregation/model_loading/backends/dp_attn/dp_engine/pp/unit/distributed/ 等文件夹。

  2. 移动测试文件:将原本位于 test/registered/distributed/ 的 15 个测试文件按实际主题移动到对应文件夹,包括分离部署、模型加载、后端、数据并行注意力、数据并行引擎、流水线并行和并行状态单元测试。

  3. 更新注释:修改 test/srt/run_suite.py 第 40 行的注释,将 test/registered/distributed/ 改为 test/registered/,以准确反映 AMD 的 per-commit-4-gpu-amd 测试套件的迁移情况。

  4. 保持 CI 配置不变:所有 register_cuda_ci 等 CI 注册注解直接内嵌在测试文件中,不依赖路径,因此移动后 CI 调度不受影响。

文件 模块 状态 重要度
test/srt/run_suite.py 测试套件 modified 3.42
test/registered/dp_engine/test_data_parallelism.py 数据并行引擎 renamed 3.53
test/registered/model_loading/test_load_weights_from_remote_instance.py 模型加载 renamed 3.53
test/registered/disaggregation/test_disaggregation_pp.py 分离部署 renamed 3.18
test/registered/backends/test_flashinfer_fusion_preflight.py 后端 renamed 3.18

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

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低。仅涉及测试文件移动和一行注释更新,不影响产品代码。潜在风险包括:如果外部工具或文档硬编码了旧路径,可能需要同步更新。但仓库内路径依赖主要集中在 CI 注册注解(在文件内),已声明不受影响。

对用户:无直接影响。对开发者:测试目录结构更清晰,易于查找特定主题的测试。对 CI 系统:路径变化可能导致自定义脚本暂时失效,但 CI 框架使用注解而非路径调度,影响可控。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论