Prhub

#49325 [CI] Wire tests/models/inkling into a B200 job

原始 PR 作者 njhill 合并时间 2026-07-21 22:43 文件变更 1 提交数 1 评论 0 代码增减 +13 / -0

执行摘要

修复 Inkling 测试套件未在 CI 中运行的问题

Inkling 测试套件包含了对关键功能的 contract validation 等测试,但未被任何 CI job 引用,导致变更(如 #48822)无法通过 CI 验证。PR body 明确指出:"The tests/models/inkling suite was not referenced by any CI job, so none of it ran in CI - including the contract-validation tests that #48822 itself modified."

此 PR 是典型的 CI 基础设施补全,值得合并。对于维护者,可以关注 Inkling 测试后续是否稳定,以及是否需要在其他 GPU 型号(如 H200)上添加类似的 job。

讨论亮点

本 PR 没有 review 评论,仅有 hmellor 的快速批准。Claude bot 自动评论指出 PR 来自 fork,但实际为同一仓库分支,未触发进一步评审。

实现拆解

  1. 修改 CI 配置文件:在 .buildkite/test_areas/models_basic.yaml 中新增一个 job 定义,标签为 "Inkling Unit Tests (B200)",键为 "inkling-unit-tests-b200"。
  2. 设备选择:指定 device 为 b200-k8s,因为 FA4 kernel 测试需要 SM100 架构,仅在 B200 等 GPU 上支持,其他设备会跳过。
  3. 源文件依赖:设置 source_file_dependencies,包括 vllm/models/inkling/vllm/cute_utils/cmake/external_projects/tml_fa4.cmake 以及 tests/models/inkling/,确保只有当相关文件变更时才会触发该 job。
  4. 命令与超时:执行 pytest -v -s models/inkling,超时时间设为 40 分钟,实测单 GPU 运行约 6.5 分钟,当前主分支 194/194 测试通过。
文件 模块 状态 重要度
.buildkite/test_areas/models_basic.yaml CI 配置 modified 4.62

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

评论区精华

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

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

风险与影响

低风险。仅修改了 CI 配置,新增一个 job,不会影响任何运行时代码或现有测试流程。需确保 B200 资源可用且测试在较长时间内稳定。

对用户无直接影响。对 CI 系统而言,新增了一个约 6.5 分钟的 job,但仅在 Inkling 相关文件变更时触发,不会显著增加整体 CI 耗时。此 PR 填补了测试覆盖空白,避免遗漏关键测试。

依赖 B200 资源可用性

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论