Regression test - #1393
Conversation
risemeup1111
left a comment
There was a problem hiding this comment.
本次 review 发现一个需要阻塞合入的问题:新增的 PR 触发流程目前没有检出 PR head,详细说明已放在行级评论里。另有一个 Daily CE 默认行为变化的建议也已内联标注。
优先级:P3 非行级:PR 描述仍是空模板,且
Check PR Description已失败。建议补齐PR Category、PR Types、变更说明和“是否引起精度变化”;例如本次 workflow 变更可填写PR Category: Execute Infrastructure、PR Types: Improvements/Devs,并说明新增 PR 触发、use_release默认值调整的测试范围与风险。
| paddle_url_cu129: "https://paddle-qa.bj.bcebos.com/paddle-pipeline/Develop-TagBuild-Training-Linux-Gpu-Cuda12.9-Cudnn9.9-Trt10.5-Mkl-Avx-Gcc11-SelfBuiltPypiUse/latest/" | ||
| paddle_url_cu130: "https://paddle-qa.bj.bcebos.com/paddle-pipeline/Develop-TagBuild-Training-Linux-Gpu-Cuda130-Cudnn913-Trt1013-Mkl-Avx-Gcc11-SelfBuiltPypiUse/latest/paddlepaddle_gpu-0.0.0-cp312-cp312-linux_x86_64.whl" | ||
| use_release: ${{ inputs.use_release || 'true' }} | ||
| use_release: ${{ inputs.use_release || 'false' }} |
There was a problem hiding this comment.
这一行把所有非 workflow_dispatch 触发的默认值都改成了 false,包括原有的定时 Daily CE;结果是定时任务不再安装上面配置的 develop 预编译 Paddle 包,而是走 pip install -e . 的依赖解析路径,和输入项 default: 'true' 以及原来的 Daily CE 语义不一致。若只是希望 PR 触发时不使用预编译包,建议按事件区分默认值,避免改变定时任务行为。
| use_release: ${{ inputs.use_release || 'false' }} | |
| use_release: ${{ github.event_name == 'pull_request' && 'false' || inputs.use_release || 'true' }} |
| name: CE Daily Dev | ||
|
|
||
| on: | ||
| pull_request: |
There was a problem hiding this comment.
这里给 Daily CE 增加了 pull_request 触发,但后续测试步骤仍在容器里重新 git clone https://github.com/PaddlePaddle/PaddleFleet.git,没有拉取 pull/${PR_ID}/head 或切到 ${COMMIT_ID}(本文件里多处克隆 PaddleFleet 的步骤都是这个模式)。这样 PR 触发的 CE 会安装并测试默认分支代码,而不是当前 PR 的修改,容易给出误导性的通过结果。合入前请在每个克隆 PaddleFleet 的步骤里显式切到 PR head;可以复用现有 PR release workflow 的做法,形如:
git clone https://github.com/PaddlePaddle/PaddleFleet.git
cd PaddleFleet
if [ "${PR_ID}" != "0" ]; then
git fetch origin "pull/${PR_ID}/head"
git checkout FETCH_HEAD
else
git checkout "${BRANCH}"
fi
git log -1
PaddleFleet Log Analysis
日志分析报告
失败的测试case: 根本原因分析: 修复建议:
🔄 每次 Re-run 后自动更新 |
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-07-04 00:30:45
📋 Review 摘要
PR 概述:为 CE Daily Dev 工作流新增 PR 触发,并调整 use_release 默认值。
变更范围:GitHub Actions CE 工作流
影响面 Tag:Execute Infrastructure
问题
| 级别 | 文件 | 概述 |
|---|---|---|
| 🔴 Bug | .github/workflows/ce_daily_dev.yml:4 |
新增 PR 触发后没有检出 PR head,CE 会验证默认分支代码 |
📝 PR 规范检查
标题缺少官方 Tag,PR 描述的必填章节仍是模板占位内容。
标题建议(可直接复制):
[Execute Infrastructure] Enable CE Daily Dev regression workflow for PRs
PR 描述建议(点击展开,可直接复制)
### PR Category
Execute Infrastructure
### PR Types
Improvements
### Description
为 `CE Daily Dev` GitHub Actions 工作流新增 `develop` 分支的 `pull_request` 触发,并将未传入 `use_release` 时的默认值从 `true` 调整为 `false`,使回归任务默认不额外安装预编译 Paddle 包。总体评价
当前 PR 方向是补充回归触发,但新增 pull_request 后工作流没有检出 PR 提交,导致回归结果无法覆盖待合入代码,需要先修复。另请同步补全 PR 标题和描述。
| name: CE Daily Dev | ||
|
|
||
| on: | ||
| pull_request: |
There was a problem hiding this comment.
🔴 Bug 现在 pull_request 会触发整套 CE,但测试代码没有切到 PR head。
下面的安装步骤仍然只是 git clone https://github.com/PaddlePaddle/PaddleFleet.git,文件内搜索不到 checkout / fetch pull/${PR_ID}/head,传入容器的 COMMIT_ID 也没有被使用;coverage job 里对应的 git pull --no-edit origin pull/${PR_ID}/head 还是注释状态。这样 PR 触发后实际跑的是默认分支代码,不能验证本 PR 的回归结果。
建议修复方式:在每个 clone PaddleFleet 的步骤后显式切到本次 PR commit,例如:
git fetch origin pull/${PR_ID}/head
git checkout ${COMMIT_ID}或统一改用 actions/checkout 检出 github.event.pull_request.head.sha,确保单卡、多卡、集成测试和 coverage 检查都使用同一个 PR head。
PR Category
PR Types
Description
是否引起精度变化