Skip to content

Update:Start Docker services with Docker Compose, speed up the docker build process - #209

Merged
hustcc merged 6 commits into
antvis:mainfrom
Kevin-Glaser:kevinglaser
Oct 11, 2025
Merged

Update:Start Docker services with Docker Compose, speed up the docker build process#209
hustcc merged 6 commits into
antvis:mainfrom
Kevin-Glaser:kevinglaser

Conversation

@Kevin-Glaser

Copy link
Copy Markdown

Update:Start Docker services with Docker Compose, removing some unnecessary files during the Docker image build process to speed up the build.
Use the Dockerfile in the root directory to execute docker build process and Run docker compose up -d

docker build -t mcp-server-chart:stable .

Run the command:

docker logs -f mcp-server-chart-1

The log should be:

SSE Server listening on http://0.0.0.0:1122/sse

II have already tested the result of this docker build using the Dify platform, as shown in the picture.
微信图片_20251004232628_93_106
original

…essary files during the Docker image build process to speed up the build.
@Kevin-Glaser

Kevin-Glaser commented Oct 4, 2025

Copy link
Copy Markdown
Author

@hustcc

hustcc commented Oct 9, 2025

Copy link
Copy Markdown
Member

@Kevin-Glaser The ability of --host has been merged in this PR #185, and there are some conflicts with the current branch. I need your help to resolve them, thanks!

@Kevin-Glaser

Copy link
Copy Markdown
Author

@Kevin-Glaser The ability of --host has been merged in this PR #185, and there are some conflicts with the current branch. I need your help to resolve them, thanks!

OK! I've merged it. I will run a test to see if it still works

Comment out the duplicated host field
@Kevin-Glaser

Kevin-Glaser commented Oct 9, 2025

Copy link
Copy Markdown
Author

@Kevin-Glaser The ability of --host has been merged in this PR #185, and there are some conflicts with the current branch. I need your help to resolve them, thanks!

I found a duplicate 'host' field in src/index.js. I am not sure about the purpose of adding this, so I temporarily commented out this field and then rebuilt the Docker image, and the result was fine.
6bd285bf-55ed-46be-a215-0adc69b79bed
original

Comment thread src/index.ts Outdated
Comment thread src/index.ts Outdated
Comment thread src/server.ts Outdated
Kevin-Glaser and others added 3 commits October 10, 2025 21:16
Removed duplicate host configuration and updated structure.
Updated comments in docker-compose.yaml for clarity.
@hustcc

hustcc commented Oct 11, 2025

Copy link
Copy Markdown
Member

@Kevin-Glaser Thank you for your professional PR, which has made this project better on Docker components. We also hope that you can continue to pay attention in the future.

@hustcc
hustcc merged commit bd46b3c into antvis:main Oct 11, 2025
1 check passed
@hustcc

hustcc commented Oct 11, 2025

Copy link
Copy Markdown
Member

@hustcc

hustcc commented Oct 28, 2025

Copy link
Copy Markdown
Member

@Kevin-Glaser Whether the docker dir can be removed?

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.

2 participants