Prhub

#45654 [CI/Build] Avoid duplicate ViT CG test introduced by accident

原始 PR 作者 Isotr0py 合并时间 2026-06-17 20:23 文件变更 1 提交数 5 评论 2 代码增减 +8 / -2

执行摘要

移除重复的 ViT CUDA Graph 测试用例

PR body 指出:"Currently, ViT CG tests run on both Multi-Modal Models (Standard) 2: qwen3 + gemma and Multi-Modal Models (Standard) 4: other + whisper, which waste CI resource." 因此移除 qwen+gemma 套件中的重复用例。

可直接合并。该 PR 是纯粹的 CI 优化,逻辑清晰,适合快速批准。

讨论亮点

讨论集中在 AMD CI 的适配:AndreasKaratzas 指出 test_phi3v.py 在 AMD 上失败,建议添加 soft_fail: true,并承诺后续修复。PR 作者采纳了建议。

实现拆解

  1. 删除重复测试:在 Multi-Modal Models (Standard) 2: qwen3 + gemma 的 commands 中移除 pytest -v -s models/multimodal/generation/test_vit_cudagraph.py -m core_model 这一行。
  2. 调整测试归属:在 Multi-Modal Models (Standard) 4: other + whisper 的 commands 中,将原本通过 --ignore 排除的 test_vit_cudagraph.py 从忽略列表中移除,并显式添加 pytest -v -s models/multimodal/generation/test_vit_cudagraph.py -m core_model 命令。
  3. AMD 镜像同步:为 Standard 4 套件添加 AMD mirror 配置,设置 soft_fail: truemi325_1 设备和 image-build-amd 依赖。此调整基于 AndreasKaratzas 的评论,他表示目前 AMD 上有 test_phi3v.py 失败,先标记为 soft_fail 以保持 CI 绿色。
  4. 仅涉及 CI 配置:变更仅限 .buildkite/test_areas/models_multimodal.yaml 文件,无代码逻辑改动。
文件 模块 状态 重要度
.buildkite/test_areas/models_multimodal.yaml CI 配置 modified 4.13

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

评论区精华

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

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

风险与影响

低风险。仅 CI 配置变更,测试逻辑无修改。唯一风险是 AMD mirror 新增 soft_fail 后可能掩盖真正的失败,但作者已计划后续修复。

对系统性能无影响。CI 资源消耗降低:每次运行减少一次 test_vit_cudagraph.py 的执行。团队协作上,AMD CI 配置的更新为后续维护提供了明确入口。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论