执行摘要
- 一句话:拆分 distributed 测试目录为主题子文件夹
- 推荐动作:此 PR 是一次测试基础设施的清理,值得了解测试组织方式,但技术洞察有限。主要价值在于维护测试目录的可管理性。
功能与动机
PR body 说明:'Eliminate the catch-all test/registered/distributed/ folder; route each test to the topic folder that matches its actual subject'。目的是消除通配文件夹,按主题组织测试,便于维护和导航。
实现拆解
-
创建目标文件夹:在 test/registered/ 下新增 disaggregation/、model_loading/、backends/、dp_attn/、dp_engine/、pp/、unit/distributed/ 等文件夹。
-
移动测试文件:将原本位于 test/registered/distributed/ 的 15 个测试文件按实际主题移动到对应文件夹,包括分离部署、模型加载、后端、数据并行注意力、数据并行引擎、流水线并行和并行状态单元测试。
-
更新注释:修改 test/srt/run_suite.py 第 40 行的注释,将 test/registered/distributed/ 改为 test/registered/,以准确反映 AMD 的 per-commit-4-gpu-amd 测试套件的迁移情况。
-
保持 CI 配置不变:所有 register_cuda_ci 等 CI 注册注解直接内嵌在测试文件中,不依赖路径,因此移动后 CI 调度不受影响。
关键文件:
test/srt/run_suite.py(模块 测试套件;类别 test;类型 test-coverage): 唯一一个不是纯重命名的文件,包含注释更新,反映迁移路径变化。
test/registered/dp_engine/test_data_parallelism.py(模块 数据并行引擎;类别 test;类型 rename-or-move): 代表数据并行引擎测试迁移,是分布式测试中重要的并行策略测试。
test/registered/model_loading/test_load_weights_from_remote_instance.py(模块 模型加载;类别 test;类型 rename-or-move): 代表模型加载测试迁移,涉及远程加载权重,关键功能测试。
test/registered/disaggregation/test_disaggregation_pp.py(模块 分离部署;类别 test;类型 rename-or-move): 代表分离部署中的流水线并行测试迁移,是分布式部署核心测试之一。
test/registered/backends/test_flashinfer_fusion_preflight.py(模块 后端;类别 test;类型 rename-or-move): 代表后端测试迁移,涉及 flashinfer 融合预检,与底层推理后端相关。
关键符号:未识别
评论区精华
只有一条 review comment 来自 gemini-code-assist[bot],内容为无反馈。无实质讨论。
风险与影响
- 风险:风险极低。仅涉及测试文件移动和一行注释更新,不影响产品代码。潜在风险包括:如果外部工具或文档硬编码了旧路径,可能需要同步更新。但仓库内路径依赖主要集中在 CI 注册注解(在文件内),已声明不受影响。
- 影响:对用户:无直接影响。对开发者:测试目录结构更清晰,易于查找特定主题的测试。对 CI 系统:路径变化可能导致自定义脚本暂时失效,但 CI 框架使用注解而非路径调度,影响可控。
- 风险标记:暂无
关联脉络
参与讨论