Prhub

#43974 [CI] Fix smoke test step key to bypass block gate

原始 PR 作者 khluu 合并时间 2026-05-30 07:28 文件变更 1 提交数 1 评论 0 代码增减 +1 / -1

执行摘要

修复 CI smoke 测试步骤键名使其绕过手动阻塞门

PR #43712 将非根 smoke 测试从镜像构建步骤中分离出来,但新步骤的键名 image-smoke-test 未以 image-build 开头,导致 CI 流水线生成器插入手动阻塞步骤。PR body 明确说明:“The pipeline-generator's _step_should_run() auto-runs steps whose key starts with "image-build" — the previous key didn't match, causing a manual block step to be inserted.”

值得合并,修复了之前 PR 引入的 CI 流程问题。CI 维护者可关注是否有其他步骤键名也需按此命名规范调整。

讨论亮点

该 PR 无 review 评论。唯一审核人 njhill 已批准,无讨论。

实现拆解

  1. 修改键名:在 .buildkite/image_build/image_build.yaml 文件中,将非根 smoke 测试步骤的 key 字段从 image-smoke-test 改为 image-build-smoke-test
  2. 依赖关系不变:步骤的 depends_on 仍指向 image-build,保证下游步骤继续等待该步骤完成后才开始。
  3. 单一文件、单行变更:仅一行 YAML 配置变更,无其他代码或测试改动。
文件 模块 状态 重要度
.buildkite/image_build/image_build.yaml CI 配置 modified 2.27

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

评论区精华

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

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

风险与影响

风险极低。仅修改了 CI 流水线配置中的一个步骤键名,不影响任何源码、模型推理或外部 API。若流水线生成器对新前缀有意外行为(如名称冲突),可能导致 smoke 测试重复运行或顺序异常,但同一文件中不存在其他以 image-build 开头的键,因此风险可控。

仅影响 vLLM 仓库的 CI 流水线行为:非根 smoke 测试步骤将自动运行而非等待手动触发,提高 CI 自动化程度和开发效率。对用户无直接影响,对系统无性能或功能影响。

关联 Issue

#43712 [CI] Separate non-root smoke tests from image build step

完整报告

参与讨论