Prhub

#37025 [CI] Add reasoning parser tests to CI

原始 PR 作者 sfeng33 合并时间 2026-04-08 08:57 文件变更 1 提交数 6 评论 5 代码增减 +2 / -0

执行摘要

将推理解析器测试添加到 CI 流水线,并暂时忽略已知失败测试。

根据PR body,目的是'Add reasoning parser tests to CI to prevent future regressions',并关联issue #37022和#37023跟踪已知测试失败。这确保推理解析器功能在后续变更中不会意外引入回归,同时处理当前测试失败问题。

这是一个简单的CI变更,无需精读代码。值得关注的是讨论中关于pytest路径和工作目录的决策(反映CI环境细节),以及优先合并以保护通过测试的策略(平衡及时保护与问题延迟处理)。

讨论亮点

讨论焦点包括:

  1. gemini-code-assist[bot]指出pytest命令中--ignore路径可能错误,认为应从仓库根目录指定,但sfeng33澄清由于工作目录是/vllm-workspace/tests,路径reasoning/...正确。
  2. chaunceyjiang建议等待测试修复后再合并,但sfeng33主张先合并以保护其他316个通过测试,避免进一步回归,最终bbrowning批准该策略。

实现拆解

实现很简单,只修改了CI配置文件.buildkite/test_areas/misc.yaml:首先在测试路径列表中添加tests/reasoning/,然后在pytest命令中添加运行推理测试的步骤,并指定忽略三个失败测试文件(test_seedoss_reasoning_parser.pytest_glm4_moe_reasoning_parser.pytest_gemma4_reasoning_parser.py)。

文件 模块 状态 重要度
.buildkite/test_areas/misc.yaml CI/Infrastructure modified 4.0

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

评论区精华

pytest 命令路径正确性 正确性

gemini-code-assist[bot] 指出 `--ignore=reasoning/test_seedoss_reasoning_parser.py` 等路径可能错误,因为 pytest 根目录是仓库根目录,但 sfeng33 反驳工作目录是 `/vllm-workspace/tests`,所以路径正确。

结论:sfeng33 的路径正确,pytest 命令将按预期忽略指定测试文件,确保 CI 运行成功。 · 已解决

合并策略:是否等待测试修复 设计

chaunceyjiang 建议等待 `test_glm4_moe_reasoning_parser.py` 和 `test_seedoss_reasoning_parser.py` 修复后再合并,但 sfeng33 主张先合并以保护其他 316 个通过测试,避免进一步回归。

结论:团队决定先合并 PR,后续通过关联 issue #37022 和 #37023 修复失败测试。 · 已解决

风险与影响

风险较低:CI配置更改可能影响测试运行,但路径争议已解决;忽略失败测试可能掩盖回归,但有关联issue跟踪确保后续修复;无性能或安全风险,因为只影响测试流程。

直接影响CI流程,确保推理解析器测试在每次CI中运行,提供回归保护;暂时忽略失败测试,不影响现有CI状态,但需后续修复issue以全面测试;对用户和系统无直接影响。

CI 配置更改 忽略失败测试

关联 Issue

#37022 [CI Failure]: Seedoss reasoning parser test failure
#37023 [CI Failure]: GLM4 moe reasoning parser test failure

完整报告

参与讨论