Skip to content

Revert "[Cherry-Pick][CI] Temporarily pin paddlepaddle-gpu to 3.5.0.dev20260417(#7486)"#7616

Open
EmmonsCurse wants to merge 1 commit intorelease/2.6from
revert-7519-cherry-pick/7486/release/2.6
Open

Revert "[Cherry-Pick][CI] Temporarily pin paddlepaddle-gpu to 3.5.0.dev20260417(#7486)"#7616
EmmonsCurse wants to merge 1 commit intorelease/2.6from
revert-7519-cherry-pick/7486/release/2.6

Conversation

@EmmonsCurse
Copy link
Copy Markdown
Collaborator

@EmmonsCurse EmmonsCurse commented Apr 24, 2026

Reverts #7519

Motivation

The temporary pin of paddlepaddle-gpu to 3.5.0.dev20260417 is no longer required. The underlying issue has been resolved or is no longer impacting CI, so keeping the pin would unnecessarily block version upgrades.

Modifications

  • Reverted the previous change that pinned paddlepaddle-gpu to 3.5.0.dev20260417.
  • Restored default version resolution behavior in CI.
  • Allowed CI to use newer Paddle versions going forward.

Usage or Command

N/A

Accuracy Tests

N/A

Checklist

  • Add at least a tag in the PR title.
    • Tag list: [[FDConfig],[APIServer],[Engine], [Scheduler], [PD Disaggregation], [Executor], [Graph Optimization], [Speculative Decoding], [RL], [Models], [Quantization], [Loader], [OP], [KVCache], [DataProcessor], [BugFix], [Docs], [CI], [Optimization], [Feature], [Benchmark], [Others], [XPU], [HPU], [GCU], [DCU], [Iluvatar], [Metax]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • Format your code, run pre-commit before commit.
  • Add unit tests. Please write the reason in this PR if no unit tests.
  • Provide accuracy results.
  • If the current PR is submitting to the release branch, make sure the PR has been submitted to the develop branch, then cherry-pick it to the release branch with the [Cherry-Pick] PR tag.

@EmmonsCurse
Copy link
Copy Markdown
Collaborator Author

/skip-ci ci_iluvatar
/skip-ci ci_hpu
/skip-ci build_xpu

@paddle-bot
Copy link
Copy Markdown

paddle-bot Bot commented Apr 24, 2026

Thanks for your contribution!

Copy link
Copy Markdown

@PaddlePaddle-bot PaddlePaddle-bot left a comment

Choose a reason for hiding this comment

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

🤖 AI Code Review | 2026-04-24 20:48:02

📋 Review 摘要

PR 概述:撤销对 paddlepaddle-gpu 版本的临时锁定,恢复 CI 使用 nightly 动态版本的默认行为。
变更范围.github/workflows/ 下 12 个 CI workflow 配置文件
影响面 TagCI


📝 PR 规范检查

PR 目标分支为 release/2.6(非 develop),按照规范,标题格式应为 [Cherry-Pick][Tag] 标题总结(#原PR编号)。当前标题 Revert "[Cherry-Pick][CI] Temporarily pin paddlepaddle-gpu to 3.5.0.dev20260417(#7486)" 为 GitHub 自动生成的 Revert 格式,不符合仓库规范。

标题建议(可直接复制):

  • [Cherry-Pick][CI] Revert temporarily pin paddlepaddle-gpu to 3.5.0.dev20260417(#7519)

问题

级别 文件 概述
🟡 建议 PR 标题 目标分支为 release/2.6,标题缺少 [Cherry-Pick] 前缀和规范 [Tag] 格式
❓ 疑问 _build_linux_rl.yml RL 构建使用 wget 下载 cp310 固定 whl,与其他 workflow 的 --pre 安装方式不一致,且硬编码 Python 3.10 版本

_build_linux_rl.yml 补充说明:此处通过 wget 从 bcebos.com 下载 paddlepaddle_gpu-0.0.0-cp310-cp310-linux_x86_64.whl 是 revert 前的原始逻辑,属于恢复的预存代码。需确认该 RL 构建环境确实使用 Python 3.10,且 bcebos 内网地址在 CI runner 上可访问。其他 workflow 均使用 --pre paddlepaddle-gpu 方式,如果可行,建议 RL 环境也统一使用 pip 方式以降低维护成本。


总体评价

这是一个纯 CI 配置回滚操作,整体逻辑清晰,12 个文件的改动内容合理且一致。主要建议修正 PR 标题格式以符合 release 分支规范,并确认 RL 构建 wget 方式的可靠性。

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