Prhub

#26110 [CI] pr-test-extra: add run_all_tests to workflow_dispatch inputs

原始 PR 作者 Kangyan-Zhou 合并时间 2026-05-22 18:03 文件变更 1 提交数 1 评论 1 代码增减 +5 / -0

执行摘要

为 pr-test-extra 的 workflow_dispatch 添加 run_all_tests 选项

手动触发 pr-test-extra.yml 测试 release 分支时,因 _pr-test-check-changes.yml 中的 dorny/paths-filter 步骤在浅克隆下无法找到合并基,会调用 git fetch --deepen=200 扩展历史,但该操作与并发索引更新冲突,导致 fatal: shallow file has changed since we read it 错误。添加 run_all_tests 输入后可跳过路径过滤,直接运行所有测试,避免此问题。PR body 中引用了三次失败运行记录作为证据。

推荐快速合并。这是一个定向修复的小配置补齐,正确性明确,且作者已计划后续 cherry-pick 到 release 分支。

讨论亮点

无 review 讨论或评论。PR 由作者自行合并,说明变更清晰且无争议。

实现拆解

  1. .github/workflows/pr-test-extra.ymlworkflow_dispatch 事件定义下新增 run_all_tests 输入,类型为 boolean,默认值为 false,并添加描述说明其用途。
  2. 该输入与已有的 workflow_call 接口和 pr-test.yml 中的输入对称,下游 _pr-test-check-changes.yml 已经支持(第 77 行读取 inputs.run_all_tests 来条件跳过路径过滤步骤),因此无需修改其他文件。
  3. 变更仅 5 行新增,无逻辑改动,属于基础设施配置补齐。
文件 模块 状态 重要度
.github/workflows/pr-test-extra.yml CI 脚本 modified 3.31

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

评论区精华

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

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

风险与影响

风险极低。变更仅为添加一个可选输入,默认行为不变(false),不会影响现有手动或自动触发流程。下游逻辑已就绪,无回归风险。

影响范围局限在 pr-test-extra.yml 的手动触发场景。允许运维人员在测试 release 分支等无法自动计算路径过滤的场景下,通过勾选 run_all_tests 直接跳过路径过滤,运行全部 extra 测试,提高手动调测效率。

低影响配置变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论