Prhub

#42039 [CI][Bugfix] Drop duplicated examples/ prefix in tensorize_vllm_model command

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

执行摘要

修正 CI 配置中 tensorize 脚本多余路径前缀

PR #41084 将 examples 重新组织后,tensorize_vllm_model.py 脚本的调用命令在 3 个 Buildkite YAML 配置中(.builldkite/test-amd.yaml、.buildkite/test_areas/model_runner_v2.yaml、.buildkite/test_areas/misc.yaml)保留了多余的 examples/ 前缀。由于这些 CI job 的 working_dir 已设为 /vllm-workspace/examples,该前缀导致命令解析为 /vllm-workspace/examples/examples/features/tensorize_vllm_model.py,使得 Model Runner V2 Examples 和 Examples 两个 CI 步骤失败。

该 PR 是一个小型配置修复,变更意图明确、改动简单。无需深入审查,可以直接合并。对于维护者,值得注意类似 examples/ 重命名(PR #41084)可能遗留的路径问题,后续同类重构应检查所有引用点。

讨论亮点

审核中来自 noooop 的 Approved(感谢),和两个机器人评论。无实质讨论。

实现拆解

  1. 修改 .buildkite/test-amd.yaml(3 处修改):将三个 GPU agent pool(mi250、mi300、gfx942)的 Examples 步骤中的 python3 examples/features/tensorize_vllm_model.py 改为 python3 features/tensorize_vllm_model.py
  2. 修改 .buildkite/test_areas/model_runner_v2.yaml(1 处修改):将 Model Runner V2 Examples 步骤中相同的命令路径前缀去除。
  3. 修改 .buildkite/test_areas/misc.yaml(1 处修改):将 Examples 步骤中相同的命令路径前缀去除。
  4. 所有修改均只去除路径中的 examples/ 前缀,保持命令的其他参数不变,确保与其他相邻命令的路径风格一致。
文件 模块 状态 重要度
.buildkite/test-amd.yaml CI 配置 modified 3.33
.buildkite/test_areas/model_runner_v2.yaml CI 配置 modified 3.19
.buildkite/test_areas/misc.yaml CI 配置 modified 2.84

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

评论区精华

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

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

风险与影响

该 PR 仅修改了 CI 配置文件中的命令路径,无源码或测试变更。风险极低。变更后命令路径与 working_dir 协调一致,且与其他命令的路径风格相同。唯一可能的回归是如果将来 working_dir 再次变更,但这是配置本身的特性,与本修改无关。

影响范围仅限 CI 的 3 个 job:AMD 的 Examples 步骤、Model Runner V2 Examples 步骤和常规 Examples 步骤。对用户、系统性能、API 均无影响。修复后 tensorize_vllm_model.py 示例能够在 CI 中正确运行。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论