Prhub

#21554 [CI] Remove more redundant PCG tests

原始 PR 作者 mmangkad 合并时间 2026-04-01 07:25 文件变更 3 提交数 1 评论 2 代码增减 +0 / -170

test run-ci refactor

执行摘要

移除 GPT-OSS、Kimi-Linear 和 Qwen3 Next 模型的冗余 PCG 测试文件。

根据PR描述（'Similar to #21485, more PCG tests that are redundant now that PCG is enabled by default.'），动机是PCG（Piecewise CUDA Graph）已默认启用，使得专门测试PCG的用例不再必要。在关联Issue的评论中，Fridge003询问'Have you tested that these models will enable PCG by default?'，作者mmangkad回复'Yes except Qwen3 Next'，并指出Qwen3 Next在模型配置中默认禁用PCG（引用代码行），进一步确认了清理的合理性。

该PR变更简单，不值得深入精读，但可以作为了解PCG默认启用策略和测试清理趋势的参考。建议团队关注类似#21485的PR，以把握测试基础设施的演进方向。

讨论亮点

review讨论主要集中在关联Issue的评论中。Fridge003提问'Have you tested that these models will enable PCG by default?'，作者mmangkad回应'Yes except Qwen3 Next'，并引用模型配置文件中的默认禁用设置，解决了测试覆盖的疑虑。在PR的review中，gemini-code-assist[bot]表示无反馈，Fridge003直接批准，表明变更得到认可，无重大争议。

实现拆解

实现方案非常简单：直接删除了三个测试文件：1) test/registered/models/test_gpt_oss_models_pcg.py，包含TestGptOssPiecewiseCudaGraph类，原本用于测试GPT-OSS模型的PCG功能；2) test/registered/models/test_kimi_linear_models_pcg.py，包含TestKimiLinearPiecewiseCudaGraph类；3) test/registered/models/test_qwen3_next_models_pcg.py，包含TestQwen3NextPiecewiseCudaGraph类。这些文件都通过unittest框架定义测试类，并注册到CI套件中，现在由于PCG默认启用，其逻辑可能已集成到其他通用测试中。

文件 模块 状态 重要度
test/registered/models/test_gpt_oss_models_pcg.py 测试 removed 3.0
test/registered/models/test_kimi_linear_models_pcg.py 测试 removed 3.0
test/registered/models/test_qwen3_next_models_pcg.py 测试 removed 3.0

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

关键符号

TestGptOssPiecewiseCudaGraph TestKimiLinearPiecewiseCudaGraph TestQwen3NextPiecewiseCudaGraph

评论区精华

测试覆盖确认 question

Fridge003 询问 'Have you tested that these models will enable PCG by default?'，作者 mmangkad 回复 'Yes except Qwen3 Next'，并引用模型配置文件中的默认禁用设置。

结论：作者确认除 Qwen3 Next 外，其他模型已测试 PCG 默认启用，解决了疑虑。 · 已解决

风险与影响

风险较低：1) 测试覆盖可能略微减少，但由于PCG已默认启用，这些专用测试的移除不会影响功能正确性；2) 对于Qwen3 Next模型，作者指出其在配置中默认禁用PCG，因此该测试原本可能无效或冗余，但移除后需确保其他测试覆盖PCG相关逻辑；3) 无回归风险，因为变更仅删除文件，不修改生产代码或引入新逻辑。

影响范围有限：1) 对用户无直接影响，这是内部CI优化；2) 系统层面，CI测试执行时间可能减少，因为移除了冗余测试用例；3) 团队开发效率提升，代码库更简洁，减少维护负担；4) 影响程度轻微，属于常规清理工作，不改变核心功能。

测试覆盖减少 配置依赖风险

关联 Issue

未识别关联 Issue

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

完整报告

Markdown PDF

PR 分析报告：移除冗余PCG测试

执行摘要

本PR移除了三个针对GPT-OSS、Kimi-Linear和Qwen3 Next模型的Piecewise CUDA Graph（PCG）专用测试文件，因为PCG功能现已默认启用，这些测试变得冗余。变更简化了CI测试套件，减少维护负担，对用户无直接影响，风险较低。

功能与动机

动机源于PCG已默认启用，使得专门测试PCG的用例不再必要。PR描述指出：“Similar to #21485, more PCG tests that are redundant now that PCG is enabled by default.” 在关联Issue的评论中，Fridge003询问测试覆盖情况，作者mmangkad确认除Qwen3 Next外，其他模型已测试PCG默认启用，Qwen3 Next在配置中默认禁用PCG。

实现拆解

实现非常简单，直接删除了以下三个测试文件：

文件路径 关键内容 状态
test/registered/models/test_gpt_oss_models_pcg.py 包含TestGptOssPiecewiseCudaGraph类，测试GPT-OSS模型的PCG功能。 已删除
test/registered/models/test_kimi_linear_models_pcg.py 包含TestKimiLinearPiecewiseCudaGraph类，测试Kimi-Linear模型的PCG功能。 已删除
test/registered/models/test_qwen3_next_models_pcg.py 包含TestQwen3NextPiecewiseCudaGraph类，测试Qwen3 Next模型的PCG功能。 已删除

这些文件都使用unittest框架，并注册到CI套件（如stage-b-test-2-gpu-large），现在由于PCG默认启用，其逻辑可能已集成到其他通用测试中。

评论区精华

主要讨论发生在关联Issue的评论中：

Fridge003: “Have you tested that these models will enable PCG by default?”
mmangkad: “Yes except Qwen3 Next, not sure why it has a test since it is disabled by default in [配置文件]”

这解决了测试覆盖的疑虑，确保移除不会影响核心功能验证。PR的review中，bot无反馈，Fridge003直接批准，表明变更得到认可。

风险与影响

  • 风险分析：测试覆盖可能略微减少，但由于PCG默认启用，专用测试的移除不影响功能正确性。对于Qwen3 Next模型，需注意配置默认禁用PCG，但其他测试应覆盖相关逻辑。无回归风险。
  • 影响分析：对用户无影响；CI测试执行时间可能减少；团队开发效率提升，代码库更简洁；影响程度轻微，属于常规清理。

关联脉络

  • 相关PR：本PR直接引用#21485（移除冗余PCG测试），同为测试清理工作；另可参考#22058（删除回滚测试），反映仓库中测试基础设施的持续优化趋势。
  • 演进方向：这表明PCG功能已成熟并默认启用，测试策略从专项验证转向集成验证，有助于简化CI流程并聚焦核心测试用例。

参与讨论