Prhub

sgl-project/sglang · 标签视图

标签列表

聚合结果

docker 相关 PR

2026-09-16

#39732 [NPU] fix npu docker workspace directory

原始 PR · 作者 xiedeyantu · 合并时间 2026-09-16 16:51

基础设施 重要性 2.52 洞察度 2.00

NPU Dockerfile 默认工作目录改到仓库根目录

不建议为此 PR 投入深度精读——它就是一行 `WORKDIR` 修正加一行注释,读完 patch 即可理解全部语义;真正值得略作停留的是两个设计层面的信号:(1) 参照同文件里 `TARGETARCH` 的 fail-fast 校验,是否应为 workdir 增加一个轻量断言(例如构建期检查 clone 目录存在),避免 `WORKDIR` 静默创建空目录这一失败模式;(2) 其他 Dockerfile(`docker/Dockerfile`、各平台镜像)是否也应统一到 `/sgl-workspace/sglang`,否则不同镜像的 CWD 约定会继续分叉,给文档和 CI 脚本带来长期维护成本。另外建议关注 `Latest PR Test (Extra)` 的那次失败是否有后续 issue 跟踪。

2026-09-12
基础设施 重要性 2.95 洞察度 3.00

DeepGEMM wheel 构建镜像补装 elfutils 头文件

值得快速阅读,属于典型的 CI 构建依赖补丁,复杂度低。建议关注两点:一是修复后应在真实 release workflow 中复跑确认两个平台都能通过,避免修复一个头文件后暴露下一个缺失;二是未来 DeepJIT 或 DeepGEMM 升级时,将系统依赖检查纳入发布前置校验。

2026-09-11
基础设施 重要性 2.42 洞察度 2.00

NPU 镜像工作目录改为 /sgl-workspace

不值得精读代码,但值得作为“容器镜像契约”类变更的样板看一眼:1)确认改动位置后是否还有 `WORKDIR` 覆盖(否则改动可能不生效);2)确认 `docs/` 与 NPU CI 脚本中没有硬编码 `/workspace`;3)关注 PR 描述里被 ping1jing2 要求补齐的模板章节——这类基础设施 PR 常被当作免检,但路径语义变更恰恰需要在描述中显式声明。对于 NPU 镜像维护者,建议后续补一条最小化的镜像冒烟检查(`docker run --rm <image> pwd` 断言工作目录),把这类“看不见的契约”纳入自动回归。

基础设施 重要性 6.18 洞察度 6.00

新增 gfx1151(Strix Halo)ROCm 镜像、构建期补丁与 nightly 发布流水线

值得精读,但读者按角色区分:负责 ROCm/发布基础设施的工程师应逐行看 docker/patches/sgl-kernel-gfx1151.sh,它是本 PR 信息密度最高的部分——wave32 部件上 host/device 两遍编译 WARP_SIZE 不一致会引发 `hipErrorLaunchFailure` 并把报错“甩锅”给后续 kernel,这个排障经验可迁移到其他非 gfx9 目标;同时“先 grep 断言原文、再就地替换、上游改动即 build fail”的构建期打补丁模式,对任何需要在固定上游源码上做临时适配的场景都值得借鉴。只做模型侧开发的读者可略读。需要重点盯的两个设计决策:一是 AITER“装但不用”(`SGLANG_USE_AITER=0` + 不设 PREBUILD_KERNELS)这一妥协,其代价是 Triton 被换成 3.7.0,目前无 CI 验证;二是把 gfx1151 支持完全寄托于镜像构建期补丁而非源码,删除条件已写进 PR body(PR#31137),建议在后续 PR 合并时同步清理并与 PR#31403 一起收口。

#38404 [Deps] Retire the CUDA 12 lane

原始 PR · 作者 mmangkad · 合并时间 2026-09-11 07:58

基础设施 重要性 6.91 洞察度 6.00

退役 CUDA 12 构建发布通道,为 PyTorch 2.14 铺路

值得精读,尤其对负责发布流水线、镜像与依赖管理的工程师:这是一次「删除为主」的 PR,真正的决策密度集中在少数几处。建议重点关注三点:一是 `scripts/update_kernel_whl_index.py` 把 CUDA 版本白名单替换为 `+cuNNN` 正则匹配——这是 fail-closed 的稳健性提升,杜绝了未来新 CUDA 版本的 wheel 误入 cu130 索引,值得作为同类脚本的范式;二是 `_docker-build-and-publish.yml` 中 manifest 固定使用 cu130 digest 后 `tag_config.cuda` 字段被静默忽略,建议合并后排查所有调用方并显式拒绝(或警告)非 cu130 的 `cuda` 取值;三是给 `check_wheel_cuda_version` 补一组针对 `+cu129` / `+cu130` / 无标记 / `+rocm720` 的单测。对 CUDA 12 用户,应在 release notes 中明确「v0.5.19 为最后一个 CUDA 12 版本」以及升级到 CUDA 13 的路径,避免他们从新的报错提示里得不到有效指引。

2026-09-10
缺陷修复 重要性 3.09 洞察度 4.00

移除 XPU pyproject 中的 human-eval git 依赖,修复镜像构建失败

值得快速浏览而非精读:单个 TOML 文件、单行删除、零逻辑变更,阅读成本极低,重点是理解失败根因(setuptools >= 81 移除 pkg_resources 与 PEP 517 隔离构建的交互),这个模式对任何依赖老式 setup.py + git 直装的仓库都有复用价值。阅读时建议关注两点:(1) 是否应在 `test` extra 处留下注释,标注 human-eval 被临时摘除及其恢复条件;(2) 是否存在替代路径(如换成带 pyproject.toml 的 fork、或在 Dockerfile 里固定 setuptools < 81)比直接删除依赖更完整。若你在 XPU 上依赖 human-eval 做评测,需自行补装并留意运行期 ImportError。

缺陷修复 重要性 2.26 洞察度 1.00

为 Xeon Docker 镜像添加 libssl-dev 依赖,修复原生 KV 哈希扩展编译失败问题。

该 PR 是一个非常简单的依赖修复,价值明确,风险极低。对于关注 CPU 部署和 Docker 镜像构建的工程师来说,值得快速了解其内容,但无需深入精读其技术实现。

2026-09-09
缺陷修复 重要性 3.21 洞察度 4.00

修复 Intel XPU Docker 夜间构建因 setvars.sh 退出码和依赖包名不匹配导致的失败。

这是一个低风险、高必要性的基础设施修复 PR,值得精读以理解 OneAPI 构建环境的陷阱和依赖同步的重要性。主要设计决策是容忍 `setvars.sh` 的非零退出码而不是尝试选择性地源文件,这是一种务实的做法。