Prhub

#29710 [HiSparse]: Skip flaky hisparse-nixl ci

原始 PR 作者 hzh0425 合并时间 2026-06-30 14:04 文件变更 1 提交数 1 评论 5 代码增减 +1 / -4

执行摘要

跳过不稳定的 NIXL HiSparse 测试

HiSparse NIXL 测试在 CI 中不稳定(skip until stabilized),通过跳过 CI 执行来避免频繁失败,同时保留本地手动运行能力。

可合并。作为临时缓解措施,长期应分析测试失败根因并修复。

讨论亮点

无 review 讨论。

实现拆解

  1. 修改测试装饰器:在 test/registered/disaggregation/test_disaggregation_hisparse.py 中,将 TestDisaggregationDSV4HiSparseNixl 类的 @unittest.skipUnless(is_in_ci() or _has_nixl(), ...) 替换为 @unittest.skipIf(is_in_ci(), ...)
  2. 行为变化:变更前,CI 中如果 _has_nixl() 返回 True 则会执行测试;变更后,CI 中始终跳过,非 CI 环境(如本地)仍可运行。
  3. 删除冗余注释:移除了旧的 skipUnless 消息参数。
文件 模块 状态 重要度
test/registered/disaggregation/test_disaggregation_hisparse.py 测试 modified 3.49

关键源码片段

test/registered/disaggregation/test_disaggregation_hisparse.py test-coverage

修改了测试装饰器,跳过了 CI 中不稳定的 HiSparse NIXL 测试。

# 变更前:CI 中如果 NIXL 可用则执行测试
# @unittest.skipUnless(is_in_ci() or _has_nixl(), ...)
# 变更后:CI 中始终跳过,非 CI 环境仍可运行
@unittest.skipIf(is_in_ci(), "Flaky in CI — skip until stabilized.")
class TestDisaggregationDSV4HiSparseNixl(TestDisaggregationDSV4HiSparseBase):
    ...

评论区精华

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

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

风险与影响

低风险。仅修改测试装饰器,不影响任何生产代码。CI 中 HiSparse NIXL 测试覆盖暂时缺失,但测试本身不稳定,跳过可减少 CI 噪音。

  • 用户:无影响。
  • 系统:CI 中断言失败减少。
  • 团队:需后续跟进修复测试稳定性后再恢复 CI 执行。
测试覆盖临时丢失

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论