Skip to content
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

Deployment: Dockerfile and Smithery config #16

Merged
merged 3 commits into from
Feb 12, 2025
Merged

Conversation

smithery-ai[bot]
Copy link
Contributor

@smithery-ai smithery-ai bot commented Feb 12, 2025

This pull request introduces the following updates:

  • Dockerfile: Introduces a Dockerfile to package the MCP for deployment across various environments.
  • Smithery Configuration: Adds a Smithery YAML file, which specifies how to start the MCP and the configuration options it supports. It allows you to deploy your MCP to Smithery, serving it over WebSockets so end-users do not need to install additional dependencies. To deploy, merge this PR, then visit your server page and click "Deploy" under the deployments page.
  • README: Updates the README to include installation instructions via Smithery and a popularity badge. Note that the installation only works after the server is deployed on Smithery.

Please review these updates to verify their accuracy for your server and feel free to customize it to your needs. Let me know if you have any questions. 🙂

@smithery-ai smithery-ai bot requested a review from jfdenise as a code owner February 12, 2025 13:23
WORKDIR /app

# Copy the pom.xml and source code
COPY wildfly-mcp-server/pom.xml ./
Copy link
Collaborator

Choose a reason for hiding this comment

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

the pom.xml is in the current directory, should be COPY pom.xml ./, same for the src directory

COPY --from=builder /app/target/wildfly-mcp-server-1.0.0.Final-SNAPSHOT-runner.jar ./wildfly-mcp-server.jar

# Expose the default port the application will run on
EXPOSE 9990
Copy link
Collaborator

Choose a reason for hiding this comment

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

We are not exposing any port. The server is using STDIO protocol.

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