Skip to content

fix: 修复 sandbox 网络名称不匹配导致 agent 执行超时的问题#613

Merged
xerrors merged 2 commits intoxerrors:mainfrom
Utopin:fix/sandbox-network-name
Apr 5, 2026
Merged

fix: 修复 sandbox 网络名称不匹配导致 agent 执行超时的问题#613
xerrors merged 2 commits intoxerrors:mainfrom
Utopin:fix/sandbox-network-name

Conversation

@Utopin
Copy link
Copy Markdown

@Utopin Utopin commented Apr 5, 2026

  • 在 docker-compose.yml 中显式指定网络名称为 yuxi-know_app-network
  • 更新 .env.template 添加说明注释

问题原因:sandbox-provisioner 期望连接到 yuxi-know_app-network 网络, 但 docker compose 默认根据项目目录名创建网络,导致网络名称不匹配,
sandbox 容器无法正确获取端口映射,sandbox_url 为空,所有操作超时。

变更描述

简要描述这个 PR 做了什么

变更类型

  • 新功能
  • Bug 修复
  • 文档更新
  • 其他

测试

  • 已在 Docker 环境测试
  • 相关功能正常工作

相关日志或者截图

说明

(可选)有什么需要特别说明的吗?


💡 提示: 提交前可以运行 make lintmake format 检查代码规范

- 在 docker-compose.yml 中显式指定网络名称为 yuxi-know_app-network
- 更新 .env.template 添加说明注释

问题原因:sandbox-provisioner 期望连接到 yuxi-know_app-network 网络,
但 docker compose 默认根据项目目录名创建网络,导致网络名称不匹配,
sandbox 容器无法正确获取端口映射,sandbox_url 为空,所有操作超时。
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

该 PR 通过在 docker-compose.yml 中固定 Compose 网络的实际名称,解决 sandbox-provisioner 依赖的网络名与 Docker Compose 默认生成网络名不一致,从而导致 sandbox_url 为空、agent 执行超时的问题。

Changes:

  • docker-compose.yml 中为 app-network 显式指定网络名 yuxi-know_app-network
  • .env.template 中补充注释说明该网络名约定

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docker-compose.yml 为 app-network 指定固定的 Docker 网络名,避免 compose 项目名变化导致网络名不匹配
.env.template 增加注释,提示 sandbox Docker backend 使用的网络名约定

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker-compose.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@xerrors xerrors merged commit 43188c7 into xerrors:main Apr 5, 2026
@supreme0597
Copy link
Copy Markdown
Contributor

这可能就是我之前遇到的问题

@Utopin Utopin deleted the fix/sandbox-network-name branch April 20, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants