Prhub

#22653 [Docker] Remove flashinfer cache copy

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

执行摘要

移除 Dockerfile 中已失效的 flashinfer 缓存复制指令,修复 CI 构建失败。

PR body直接引用CI失败日志链接(https://github.com/sgl-project/sglang/actions/runs/24320223003/job/71004907738#step:8:8520),并说明原因:在#22491移除了手动flashinfer cubin下载步骤后,/root/.cache/flashinfer目录不再被创建,导致Dockerfile中的COPY指令因源路径不存在而失败。这是一个明确的修复性变更,旨在解决CI构建中断问题。

此PR变更简单,无需精读。对于关注CI/Docker基础设施的工程师,可快速浏览以了解flashinfer缓存清理的后续动作。建议结合#22491理解完整上下文。

讨论亮点

review讨论极少,仅gemini-code-assist[bot]自动评论确认变更内容,无人工review或争议点。这表明变更简单直接,团队对修复必要性有共识。

实现拆解

仅修改一个文件:docker/Dockerfile。关键改动是删除第428行复制/root/.cache/flashinfer目录的指令(原行:COPY --from=flashinfer_cache /root/.cache/flashinfer /root/.cache/flashinfer),同时更新注释以反映变更。保留复制jit-cache包的指令(COPY --from=flashinfer_cache /flashinfer_jit_output/ /usr/local/lib/python3.12/dist-packages/),确保flashinfer的JIT编译输出仍被正确安装。

文件 模块 状态 重要度
docker/Dockerfile infra/docker modified 8.0

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

评论区精华

变更确认 other

gemini-code-assist[bot] 自动评论确认 PR 更新了 Dockerfile,移除 flashinfer 缓存复制而保留 jit-cache 包复制。

结论:无争议,变更被接受。 · 已解决

风险与影响

风险极低:

  1. 回归风险:删除的是已失效的复制指令,不会影响现有功能;保留的jit-cache包复制确保flashinfer依赖仍被安装。
  2. 兼容性:与#22491的变更完全对齐,无breaking change。
  3. 安全:不涉及敏感逻辑。
    唯一潜在风险是如果未来flashinfer缓存机制恢复,此变更可能需要调整,但当前上下文无此迹象。

影响范围有限:

  1. 对用户:无直接影响,仅涉及CI/Docker构建流程。
  2. 对系统:修复CI构建失败,确保Docker镜像能正常生成。
  3. 对团队:消除CI噪音,提升开发效率。
    影响程度为低,属于基础设施维护。

关联 Issue

#22491 [CI/Docker] Clean up redundant flashinfer cubin downloads

完整报告

参与讨论