Prhub

#32049 [CI] Fix Marlin MoE test ServerArgs initialization

原始 PR 作者 mmangkad 合并时间 2026-07-22 20:16 文件变更 1 提交数 1 评论 3 代码增减 +1 / -1

执行摘要

修复 Marlin MoE 测试中 ServerArgs 初始化方式

PR #31810 重构了配置加载流程,原先的 object.__new__(ServerArgs) 不再符合新的只读配置要求,导致 Marlin MoE 测试在 CI 中失败。此 PR 通过使用带参数的 ServerArgs 构造来修复测试。

建议合并。变更虽小,但消除了由配置重构引发的测试失败,有助于保持 CI 的稳定性。

讨论亮点

无 review 评论。

实现拆解

  1. test/registered/quant/test_marlin_moe.py 中,将第 17 行的 set_global_server_args_for_scheduler(object.__new__(ServerArgs)) 替换为 set_global_server_args_for_scheduler(ServerArgs(model_path="dummy"))
  2. 这一修改确保 ServerArgs 实例经过正常构造,从而满足配置命名空间相关的检查。
文件 模块 状态 重要度
test/registered/quant/test_marlin_moe.py 测试 modified 3.25

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

评论区精华

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

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

风险与影响

风险极低:仅修改一行测试初始化代码,不会影响生产逻辑。若 ServerArgs(model_path="dummy") 后续触发额外的参数校验,理论上可能导致其他测试失败,但该风险几乎为零。

影响范围仅限于 Marlin MoE 这一 CI 测试用例,修复后该测试将恢复正常运行。无其他系统影响。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论