# PR #26764 完整报告

- 仓库：`sgl-project/sglang`
- 标题：[Hicache]: skip flaky test
- 合并时间：2026-05-30 15:16
- 原文链接：http://prhub.com.cn/sgl-project/sglang/pull/26764

---

# 执行摘要

- 一句话：跳过 HiCache 不稳定测试避免 CI 失败
- 推荐动作：作为临时措施合理，但建议创建 Issue 跟踪 flaky 测试的根因（如时序依赖或硬件差异），并计划后续修复后移除 skipIf。精读价值低。

# 功能与动机

HiCache 的 `test_eval_accuracy` 测试在 CI 环境中频繁失败，主要原因是缓存 flush 后远端存储访问延迟不稳定，导致精度阈值断言随机失败。作者通过添加 `@unittest.skipIf` 跳过该测试，避免 CI 阻塞，同时允许开发者在本地运行调试。

# 实现拆解

在 `test/registered/hicache/test_hicache_storage_file_backend.py` 的 `TestHiCacheStorageAccuracy` 类中，为 `test_eval_accuracy` 方法添加 `@unittest.skipIf(is_in_ci(), "To skip flaky test")` 装饰器。当环境变量 `CI` 存在时，该测试自动跳过。

关键文件：
- `test/registered/hicache/test_hicache_storage_file_backend.py`（模块 测试；类别 test；类型 test-coverage）: 唯一变更文件，添加 @unittest.skipIf 跳过 flaky 测试。

关键符号：未识别

## 关键源码片段

### `test/registered/hicache/test_hicache_storage_file_backend.py`

唯一变更文件，添加 @unittest.skipIf 跳过 flaky 测试。

```python
# test/registered/hicache/test_hicache_storage_file_backend.py
# 在 CI 环境中跳过这个容易因缓存同步延迟而失败的测试
@unittest.skipIf(is_in_ci(), "To skip flaky test")
def test_eval_accuracy(self):
    """Test eval accuracy with cache persistence across cache flushes"""
    run_eval_accuracy_test(self)

```

# 评论区精华

无 review 评论。PR body 使用标准模板，未提及具体讨论。作者通过 `/rerun-group hicache` 触发 CI 重跑，结果均通过（跳过该测试）。

- 暂无高价值评论线程

# 风险与影响

- 风险：低风险。仅影响 CI 测试执行，不影响生产代码。跳过测试意味着 CI 不再覆盖 HiCache 精度验证，但开发者仍可在本地手动运行。若该测试的 flaky 根本原因后续修复，需记得移除 skipIf。
- 影响：影响范围：仅 CI 流水线。`test_eval_accuracy` 在 CI 中不再执行，HiCache 精度回归风险由本地运行或专用测试覆盖。团队需关注该测试的长期稳定性，计划修复而非永久跳过。
- 风险标记：测试跳过 , 临时措施

# 关联脉络

- 暂无明显关联 PR