已开启
fix(deployment): explain supervisor connection failures #59
fix(deployment): explain supervisor connection failures #59
已开启
庄昭雄创建于 15 天前
庄昭雄
庄昭雄
15 天前

Paired: GitHub #48 ↔ GitCode !59

probe

likedislike
合并受阻
atomgit-bot
atomgit-bot
15 天前 评论:

变更摘要

本次变更聚焦于 platform/deployment/src/supervisor.rs 中 request 函数的连接诊断能力,在连接 supervisor.sock 失败或超时时给出更明确的错误提示,帮助定位 supervisor 未运行或 state_dir 与运行实例不一致的问题;同时在 platform/deployment/tests/process.rs 中补充了对应的测试用例。

主要改动

  • 补充缺失 supervisor 的诊断信息: 在 request 函数中新增 socket 变量(root.join("supervisor.sock")),并将 UnixStream::connect 的错误包装为提示连接失败原因(supervisor 未运行或 deployment state_dir 与运行实例不同)的信息。
  • 补充超时未响应的诊断信息: 将超时结果由直接 ? 传递改为 map_err,返回包含超时时间与 socket 路径、说明 supervisor 可能已停止或 state_dir 不一致的错误信息。
  • 新增缺失 supervisor 的测试: 在 process.rs 中新增 request_explains_a_missing_supervisor 测试,验证连接不存在的 socket 时错误中包含 supervisor 与 state_dir。
  • 新增陈旧 socket 的测试: 新增 request_explains_a_stale_supervisor_socket 测试,通过绑定后立即释放监听器模拟陈旧 socket,验证错误提示同样包含 supervisor 与 state_dir。
likedislike
不准确?
atomgit-bot
atomgit-bot
15 天前 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
openJiuwen-bot成员
15 天前 评论:

欢迎来到 openJiuwen 社区

Hey @grandocean , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 此处 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。

likedislike
atomgit-bot
atomgit-bot
15 天前 评论:

AtomGit AI 助手使用指南

在 PR 评论中使用命令即可触发。支持在普通评论和代码行评论中使用。

功能 命令 说明
代码审查 /ai review 检查代码质量、潜在问题、安全风险
PR 摘要 /ai summary 生成 PR 变更内容的结构化摘要
代码解释 /ai explain 解释代码变更的逻辑和意图
自由提问 @atomgit-bot <问题> 询问关于本 PR 的任何问题
帮助 /ai help 显示此帮助信息

免责声明

AI 助手可能存在误判,请结合自身判断。可以对评论点 👍 或 👎 帮助我们改进。

likedislike
不准确?
OopenJiuwen-bot成员
15 天前 添加了label:openJiuwen-cla/yes
openJiuwen-bot成员
15 天前 评论:

CLA 签署成功

grandocean, 感谢您的代码贡献。 所有的代码贡献者都已完成了 CLA 签署。 👍

likedislike
庄昭雄庄昭雄
15 天前 修改了pull request 的描述
Oopenjiuwen-sync成员
15 天前 修改了pull request 的描述
openJiuwen-bot成员
15 天前 评论:

The pipeline(pipeline number:59) is running. Please wait a moment...

likedislike
OopenJiuwen-bot成员
15 天前 添加了label:ci-running
OopenJiuwen-bot成员
15 天前 删除了label:ci-running
openJiuwen-bot成员
15 天前 评论:
任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
禁用词扫描 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 N/A N/A
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck ✅SUCCESS N/A
issue关联检查 N/A N/A
pr size检查 N/A N/A
likedislike
OopenJiuwen-bot成员
15 天前 添加了label:ci-successful