Prhub

#25329 Skip CI tests added in #24816 (broken on main)

原始 PR 作者 ch-wan 合并时间 2026-05-15 09:12 文件变更 2 提交数 1 评论 1 代码增减 +7 / -1

执行摘要

临时跳过 CI 中断的测试

PR #24816 添加的 FlashInfer SM90 cutlass MXFP4 单元测试和 DSv4-Flash FP4 FlashInferCutlass 端到端测试目前在 main 分支上运行失败,导致 CI 不能通过。作者希望暂时跳过这些测试,保持 CI 绿色,同时不影响已有的测试覆盖。

该 PR 是临时性 CI 维护变更,技术含量低。建议合并以保持 CI 绿色,但应尽快跟进根本原因修复。

讨论亮点

该 PR 没有审核评论或讨论。作者在 body 中说明了动机:PR #24816 添加的测试在 main 上已损坏,需要暂时禁用以保持 CI 绿色。

实现拆解

  1. 禁用整个 MXFP4 单元测试文件:在 test/registered/unit/layers/quantization/test_mxfp4_sm90_cutlass.py 中,将 register_cuda_ci(est_time=120, stage="stage-b", runner_config="1-gpu-large") 改为 register_cuda_ci(est_time=120, stage="stage-b", runner_config="1-gpu-large", disabled="broken on main, see #24816")。该参数是 SGLang 标准约定,使该文件不再被 CI 调度。
  2. 禁用单个测试类:在 test/registered/dsv4/test_deepseek_v4_flash_fp4_h200.py 中,在 TestDSV4FlashFP4H200FlashInferCutlass 类前添加 @unittest.skip("broken on main, see #24816"),仅跳过该损坏的类,而原有的 TestDSV4FlashFP4H200 (Marlin 路径) 继续运行。
文件 模块 状态 重要度
test/registered/unit/layers/quantization/test_mxfp4_sm90_cutlass.py 量化测试 modified 4.27
test/registered/dsv4/test_deepseek_v4_flash_fp4_h200.py DeepSeek 测试 modified 3.18

关键源码片段

test/registered/unit/layers/quantization/test_mxfp4_sm90_cutlass.py test-coverage

通过 register_cuda_ci 的 disabled 参数禁用整个 MXFP4 单元测试文件,避免 CI 调度。

"""Unit test for the SM90 cutlass MXFP4 path in :class:`Mxfp4MoEMethod`.
...
"""from sglang.test.ci.ci_register import register_cuda_ci# 变更前:register_cuda_ci(est_time=120, stage="stage-b", runner_config="1-gpu-large")
# 变更后:添加 disabled 参数,使该文件被 CI 排除
register_cuda_ci(
    est_time=120,
    stage="stage-b",
    runner_config="1-gpu-large",
    disabled="broken on main, see #24816", # 标准跳过机制
)
test/registered/dsv4/test_deepseek_v4_flash_fp4_h200.py test-coverage

为损坏的 TestDSV4FlashFP4H200FlashInferCutlass 类添加 @unittest.skip,仅跳过该类而不影响 Marlin 路径。

@unittest.skip("broken on main, see #24816") # 新增行,跳过该损坏的测试类
@unittest.skipUnless(
    _flashinfer_has_sm90_cutlass_mxfp4(),
    "FlashInfer build lacks SM90 mixed-input MXFP4 helpers (PR #3084, >= 0.6.11)",
)
class TestDSV4FlashFP4H200FlashInferCutlass(ServerSanityMixin, CustomTestCase):
    """FlashInfer SM90 mixed-input cutlass MXFP4 backend..."""
    ...

评论区精华

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

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

风险与影响

风险极低。变更仅限于测试注册/跳过逻辑,不涉及任何生产代码。禁用的测试文件是 PR #24816 新添加的,原 main 分支没有这些测试,因此禁用不会降低现有测试覆盖率。需要确保后续修复后重新启用这些测试。

影响范围:仅 CI 流程,不涉及用户或系统行为。
影响程度:低。临时禁用了两个与 FlashInfer SM90 MXFP4 相关的测试套件,不影响现有的 Marlin 路径测试。
后续行动:需要一个后续修复 PR 解决根本问题并重新启用这些测试。

临时跳过 测试覆盖缺失 需要后续修复

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论