Skip to content

fix(docker): 修复重复 Cargo registry 定义并增加容器 CI - #2

Merged
NetLops merged 2 commits into
mainfrom
fix/docker-cargo-registry
Oct 12, 2026
Merged

NetLops merged 2 commits into
mainfrom
fix/docker-cargo-registry

Conversation

@NetLops

@NetLops NetLops commented Oct 12, 2026 •

Copy link
Copy Markdown
Owner

执行 make docker-compose-up 时,Dockerfile 将同一个 rsproxy sparse registry 同时声明为 source.rsproxy 和 source.rsproxy-sparse,导致 Cargo 在编译前以退出码 101 中止。

此 PR 删除重复 source,保留现有 rsproxy 镜像和 replace-with 配置,并使用 --locked 构建。另清理 issue 日志中的 Docker/Compose 警告,排除本地 .cargo 和 target,避免覆盖镜像配置或传输本机构建产物。

新增 Docker Actions workflow,并将 Makefile 纳入 push / pull_request 的触发路径,运行原有 make docker-compose-up,随后验证 HTTP 缺少查询参数时返回 400,以及容器内 MCP initialize、tools/list 和 ping。测试不依赖公共搜索引擎的可用性。

验证结果:

  • PR Actions 全部成功:Compose 配置、完整镜像构建、容器启动、HTTP 与 MCP smoke checks。
  • Push Actions 全部成功。
  • 本地 Cargo 回归复现:原 Dockerfile 生成的配置报重复源错误;修复配置执行 offline cargo check 成功。
  • 本地 Compose 配置检查和原生 locked release build 成功。

关联 #1。按要求等待 issue 作者 @320656 在原环境测试确认后,再合并并关闭 issue。

@NetLops
NetLops marked this pull request as ready for review October 12, 2026 03:12
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-12T03:14:51.571160Z 7e64e9f Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7e64e9fdea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/docker.yml Resolved
@NetLops
NetLops merged commit 1232f2f into main Oct 12, 2026
2 checks passed
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.

1 participant