执行摘要
临时禁用不稳定的 NIXL 测试
PR body 中明确指出:"Seeing these errors on main",并提供了两个 CI 失败的链接。作者希望临时禁用 NIXL 测试,直到后端稳定,以免阻塞主线开发。
这是一个紧急止血的临时变更,适合快速合并。建议团队在后续 PR 中优先解决 NIXL 后端稳定性问题,并恢复测试覆盖。
无 review 讨论,仅作者自行合并。作者在评论中提供了 CI 通过的截图,确认修改生效。
PR body 中明确指出:"Seeing these errors on main",并提供了两个 CI 失败的链接。作者希望临时禁用 NIXL 测试,直到后端稳定,以免阻塞主线开发。
这是一个紧急止血的临时变更,适合快速合并。建议团队在后续 PR 中优先解决 NIXL 后端稳定性问题,并恢复测试覆盖。
无 review 讨论,仅作者自行合并。作者在评论中提供了 CI 通过的截图,确认修改生效。
test/registered/distributed/test_disaggregation_decode_radix_cache.py 中,将 TestDisaggregationDecodeRadixCacheNixl 类的 @unittest.skipUnless 装饰器替换为 @unittest.skip("Temporarily disabled until nixl backend is stable.")。TestDisaggregationDecodeRadixCacheMooncake 及其它代码未受影响。| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
test/registered/distributed/test_disaggregation_decode_radix_cache.py |
测试 | modified | 3.49 |
test/registered/distributed/test_disaggregation_decode_radix_cache.py
test-coverage
这是唯一修改的文件,通过修改装饰器临时禁用了 NIXL 后端的 decode radix cache 测试。
# 变更前:仅在 CI 或本地有 NIXL 环境时运行
# @unittest.skipUnless(
# is_in_ci() or _has_nixl(),
# "NIXL is required for decode radix cache disaggregation coverage.",
# )
# 变更后:始终跳过,直到 NIXL 后端稳定
@unittest.skip("Temporarily disabled until nixl backend is stable.")
class TestDisaggregationDecodeRadixCacheNixl(
DisaggregationDecodeRadixCacheTestMixin, PDDisaggregationServerBase
):
transfer_backend_name = "nixl"
当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。
此变更仅禁用一个测试类,不会影响生产代码。风险极低:
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论