Prhub

#42293 [CI] Make Python-only Installation optional

原始 PR 作者 haosdent 合并时间 2026-05-11 17:47 文件变更 1 提交数 1 评论 1 代码增减 +1 / -0

执行摘要

Python-only 安装测试改为可选步骤

"Python-only Installation" 测试步骤依赖于 wheels.vllm.ai/<commit>/vllm/metadata.json,该文件由 release-v2 的 wheel-publish 流水线生成。在每次 postmerge 运行或触发 RUN_ALL=1 的 PR 中,该测试会与 publish 步骤产生竞态,导致 404 错误。将测试标记为 optional 后,它只会在 nightly 调度(每日两次,NIGHTLY=1)中执行。

该 PR 改动极小且合理,建议合并。对于超时问题的建议,可考虑在后续 PR 中评估是否需要增加时间。

讨论亮点

gemini-code-assist 机器人提出,当前 20 分钟的超时可能不足以容纳脚本内部的 4 次重试(每次 sleep 5 分钟),建议增加到 30 分钟。该建议未在后续评论中得到确认或处理。

实现拆解

  1. 修改 CI 配置:在 .buildkite/test_areas/misc.yaml 中,为 "Python-only Installation" 步骤添加 optional: true 配置项。
  2. 保留依赖和命令:保持原有的 source_file_dependenciestests/standalone_tests/python_only_compile.sh, setup.py)和 commands 不变。
  3. 无其他代码或测试变更:仅这一行配置变更。
文件 模块 状态 重要度
.buildkite/test_areas/misc.yaml CI 配置 modified 3.33

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

评论区精华

超时时间可能不足 性能

gemini-code-assist 机器人指出,当前 20 分钟超时可能不足以容纳脚本内部的 4 次重试(每次 sleep 5 分钟),建议增加到 30 分钟。

结论:该建议未被采纳或进一步讨论,PR 保持 20 分钟超时。 · 已解决

风险与影响

  • 测试覆盖延迟:修改 setup.pytests/standalone_tests/python_only_compile.sh 的 PR 将不再自动触发该测试,直到下一次 nightly 运行。这可能导致相关问题被发现的时间延迟最长可达 12 小时。
  • 轻微风险:由于该测试仅在 nightly 执行,CI 流程中针对 Python-only 安装的回归检测将不再实时生效。
  • CI 稳定性:消除与 wheel-publish 流水线的竞态导致的 404 失败,提高 CI 整体稳定性。
  • 测试触发频率:从每次 postmerge/PR 触发变为每日两次,显著降低执行频率。
  • 无用户功能影响:纯 CI 变更,不影响 vLLM 运行时。
轻微风险:测试覆盖延迟

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论