#42196 [CI] Trigger LoRA test when changing MoE code.

原始 PR 作者 jeejeelee 合并时间 2026-05-10 16:26 文件变更 1 提交数 4 评论 2 代码增减 +1 / -0

执行摘要

PR body 明确指出目的为 'Avoid MoE LoRA regression when changing MoE code.'，确保 MoE 代码变更能够触发 LoRA 测试，防止回归问题。

该 PR 为简单的 CI 配置变更，实现意图明确。建议关注 bot 评论中提到的路径统一性问题，可在后续清理或调整 CI 配置时统一格式。对于 LoRA %N 步骤是否也应添加该依赖，可根据实际回归情况决定。

讨论亮点

bot 评论指出新增路径尾随斜杠与 vllm/lora 格式不一致，可能导致 CI 路径匹配问题；同时建议对 LoRA %N 步骤也添加相同依赖以覆盖非 TP 测试。但该评论未被采纳，最终合入时仍保留了尾随斜杠，也未修改 LoRA %N 步骤。

修改 .buildkite/test_areas/lora.yaml 中 LoRA TP (Distributed) 步骤的 source_file_dependencies 列表，添加 - vllm/model_executor/layers/fused_moe/ 条目。
其他步骤（如 LoRA %N）未做修改，仅 LoRA TP（分布式）步骤新增依赖。
使 CI 在检测到 vllm/model_executor/layers/fused_moe/ 目录下文件变更时，自动触发 LoRA TP 分布式测试。

文件	模块	状态	重要度
`.buildkite/test_areas/lora.yaml`	CI 配置	modified	3.16

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

路径格式统一及覆盖范围 style

bot 评论指出新增的 `vllm/model_executor/layers/fused_moe/` 尾随斜杠与其他条目（如 `vllm/lora`）不一致，可能导致 CI 路径匹配问题；并建议在 `LoRA %N` 步骤中也添加此依赖，以确保非 TP 测试也能被 MoE 变更触发。

结论：未采纳建议：尾随斜杠保留，`LoRA %N` 步骤未添加此依赖。PR 仍被批准合并。 · 已解决

风险极低：仅新增 CI 触发条件，不涉及逻辑变更。尾随斜杠在不同 CI 系统中的行为可能略有差异，但已知 source_file_dependencies 支持目录检测，尾随斜杠通常不会导致错误。此外，LoRA %N 步骤未加入此路径，可能遗漏某些 MoE 相关回归的触发。

对开发者：MoE 代码变更后会自动运行 LoRA TP 分布式测试，增加 CI 负担但降低回归风险。对系统：CI 运行频率微小增加。对用户：无直接影响。

CI 配置变更

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

一句话：LoRA 测试触发条件增加 MoE 路径
推荐动作：该 PR 为简单的 CI 配置变更，实现意图明确。建议关注 bot 评论中提到的路径统一性问题，可在后续清理或调整 CI 配置时统一格式。对于 LoRA %N 步骤是否也应添加该依赖，可根据实际回归情况决定。

PR body 明确指出目的为 'Avoid MoE LoRA regression when changing MoE code.'，确保 MoE 代码变更能够触发 LoRA 测试，防止回归问题。

修改 .buildkite/test_areas/lora.yaml 中 LoRA TP (Distributed) 步骤的 source_file_dependencies 列表，添加 - vllm/model_executor/layers/fused_moe/ 条目。
其他步骤（如 LoRA %N）未做修改，仅 LoRA TP（分布式）步骤新增依赖。
使 CI 在检测到 vllm/model_executor/layers/fused_moe/ 目录下文件变更时，自动触发 LoRA TP 分布式测试。

关键文件：

.buildkite/test_areas/lora.yaml（模块 CI 配置；类别 config；类型 configuration）: 核心变更文件：在 LoRA TP 分布式测试步骤的 source_file_dependencies 中新增了 vllm/model_executor/layers/fused_moe/，使 MoE 代码变更时自动触发 LoRA 测试。

关键符号：未识别

风险：风险极低：仅新增 CI 触发条件，不涉及逻辑变更。尾随斜杠在不同 CI 系统中的行为可能略有差异，但已知 source_file_dependencies 支持目录检测，尾随斜杠通常不会导致错误。此外，LoRA %N 步骤未加入此路径，可能遗漏某些 MoE 相关回归的触发。
影响：对开发者：MoE 代码变更后会自动运行 LoRA TP 分布式测试，增加 CI 负担但降低回归风险。对系统：CI 运行频率微小增加。对用户：无直接影响。
风险标记：CI 配置变更