[CI] Add no_proxy configuration for docker execution#7283
Merged
EmmonsCurse merged 1 commit intoPaddlePaddle:developfrom Apr 9, 2026
Merged
[CI] Add no_proxy configuration for docker execution#7283EmmonsCurse merged 1 commit intoPaddlePaddle:developfrom
EmmonsCurse merged 1 commit intoPaddlePaddle:developfrom
Conversation
Collaborator
Author
|
/skip-ci ci_iluvatar |
|
Thanks for your contribution! |
fastdeploy-bot
left a comment
There was a problem hiding this comment.
🤖 AI Code Review |
2026-04-09 17:23 CST
📋 Review 摘要
PR 概述:为 14 个 CI workflow 文件添加统一的 no_proxy 环境变量配置,避免内部服务和镜像源通过代理访问
变更范围:.github/workflows/ 目录下的 CI 配置文件
影响面 Tag:CI
问题
| 级别 | 文件 | 概述 |
|---|---|---|
| 🟡 建议 | 多个文件 | 存在遗漏的 XPU/HPU 相关 workflow 文件,其 no_proxy 配置与本次 PR 不一致 |
| 🟡 建议 | 全局 | 相同的 no_proxy 字符串在 14 个文件中重复,后续维护成本较高 |
总体评价
本次变更意图清晰,通过配置 no_proxy 提高 CI 稳定性是合理的。但建议同步更新以下遗漏的文件以保持一致性:
_build_xpu.yml_xpu_4cards_case_test.yml_xpu_8cards_case_test.ymlci_hpu.yml
这些文件中已有旧的 no_proxy 配置(值较短),建议一并更新为本次 PR 中的完整列表。
| -v "${CACHE_DIR}/.cache:/root/.cache" \ | ||
| -v "${CACHE_DIR}/ConfigDir:/root/.config" \ | ||
| -e TZ="Asia/Shanghai" \ | ||
| -e "no_proxy=localhost,127.0.0.1,0.0.0.0,bcebos.com,.bcebos.com,bj.bcebos.com,su.bcebos.com,paddle-ci.gz.bcebos.com,apiin.im.baidu.com,baidu-int.com,.baidu.com,aliyun.com,gitee.com,pypi.tuna.tsinghua.edu.cn,.tuna.tsinghua.edu.cn" \ |
There was a problem hiding this comment.
🟡 建议 发现以下 workflow 文件未被本次 PR 覆盖,但已有旧的 no_proxy 配置,建议同步更新以保持一致性:
| 文件 | 当前值 |
|---|---|
_build_xpu.yml:130 |
bcebos.com,mirrors.tuna.tsinghua.edu.cn,127.0.0.1,localhost |
_xpu_4cards_case_test.yml:137 |
同上 |
_xpu_8cards_case_test.yml:132 |
同上 |
ci_hpu.yml:86 |
bcebos.com,mirrors.tuna.tsinghua.edu.cn,localhost,127.0.0.1,0.0.0.0,10.0.0.0/8,192.168.1.0/24 |
建议将这些文件的 no_proxy 值也更新为本次 PR 中的完整列表,确保所有 CI 任务行为一致。
DDDivano
approved these changes
Apr 9, 2026
This was referenced Apr 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
CI tasks are affected by network instability when accessing internal services and third-party package mirrors through the proxy. This may lead to failures during dependency installation (e.g., pip install, wget), especially for domains that should be accessed directly.
To improve reliability and avoid unnecessary proxy routing, it is necessary to explicitly configure
no_proxyfor commonly used internal domains and mirror sources.Modifications
no_proxyenvironment variable in docker execution..bcebos.com,.baidu.com,.tuna.tsinghua.edu.cn, etc.) to bypass proxy.Usage or Command
N/A
Accuracy Tests
N/A
Checklist
[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]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.