Prhub

#5717 [ci] fix: fix various ci failure

原始 PR 作者 wuxibin89 合并时间 2026-03-24 12:14 文件变更 29 提交数 11 评论 0 代码增减 +77 / -86

执行摘要

修复多个 CI 失败点,包括配置、导入错误和兼容性问题。

PR body列出8个具体修复点,目的是解决CI失败,如'fix 3d position_ids in SFT trainer introduced in https://github.com/verl-project/verl/pull/5689'、'fix LinearForLastLayer import error introduced in https://github.com/verl-project/verl/pull/5707'等,以确保开发和测试流程的连续性。

建议CI维护者和相关模块开发者(如模型初始化、数据集处理)精读此PR,重点关注get_hf_rope_theta函数的实现逻辑和LinearForLastLayer的参数调整,以了解兼容性处理和模块重构。

讨论亮点

review中,gemini-code-assist[bot]评论指出import路径更改和参数替换是兼容性调整,Superjomn批准LGTM,HollowMan6建议等待CI通过后被dismissed。没有重大争议,更改被接受,但HollowMan6的评论暗示CI验证的重要性。

实现拆解

实现主要包括:

1) 在多个GitHub workflow文件中删除HTTP_PROXY环境变量,防止vllm/sglang请求被代理;
2) 新增get_hf_rope_theta函数处理transformers>5的rope_theta兼容性,更新相关配置文件;
3) 将LinearForLastLayer导入从verl.models.llama.megatron.layers.parallel_linear改为verl.models.mcore.bridge,并调整参数从configsequence_parallel
4) 在verl/utils/dataset/dataset_utils.py中添加_ragged_idx修复3D position_ids;
5) 修改vllm和sglang服务器代码避免端口冲突;
6) 临时禁用megatron critic model pp、trt-llm vlm ci和sglang reward model ci等测试。

文件 模块 状态 重要度
.github/workflows/e2e_ppo_grpo_trainer_trtllm.yml ci modified 4.0
verl/models/mcore/config_converter.py model modified 5.0
verl/models/mcore/model_initializer.py model modified 5.0
verl/utils/dataset/dataset_utils.py dataset modified 4.0
verl/workers/rollout/vllm_rollout/vllm_async_server.py rollout modified 4.0

关键符号

get_hf_rope_theta LinearForLastLayer.__init__ collate_variable_batch

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

评论区精华

LinearForLastLayer import and parameter changes 设计

gemini-code-assist[bot] commented: 'This pull request primarily focuses on updating the import path for LinearForLastLayer and modifying its initialization parameters... likely to align with an updated module structure or API.'

结论:Changes accepted as compatibility refactoring, with approval from Superjomn. · 已解决

风险与影响

风险包括:

1) 删除HTTP_PROXY可能在某些网络环境下导致连接问题,但已在多个workflow中统一移除;
2) 临时禁用测试(如megatron critic model pp)可能掩盖潜在悬挂错误,需后续修复;
3) rope_theta兼容性更改在verl/models/mcore/config_converter.py中可能对新模型配置处理不足;
4) LinearForLastLayer导入路径更改需确保所有依赖模块更新,否则可能引发ImportError。

对团队开发有积极影响,修复CI失败提升开发效率和测试稳定性;对系统影响小,主要是配置和代码微调,不涉及核心功能变更;用户无直接影响,属于内部基础设施维护。

临时禁用测试 兼容性更改 依赖调整

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论