Skip to content

[Review] Add release PR review source rules - #79739

Open
liuhao2638 wants to merge 1 commit into
PaddlePaddle:developfrom
liuhao2638:codex/release-pr-review-source-develop
Open

[Review] Add release PR review source rules#79739
liuhao2638 wants to merge 1 commit into
PaddlePaddle:developfrom
liuhao2638:codex/release-pr-review-source-develop

Conversation

@liuhao2638

@liuhao2638 liuhao2638 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

PR Category

Execute Infrastructure

PR Types

Devs

Description

为目标分支为 releaserelease/* 的 PR 增加基础评审规则:

  • 在同一仓库定位唯一对应的 develop PR,并将其 diff 与当前 release PR 的 diff 逐项比较。
  • 仅评审 develop 相对 release 新增、缺失或语义不一致的部分;已存在且语义等价的改动不重复评审。
  • 关联缺失或有歧义时,要求作者提供对应 develop PR;不回退完整 release diff 评审或批准。
  • 评论和 Review Board 仍发布在 release PR。

是否引起精度变化

@Paddle-Bot Paddle-Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Paddle-Bot Review Board (review完成)

序号 位置 优先级 规则来源 状态
1 Release 对应 source 快照 P1 仓库规则:Release 分支评审
Powered by Nyanpasu with gpt-5.6-sol 默认推理级别, please check the suggestions carefully.

## Release 分支评审

- 当 PR 的目标分支 `base_ref` 为 `release` 或匹配 `release/*` 时,适用本节;源分支名称不触发此规则。
- 不对该 release PR 的 diff 进行完整评审。必须在同一仓库定位唯一对应、目标分支为 `develop` 的 PR,并仅评审该 develop PR 的 diff。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1:当前规则要求评审对应 develop PR 的“当前 diff”,但 release cherry-pick 可能只包含该 PR 的历史提交,且冲突解决后的 release 代码也可能不同。实际 release PR #79731 关联 develop PR #79716,develop PR 当前已有第 5 个提交,而 release PR 只包含前 4 个 cherry-pick;按本条会把未发布改动纳入评审并遗漏 release 特有差异。请在规则中要求记录并校验对应的 source head/提交集合或补丁等价性,只评审与 release 实际内容一致的快照;若不一致应暂停并要求作者补充。

@liuhao2638
liuhao2638 force-pushed the codex/release-pr-review-source-develop branch from f131bc4 to dca58e4 Compare September 3, 2026 13:16
@Paddle-Bot

Paddle-Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Failed CI看板

流水线名称 问题标签 修复建议 日志片段
Check PR Template (#33760195665, attempt 1) / Check PR 元信息校验失败 【与当前改动无关】将 QA负责人 weike 更新为需求系统认可的有效负责人或修正对应 Pcard 信息后重跑 报错代码
Check PR Template (#33760144403, attempt 1) / Check PR 元信息校验失败 【与当前改动无关】按需求系统规则修正无效的 QA负责人字段后重跑 Check PR Template 报错代码
CI-Build (#33760145394, attempt 1) / Check file paths GitHub Action 下载超时 【与当前改动无关】待 runner 到 GitHub codeload 网络恢复后重跑 CI-Build 报错代码
Approval (#33760145097, attempt 2) / Check approval 缺少 .agents/skills 所需 RD 审批 【与当前改动有关】为当前 commit dca58e40f1a0a54175b5b9987703f65f4b760285 获取 zrr1999risemeup1risemeup1111 之一的审批后重跑 Approval 报错代码
日志分析报告

失败的测试 case:

1. Check PR Template (#33760195665)
   测试 case:PR 模板字段及需求卡创建校验
   步骤:读取 PR 字段,使用默认 Pcard-93406,创建需求
   关键错误:QA负责人为 weike;weike is INVAILD;进程退出码 1

2. Check PR Template (#33760144403)
   测试 case:PR 模板字段及需求卡创建校验
   步骤:读取 PR 字段,使用默认 Pcard-93406,创建需求
   关键错误:QA负责人为 weike;weike is INVAILD;进程退出码 1

3. CI-Build (#33760145394) / Check file paths
   测试 case:下载并准备 PaddlePaddle/Paddle@62d6f1f2346d8704d834eb71b0dad7731be21c9a
   步骤:连续两次从 codeload.github.com 下载 action,均等待 100 秒超时
   关键错误:The request was canceled due to the configured HttpClient.Timeout of 100 seconds elapsing

4. Approval (#33760145097) / Check approval
   测试 case:检查 .agents/skills 文件变更所需审批
   步骤:检查 RD 审批列表
   关键错误:0. You must have one RD approval for file changes in .agents/skills;
             There are 1 approved errors;进程退出码 6

根本原因分析:

  • 两次 Check PR Template 属于同一类重复失败,根因都是外部需求系统拒绝 PR 元信息中的 QA负责人 weike,不是代码执行、编译或测试失败。PR diff 只修改 .agents/skills/ai-review/references/base-rules.md,没有修改 QA负责人或 Pcard 配置,因此判定为【与当前改动无关】。
  • CI-Build / Check file paths 在执行仓库代码检查前下载 GitHub Action 时发生 codeload 网络超时,并经过重试后取消。日志没有显示编译或文件路径检查错误,PR diff 也未涉及下载配置,因此判定为【与当前改动无关】。该 job 与其他失败没有证据表明存在代码级因果关系。
  • Approval / Check approval 直接检查到了本次修改的 .agents/skills 路径。日志明确要求至少一个指定 RD 审批,但当前检查结果为 0 个,因此失败。该失败由当前 diff 触发的仓库审批门禁导致,判定为【与当前改动有关】;没有证据表明新增规则内容本身存在语法或逻辑错误。

修复建议:

  1. 【与当前改动无关】Check PR Template (#33760195665):在 PR 模板、Pcard 或关联需求信息中将 QA负责人 修正为需求系统认可的有效账号,然后重跑该检查。
  2. 【与当前改动无关】Check PR Template (#33760144403):修正同一份 PR 元信息中的无效 QA负责人;两次失败根因相同,无需修改代码。
  3. 【与当前改动无关】CI-Build (#33760145394):确认 runner 至 codeload.github.com 的网络可用后,仅重跑 CI-Build;当前快照未发现代码构建错误。
  4. 【与当前改动有关】Approval (#33760145097):让 zrr1999risemeup1risemeup1111 对当前 head 提交 RD 审批,再重跑 Approval;旧 commit 的审批不能替代当前 head 审批。

Powered by Nyanpasu with gpt-5.6-luna 默认推理级别, please check the suggestions carefully.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants