已合并
test(distributed): fix c10d rendezvous backend upstream patch #34336
Jinfan Liu创建于 4月24日
test(distributed): fix c10d rendezvous backend upstream patch #34336
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 Jinfan Liu 的贡献）4月24日 创建了 pull request，commit 95e153de
ascend-robot
4月24日 评论：
4月24日 评论：
ascend-robot
4月24日 评论：
4月24日 评论：
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 |
|---|---|---|
| repo-Ascend/pytorch | ✅ htchu, huangjingwei (2/2) | ✅ htchu (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
JfanLiu, thanks for your pull request. All authors of the commits have signed the CLA. 👍
4月24日 添加了label：ascend-cla/yes
ascend-robot
4月24日 评论：
4月24日 评论：
当前仓库存在以下 保护分支 ：
| Protected Branch | Version | Release |
|---|---|---|
| master | ||
| v2.10.0 | ||
| v2.11.0 | ||
| v2.9.0 | ||
| v2.7.1 | ||
| v2.8.0 |
评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支（创建同步 PR）：
a) 如果当前 PR 是 Open 状态，同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged，将立即执行同步操作
注意：
- /sync 命令可以指定同步到多个分支，仅最后一个 /sync 命令生效
- 如果创建的同步 PR 不正确，可通过向同步 PR 的源分支提交轻量级 PR 完善，或使用 /close 命令关闭
Jinfan Liu
4月24日 评论：
4月24日 评论：
/compile
Jinfan Liu
4月24日 评论：
4月24日 评论：
compile
4月24日 添加了label：ci-pipeline-running
4月24日 修改了pull request 的描述
4月24日 删除了label：ci-pipeline-running
4月24日 添加了label：ci-pipeline-passed
ascend-robot
4月24日 评论：
4月24日 评论：
流水线 PR-pipeline_pytorch#17055 已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | ✅ | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| CodeCheck_lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_01 | ✅ | >>> | |
| UT_DIST_X86_Part_02 | ✅ | >>> | |
| UT_DIST_X86_Part_03 | ✅ | >>> | |
| UT_DIST_X86_Part_04 | ✅ | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| UT_ARM_A2_Part_03 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
AtlasAccount
4月25日 评论：
4月25日 评论：
compile
4月25日 删除了label：ci-pipeline-passed
此处折叠了8条事件消息 查看更多
4月25日 添加了label：ci-pipeline-passed
ascend-robot
4月25日 评论：
4月25日 评论：
流水线 PR-pipeline_pytorch#17131 （重试第6次）已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | ✅ | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| CodeCheck_lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_01 | ✅ | >>> | |
| UT_DIST_X86_Part_02 | ✅ | >>> | |
| UT_DIST_X86_Part_03 | ✅ | >>> | |
| UT_DIST_X86_Part_04 | ✅ | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| UT_ARM_A2_Part_03 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
4月25日 添加了label：approved
Jingwei Huang
4月25日 评论：
4月25日 评论：
/lgtm
4月25日 添加了label：lgtm
ascend-robot
4月25日 评论：
4月25日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: htchu.
Reviewers who wrote a comment of /lgtm are: huangjingwei, htchu.
4月25日 合入了pull request
PR Title
test(distributed): fix c10d rendezvous backend upstream patch
PR Description
PR描述
【合入来源】
[Usage]: PyTorch 社区
test/distributed/elastic/rendezvous/c10d_rendezvous_backend_test.py里有些测试有些 bug，在当前环境下不能直接整文件稳定通过，因此这里提交对应的 upstream patch 修复。该社区测试文件涉及以下 API：
torch.distributed.elastic.rendezvous.api.RendezvousConnectionErrortorch.distributed.elastic.rendezvous.api.RendezvousError（extendable）请勿携带内部流程信息（需求链接、问题单、内部 issue 等）
https://gitcode.com/Ascend/pytorch/issues/1691
【修改方案】
test_upstream/test/distributed/elastic/rendezvous/c10d_rendezvous_backend_test.py.patchlocalhost调整为127.0.0.1，规避当前环境下的地址解析与连通性差异。TCPStore引用，避免临时对象过早释放，保证CreateBackendTest相关场景稳定通过。【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
test/distributed/elastic/rendezvous目录执行：python -m unittest c10d_rendezvous_backend_test.pyRan 30 tests in 32.463sOKv2.8.02.8.0+cpu2.8.0.post2整体结论
本次提交只聚焦
c10d_rendezvous_backend_test.py的 upstream patch 修复。应用 patch 后，该社区测试文件已在对应版本环境整文件验证通过。