Prhub

#39557 [CI/Build] Fix sentence-transformers version in CPU test

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

执行摘要

修复 CPU 测试 Dockerfile 中 sentence-transformers 版本号错误,确保依赖解析成功。

根据PR body中引用的链接(https://github.com/vllm-project/vllm/pull/38800#issuecomment-4228314305),该修复是为了解决CPU测试环境构建时因sentence-transformers版本号错误导致的依赖解析失败问题。review评论明确指出5.3.0版本在PyPI上不存在,会导致构建失败。

该PR变更简单直接,无需精读。值得关注的是review中版本号验证的重要性,可作为依赖管理的最佳实践参考。

讨论亮点

review中只有gemini-code-assist[bot]提出关键问题:原提交的5.3.0版本在PyPI上不存在(最新为3.3.1),会导致uv pip compile失败。建议更正为有效版本如3.3.0或2.3.0。维护者noooop直接批准,未进一步讨论。

实现拆解

仅修改一个文件docker/Dockerfile.cpu,在sed命令中将sentence-transformers依赖从任意版本固定为3.3.0版本。具体改动是在原有依赖替换命令序列中增加一行:sed -i 's/^sentence-transformers.*/sentence-transformers==3.3.0/g' requirements/test/cpu.in

文件 模块 状态 重要度
docker/Dockerfile.cpu infra modified 2.0

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

评论区精华

sentence-transformers 版本号错误 正确性

gemini-code-assist[bot] 指出 5.3.0 版本在 PyPI 上不存在,会导致 uv pip compile 失败。

结论:维护者批准,版本号应更正为有效版本(最终提交显示已改为 3.3.0)。 · 已解决

风险与影响

风险极低:

  1. 变更范围极小(仅一行版本号修正),不涉及核心逻辑。
  2. 修复前版本错误会导致构建失败,修复后依赖解析恢复正常,无回归风险。
  3. 无性能、安全或兼容性影响。

影响有限:

  1. 对用户无直接影响,仅影响CI/CD中的CPU测试环境构建。
  2. 确保CPU测试能正常执行,避免因依赖问题导致的CI失败。
  3. 团队需注意版本号准确性,避免类似问题。

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论