Prhub

#22137 Remove flaky TestToolChoiceLfm2Moe from test_tool_choice

原始 PR 作者 hnyls2002 合并时间 2026-04-05 13:53 文件变更 1 提交数 1 评论 1 代码增减 +0 / -28

test run-ci consistency

执行摘要

删除不稳定的 LFM2-MoE 模型工具调用测试类，解决 CI flaky 问题。

PR body明确指出LiquidAI/LFM2-8B-A1B模型在工具调用方面不可靠，导致test_tool_choice_specific_function_non_streaming测试频繁失败（如提供的CI运行链接所示）。移除该测试类是为了解决CI flaky问题，确保测试稳定性。

该PR变更简单，适合快速浏览以了解CI优化策略。值得关注的是团队如何处理flaky测试：通过移除不可靠的模型特定测试，同时确保核心逻辑（lfm2解析器）仍有其他测试覆盖。

讨论亮点

review中仅gemini-code-assist[bot]发表了一条评论，确认PR移除了针对LFM2-MoE模型的测试类，没有提供实质性反馈。没有其他review评论或争议点，表明变更简单直接，团队对移除flaky测试达成共识。

实现拆解

仅修改一个文件test/registered/openai_server/function_call/test_tool_choice.py，完全删除TestToolChoiceLfm2Moe类（28行代码）。该类原本继承自TestToolChoiceLlama32，使用lfm2解析器，并配置了flaky_tests集合。移除后，测试套件中不再包含针对LFM2-MoE模型的工具调用测试。

文件 模块 状态 重要度
test/registered/openai_server/function_call/test_tool_choice.py openai_server/function_call 测试 modified 5.0

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

评论区精华

移除 flaky 测试的合理性 测试

gemini-code-assist[bot] 仅确认 PR 移除了测试类，没有深入讨论。

结论：团队默认接受移除 flaky 测试以提升 CI 稳定性。 · 已解决

风险与影响

风险较低：1. 移除测试可能掩盖LFM2-MoE模型在工具调用方面的潜在问题，但PR body指出lfm2解析器逻辑已通过test_function_call_parser.py独立覆盖，降低了回归风险。2. 如果未来LFM2-MoE模型工具调用功能有变化，缺少专门测试可能无法及时发现兼容性问题。

对用户无直接影响，仅影响内部CI稳定性。移除flaky测试可减少CI失败频率，提升开发效率。对系统无功能影响，因为测试本身不改变产品代码。团队可更专注于稳定测试，但需注意潜在模型特定问题的监控。

潜在模型特定问题掩盖

关联 Issue

未识别关联 Issue

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

完整报告

Markdown PDF

执行摘要

  • 一句话：删除不稳定的LFM2-MoE模型工具调用测试类，解决CI flaky问题。
  • 推荐动作：该PR变更简单，适合快速浏览以了解CI优化策略。值得关注的是团队如何处理flaky测试：通过移除不可靠的模型特定测试，同时确保核心逻辑（lfm2解析器）仍有其他测试覆盖。

功能与动机

PR body明确指出LiquidAI/LFM2-8B-A1B模型在工具调用方面不可靠，导致test_tool_choice_specific_function_non_streaming测试频繁失败（如提供的CI运行链接所示）。移除该测试类是为了解决CI flaky问题，确保测试稳定性。

实现拆解

仅修改一个文件test/registered/openai_server/function_call/test_tool_choice.py，完全删除TestToolChoiceLfm2Moe类（28行代码）。该类原本继承自TestToolChoiceLlama32，使用lfm2解析器，并配置了flaky_tests集合。移除后，测试套件中不再包含针对LFM2-MoE模型的工具调用测试。

关键文件：

  • test/registered/openai_server/function_call/test_tool_choice.py（模块 openai_server/function_call测试）: 唯一修改的文件，完全删除TestToolChoiceLfm2Moe测试类，直接解决CI flaky问题。

关键符号：未识别

评论区精华

review中仅gemini-code-assist[bot]发表了一条评论，确认PR移除了针对LFM2-MoE模型的测试类，没有提供实质性反馈。没有其他review评论或争议点，表明变更简单直接，团队对移除flaky测试达成共识。

  • 移除flaky测试的合理性 (testing): 团队默认接受移除flaky测试以提升CI稳定性。

风险与影响

  • 风险：风险较低：1. 移除测试可能掩盖LFM2-MoE模型在工具调用方面的潜在问题，但PR body指出lfm2解析器逻辑已通过test_function_call_parser.py独立覆盖，降低了回归风险。2. 如果未来LFM2-MoE模型工具调用功能有变化，缺少专门测试可能无法及时发现兼容性问题。
  • 影响：对用户无直接影响，仅影响内部CI稳定性。移除flaky测试可减少CI失败频率，提升开发效率。对系统无功能影响，因为测试本身不改变产品代码。团队可更专注于稳定测试，但需注意潜在模型特定问题的监控。
  • 风险标记：潜在模型特定问题掩盖

关联脉络

  • PR #22138 [CI]Temporary ban auto benchmark tool test: 类似地处理CI不稳定性问题，通过临时禁用测试解决flaky问题。
  • PR #22100 Relax spec decoding accuracy threshold to fix flaky test: 同为修复CI flaky测试的策略，但通过放宽阈值而非移除测试。
  • PR #21230 Add LFM2-VL (Liquid Foundation Model 2 Vision-Language) support: 添加LFM2-VL模型支持，与本PR涉及的LFM2-MoE模型同属LFM2系列，可能共享类似测试问题。

参与讨论