已合并
add docker note #2011
Leewis创建于 7月16日
add docker note #2011
已合并
Leewis创建于 7月16日
Leewis成员
7月16日

描述

增加单机多容器说明;增加mpi说明

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Leewis 的贡献)
atomgit-bot
atomgit-bot
7月16日 评论:

变更摘要

这是一个文档更新 PR,主要针对 Docker 部署说明进行补充。改动范围为 2 个文件,新增 2 行、删除 1 行。根据 PR 描述,此次更新新增了单机多容器部署说明以及 MPI 相关说明,旨在帮助用户更好地理解在多容器环境下以及使用 MPI 进行分布式运行的配置方式。

主要改动

  • 新增单机多容器部署说明:在 Docker 相关文档中补充了单机环境下运行多个容器的配置指引。
  • 新增 MPI 说明:增加了与 MPI(消息传递接口)相关的使用说明,以便用户在使用 MPI 进行分布式计算时参考。
likedislike
不准确?
atomgit-bot
atomgit-bot
7月16日 评论:

代码审查

我已经仔细审查了两个变更文件的完整上下文。以下是我的分析:


文件 1: docs/zh/user_guide/constraints.md

变更内容: 在第 8 行新增了一条约束建议:单机多容器场景下建议所有容器配置相同的 IP 地址。

分析: 该文档是 HCCL(昇腾集合通信库)的系统约束与限制说明。这条建议处于"公共约束"章节,上下文是 HCCL 通信场景。在 HCCL 单机多容器部署中(例如使用 --network=host 模式时,所有容器共享宿主机网络栈),确保 HCCL 通信接口 IP 一致是为了让性能对比测试结果与裸机环境可比。这里的"IP 地址"指的是 HCCL 通信所使用的网络接口 IP(可通过 HCCL_IF_IP 等环境变量配置),而非一般意义上的容器独立 IP。该建议措辞在 HCCL 专业上下文中是合理的,不会导致安全性问题或生产环境误用。

文件 2: examples/05_custom_ops_allgather/README.md

变更内容: 在第 57 行将 MPI 环境说明从简单的一句话扩展为附带文档链接的完整说明,引导用户参考昇腾文档中心的 HCCL 性能测试工具使用指南中的 MPI 安装章节。

分析:

  • 链接使用 HTTPS (https://hiascend.com/...),指向华为昇腾官方文档中心,域名合法可信。
  • 链接未引入不安全的命令或配置(如 chmod 777、禁用 TLS、暴露 0.0.0.0 等)。
  • 内容为指导性文档链接,不涉及跳过测试、隐藏失败或误导用户的行为。
  • URL 中 hiascend.com 与文件中第 47 行已有的 www.hiascend.com 存在微小不一致,但两者均指向同一官方站点,属于纯格式差异,不属于可报告问题。

审查结论

两个变更文件均为纯文档更新,未引入正确性、安全性、可靠性或破坏性变更问题。

优先级 数量
P0 0
P1 0
P2 0
P3 0

整体风险判断: 无风险。本次变更是文档层面的补充说明,内容合理,无安全隐患,可以合并。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
不准确?
CANN-robotCANN-robot成员
7月16日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月16日 评论:

CLA Signature Pass

Leewis, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robot
CANN-robot成员
7月16日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
** flyswa, zhangxp1030 (2/2) flyswa (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
7月16日 将leabclove,yanglianxiang2,ccNeo824,flyswa,chengym,ccl_ligang,linyf950,yanzhenghang,chenliang267,yanyefeng,jiangtao_rts,chenhao_,hanxiaolong,dingweiqin_57,songmingyang,wenxuemin,gcw_kUomxQ2l,yangxiaoyuan,Andy-lb,laodazhao1,yangshaohua,chenke2026,zhongqin2020,bz_lee,Innoecho,chenyia,temper7620,zhangxp1030,abiggg,lilin_137,lilianlin_hw,hulk1997,zhuyiming3,ouyangxizi设为评审人
CANN-robotCANN-robot成员
7月16日 将flyswa,jiangtao_rts,hanxiaolong,gcw_kUomxQ2l,Andy-lb设为审查人
Leewis成员
7月16日 评论:

/compile

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
7月16日 评论:

流水线任务触发成功
任务链接 [69b94b1c009f486e9bb2dd4029dad61f][流水线指导]

任务名称状态日志下载链接
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
StaticCheck_codespell ✅ SUCCESS
StaticCheck_link_validity ✅ SUCCESS
StaticCheck_resource_existence ✅ SUCCESS
StaticCheck_tag_closed ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
UT_Test ✅ SUCCESS >>>>>
ST_Test ✅ SUCCESS >>>>>
codecheck ✅ SUCCESS >>>>>
precommit ✅ SUCCESS >>>>>
codecheck_codestyle ✅ SUCCESS >>>>>
PreSmoke_A900_npupool ✅ SUCCESS >>>>>
PreSmoke_A3_npupool ✅ SUCCESS >>>>>

[2026-07-16 10:02:53]    CI执行结束

likedislike
CANN-robotCANN-robot成员
7月16日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
7月16日 添加了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
7月16日 评论:
流水线 PR-pipeline_cann_hccl#5251 [ commitID:af566a71 ] 已完成
likedislike
zangyan成员
7月16日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:approved
zhangxp1030成员
7月16日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:lgtm
CANN-robotCANN-robot成员
7月16日 合入了pull request