Skip to content

Add MCP configuration for docker community edition with 'docker-ce' profile #424

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 1, 2025

Conversation

gabz57
Copy link

@gabz57 gabz57 commented May 29, 2025

Overview

  • Adds support for MCP via Docker without the need to install Docker Desktop Extension (Docker Desktop is a product under license)
  • Enabled with use of the docker-ce profile (instead of docker-desktop)

Details

Added a list of MCPs (currently used by examples)

  • mcp/fetch
  • mcp/wikipedia-mcp
  • mcp/google-maps (require env. variable: GOOGLE_MAPS_API_KEY)
  • mcp/puppeteer
  • mcp/github (require env. variable: GITHUB_PERSONAL_ACCESS_TOKEN)
  • mcp/brave-search (require env. variable: BRAVE_API_KEY)

@gabz57 gabz57 force-pushed the docker-mcp branch 3 times, most recently from f68d44d to a991c53 Compare May 29, 2025 15:30
@johnsonr
Copy link
Contributor

Thank you for the contribution. Please add a description explaining the motivation for this PR.

@slimslenderslacks (from Docker), can you please review?

@slimslenderslacks
Copy link

@gabz57 just as a side note, we are looking into opensourcing the mcp gateway so that we could also use the same configuration for the community edition of Docker.

However, at the present moment, this docker-mcp profile is definitely necessary to use the community edition of Docker. We could call this docker-ce (as opposed to docker-desktop) to highlight that it requires docker to be installed but not Docker Desktop.

@slimslenderslacks slimslenderslacks self-requested a review May 31, 2025 06:00
@gabz57
Copy link
Author

gabz57 commented May 31, 2025

Thank you for the review, I understand the intention, I applied the renaming of the file to docker-ce.
I also added the fetch mcp in the list as it is present in README.md

@gabz57 gabz57 force-pushed the docker-mcp branch 2 times, most recently from 20d8236 to 7bc1f70 Compare May 31, 2025 10:09
Signed-off-by: Arnaud Schwartz <arnaud.schwartz@octo.com>
Copy link

@gabz57 gabz57 changed the title Add mcp configuration directly with docker Add MCP configuration for docker community with 'docker-ce' profile May 31, 2025
@gabz57 gabz57 changed the title Add MCP configuration for docker community with 'docker-ce' profile Add MCP configuration for docker community edition with 'docker-ce' profile May 31, 2025
@johnsonr johnsonr merged commit 5be09aa into embabel:main Jun 1, 2025
3 checks passed
@gabz57 gabz57 deleted the docker-mcp branch June 1, 2025 23:22
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.

3 participants