Prhub

#38796 docker(xpu): install libssl-dev so JIT hicache_hash_cpp builds

原始 PR 作者 arathi-hlab 合并时间 2026-09-10 14:37 文件变更 2 提交数 4 评论 0 代码增减 +13 / -0

执行摘要

为 XPU Docker 镜像添加 libssl-dev 依赖,修复 JIT 构建失败。

XPU CI 中执行 Mamba radix 缓存相关测试(如 test_mamba_radix_cache_kv_events)时失败,原因是 JIT 构建的 hicache_hash_cpp C++ 扩展需要 OpenSSL 头文件 <openssl/sha.h>,但 XPU Docker 镜像中缺失此依赖。这与 PR #34767 中修复 Xeon 镜像的问题相同。

此 PR 是必要的修复,解决了 CI 失败问题。建议关注临时桥接修复的后续清理,确保在 Docker 镜像更新后移除 CI 工作流中的临时安装步骤。

讨论亮点

无具体讨论。PR 被直接批准合并。

实现拆解

  1. Dockerfile 依赖添加:在 docker/xpu.Dockerfile 的 apt-get install 命令中添加 libssl-dev 包,确保新构建的镜像包含 OpenSSL 开发头文件。
  2. CI 工作流桥接修复:在 .github/workflows/pr-test-xpu.yml 的两个测试作业中,添加在运行容器内安装 libssl-dev 的步骤。这是因为 CI 使用的是预构建的 intel/sglang-dev:latest 镜像,Dockerfile 的更改尚未生效,需要临时在容器中安装依赖以确保当前 PR 的测试能通过。
文件 模块 状态 重要度
docker/xpu.Dockerfile Docker 配置 modified 3.04
.github/workflows/pr-test-xpu.yml CI 工作流 modified 3.29

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

评论区精华

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

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

风险与影响

风险较低。此变更仅影响 XPU Docker 镜像和 CI 工作流,不涉及核心运行时代码。主要风险是如果临时桥接修复未按计划移除,可能导致 CI 脚本冗余。

直接影响 XPU CI 环境,使依赖 OpenSSL 的 JIT 编译扩展(如 hicache_hash_cpp)能够成功构建。间接影响使用 XPU Docker 镜像的开发者,确保环境一致性。

临时修复待清理

关联 Issue

#34767 [CPU] Fix native KV hash compilation in Xeon image

完整报告

参与讨论