Skip to content

Conversation

@mhajder
Copy link
Contributor

@mhajder mhajder commented Aug 3, 2025

This PR adds Docker file for normal usage in Claude Desktop or VS Code and Docker file with mcpo for usage with OpenWebUI.

Images have multi-stage build which allows to obtain an image with a small size.
Dockerfile -> 66.3MB
Dockerfile.mcpo -> 85.5MB

Requires #25 because the images themselves also need uv.lock for security.
I think mcpo with this MCP server is working correctly, I can't reproduce the problem as in #20.

@mhajder mhajder force-pushed the feature/dockerfile branch from ed92c96 to 19a3a06 Compare August 6, 2025 11:22
@mhajder mhajder force-pushed the feature/dockerfile branch from 19a3a06 to ed8cc65 Compare August 6, 2025 11:24
abubnalitic-nbl added a commit that referenced this pull request Oct 27, 2025
Add single Dockerfile for containerized deployment of NetBox MCP server.
Focused on mainline MCP use cases (Claude Desktop, Claude Code, MCP-aware clients).

Docker Image Details:
- Multi-stage Alpine-based build (140MB final size)
- Non-root user (appuser:1000) for security
- Build cache optimization via mount directives
- EXPOSE 8000 directive for documentation
- Follows Docker conventions (trailing newlines, .dockerignore)

Configuration:
- All settings via environment variables
- Supports HTTP transport (required for Docker)
- Full configuration system integration
- Examples in README for common scenarios

OpenWebUI Integration:
- Documented approach using standalone mcpo wrapper
- Keeps repo focused on core MCP implementation
- Users can extend with protocol translators as needed

Architectural Decision:
- Dockerfile.mcpo intentionally not included
- MCPO is a generic MCP→OpenAPI translator
- Not NetBox-specific, better as separate tool
- Documentation provided for users who need it

Testing:
- Image builds successfully (140MB)
- EXPOSE 8000 verified
- Non-root user confirmed
- HTTP transport tested

Based on excellent work in PR #26 by @mhajder
- Adopted multi-stage Alpine build pattern
- Security practices (non-root, minimal deps)
- Build optimization techniques

Co-authored-by: Mateusz Hajder <[email protected]>
Co-authored-by: Anton Bubna-Litic <[email protected]>
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