Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates Docker deployment manifests to use newer FastGPT-related container images, and adjusts the production compose template’s FE_DOMAIN default.
Changes:
- Bump
fastgpt,fastgpt-code-sandbox, andfastgpt-mcp_serverimage tags tov4.14.15across global/cn compose files. - Bump
fastgpt-pluginimage tag tov0.6.1across compose files. - Update
deploy/templates/docker-compose.prod.ymlto leaveFE_DOMAINblank; aligndeploy/args.jsontags with the new versions.
Reviewed changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| document/public/deploy/docker/global/docker-compose.zilliz.yml | Bumps FastGPT-related image tags (global registry). |
| document/public/deploy/docker/global/docker-compose.seekdb.yml | Bumps FastGPT-related image tags (global registry). |
| document/public/deploy/docker/global/docker-compose.pg.yml | Bumps FastGPT-related image tags (global registry). |
| document/public/deploy/docker/global/docker-compose.opengauss.yml | Bumps FastGPT-related image tags (global registry). |
| document/public/deploy/docker/global/docker-compose.oceanbase.yml | Bumps FastGPT-related image tags (global registry). |
| document/public/deploy/docker/global/docker-compose.milvus.yml | Bumps FastGPT-related image tags (global registry). |
| document/public/deploy/docker/cn/docker-compose.zilliz.yml | Bumps FastGPT-related image tags (CN registry). |
| document/public/deploy/docker/cn/docker-compose.seekdb.yml | Bumps FastGPT-related image tags (CN registry). |
| document/public/deploy/docker/cn/docker-compose.pg.yml | Bumps FastGPT-related image tags (CN registry). |
| document/public/deploy/docker/cn/docker-compose.opengauss.yml | Bumps FastGPT-related image tags (CN registry). |
| document/public/deploy/docker/cn/docker-compose.oceanbase.yml | Bumps FastGPT-related image tags (CN registry). |
| document/public/deploy/docker/cn/docker-compose.milvus.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/templates/docker-compose.prod.yml | Changes FE_DOMAIN default from localhost to blank. |
| deploy/docker/global/docker-compose.zilliz.yml | Bumps FastGPT-related image tags (global registry). |
| deploy/docker/global/docker-compose.seekdb.yml | Bumps FastGPT-related image tags (global registry). |
| deploy/docker/global/docker-compose.pg.yml | Bumps FastGPT-related image tags (global registry). |
| deploy/docker/global/docker-compose.opengauss.yml | Bumps FastGPT-related image tags (global registry). |
| deploy/docker/global/docker-compose.oceanbase.yml | Bumps FastGPT-related image tags (global registry). |
| deploy/docker/global/docker-compose.milvus.yml | Bumps FastGPT-related image tags (global registry). |
| deploy/docker/cn/docker-compose.zilliz.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/docker/cn/docker-compose.seekdb.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/docker/cn/docker-compose.pg.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/docker/cn/docker-compose.opengauss.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/docker/cn/docker-compose.oceanbase.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/docker/cn/docker-compose.milvus.yml | Bumps FastGPT-related image tags (CN registry). |
| deploy/dev/docker-compose.yml | Bumps dev compose images for sandbox/mcp/plugin (global registry). |
| deploy/dev/docker-compose.cn.yml | Bumps dev compose images for sandbox/mcp/plugin (CN registry). |
| deploy/args.json | Updates image tag mappings to the new versions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # ==================== 基础配置 ==================== | ||
| # 前端外部可访问的地址，用于自动补全文件资源路径。例如 https:fastgpt.cn，不能填 localhost。这个值可以不填，不填则发给模型的图片会是一个相对路径，而不是全路径，模型可能伪造Host。 | ||
| FE_DOMAIN: http://localhost:3000 | ||
| FE_DOMAIN: |
There was a problem hiding this comment.
FE_DOMAIN: with an empty value will explicitly set the container env var to an empty string, which is not the same as omitting FE_DOMAIN entirely. If the application treats an empty string as a configured (but invalid) value, this can change behavior vs leaving it unset. Prefer either removing the FE_DOMAIN entry from the template (and keeping the comment telling users when to add it), or using compose list-style pass-through (- FE_DOMAIN) so it only gets set when the user provides it in their shell/environment.
| fastgpt-app: | ||
| container_name: fastgpt-app | ||
| image: ghcr.io/labring/fastgpt:v4.14.12 | ||
| image: ghcr.io/labring/fastgpt:v4.14.15 |
There was a problem hiding this comment.
This line contains trailing whitespace after the image tag. Please remove trailing spaces to avoid noisy diffs and potential YAML lint failures (this occurs in multiple updated image: lines in this PR).
| image: ghcr.io/labring/fastgpt:v4.14.15 | |
| image: ghcr.io/labring/fastgpt:v4.14.15 |
| "fastgpt": "v4.14.15", | ||
| "fastgpt-code-sandbox": "v4.14.15", | ||
| "fastgpt-mcp_server": "v4.14.15", | ||
| "fastgpt-plugin": "v0.6.1", |
There was a problem hiding this comment.
The same version/tag values are now duplicated across deploy/args.json and many docker-compose files. This increases the chance of future inconsistencies and makes upgrades more error-prone. Consider making deploy/args.json the single source of truth (e.g., generate compose files from it), or refactor compose files to reference centralized variables/anchors so version bumps require editing only one place.
|
✅ Docs Preview Deployed! 🔗 👀 Click here to visit preview |
|
✅ Build Successful - Preview code-sandbox Image for this PR: |
|
✅ Build Successful - Preview fastgpt Image for this PR: |
|
✅ Build Successful - Preview mcp_server Image for this PR: |
No description provided.