Prhub

#26985 ci: drop redundant multimodal-server jobs from nightly (Nvidia)

原始 PR 作者 alisonshao 合并时间 2026-06-02 14:40 文件变更 1 提交数 1 评论 2 代码增减 +0 / -123

执行摘要

从 nightly 测试中删除冗余的多模态 server 作业

PR body 指出:nightly-test-multimodal-server-1-gpunightly-test-multimodal-server-2-gpu jobs 运行与 pr-test-multimodal-gen.yml 完全相同的 diffusion suites,后者的 per-commit 覆盖已经包含这些测试。因此 nightly 中的重复 job 是冗余的,不增加额外覆盖,且浪费 CI 资源。

此 PR 是一个健康的 CI 清理,建议快速合并。对于关注 CI 效率的团队有参考价值。

讨论亮点

Reviewer mickqian 直接批准,无额外讨论。

实现拆解

  1. 删除 job 定义:从 .github/workflows/nightly-test-nvidia.yml 中移除 nightly-test-multimodal-server-1-gpunightly-test-multimodal-server-2-gpu 的完整 jobs 块(包括步骤、环境配置和超时设置)。
  2. 更新 workflow_dispatch job-filter 选项:删除了 on.workflow_dispatch.inputs.job_filter.options 中对应的两个 job 名称,使手动触发时不再显示这些选项。
  3. 调整 job 依赖关系:在 consolidate-metricscheck-all-jobsneeds: 列表中移除这两个 job,确保下游任务不再等待它们。
文件 模块 状态 重要度
.github/workflows/nightly-test-nvidia.yml CI 配置 modified 4.42

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

评论区精华

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

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

风险与影响

极低风险。删除的 job 已被 pr-test-multimodal-gen.yml 完全覆盖,移除后不影响任何测试覆盖范围。需确保 consolidate-metricscheck-all-jobs 不再引用被删 job,已通过更新 needs: 列表保证。

减少了 nightly CI 的运行时间和 GPU 资源消耗,提高整体 CI 效率。对开发者无感知,不影响日常 PR 测试。

低风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论