Prhub

#26990 ci: disable cross-job fast-fail for run_all_tests dispatch

原始 PR 作者 hnyls2002 合并时间 2026-06-02 09:32 文件变更 1 提交数 1 评论 1 代码增减 +1 / -1

执行摘要

禁用全量 CI 手动触发的跨任务快速失败

作为 #26986 的后续,PR body 指出:手动全量 CI 调度中,快速失败检查(check-pr-test-health)仅当 schedule 触发或设置 SKIP_PR_TEST_HEALTH_CHECK=true 时跳过;但 stage workflow 重新推导该环境变量时只用了 skip_pr_test_health_check,漏掉了 run_all_tests / test_parallel_dispatch,导致手动全量运行仍然快速失败。

值得关注 CI 流程的演化,但本身技术含量低,普通工程师阅读即可。

讨论亮点

无 review 讨论。

实现拆解

  1. 修改 .github/workflows/_pr-test-stage.yml 中的 SKIP_PR_TEST_HEALTH_CHECK 环境变量计算逻辑,在原有 fromJson(inputs.caller_inputs).skip_pr_test_health_check 基础上,通过 || 增加 fromJson(inputs.caller_inputs).test_parallel_dispatchfromJson(inputs.caller_inputs).run_all_tests 条件。
  2. 使得当手动触发 run_all_teststest_parallel_dispatch 调度时,SKIP_PR_TEST_HEALTH_CHECK 也被设为 true,从而避免跨任务快速失败。
  3. 本变更仅一行增删,不涉及测试、配置或部署其他改动。
文件 模块 状态 重要度
.github/workflows/_pr-test-stage.yml CI 脚本 modified 2.95

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

评论区精华

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

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

风险与影响

风险极低:仅修改 CI 环境变量推导逻辑,不影响任何源码或运行时行为。若条件表达式有语法错误,会导致 CI 阶段环境变量异常,但已在 PR 中验证通过。

影响范围仅限于手动触发的全量 CI 运行流程。变更后,手动全量调度将像定时调度一样收集所有测试任务失败,而非快速失败,便于开发者一次获取完整失败信息。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论