Skip to content

fix(mcp-deployment): Add comprehensive MCP deployment documentation #594

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 4 commits into from
Aug 14, 2025

Conversation

sfkrieger
Copy link
Collaborator

  • Add detailed deployment section for ADK agents with MCP tools
  • Document critical synchronous agent definition requirement for deployment
  • Cover three deployment patterns: self-contained stdio, remote SSE/HTTP, and sidecar containers
  • Include production deployment checklist with security, monitoring, and scalability considerations
  • Provide environment-specific configurations for Cloud Run, GKE, and Vertex AI Agent Engine
  • Add troubleshooting guide for common MCP deployment issues
  • Include quick deployment commands for Agent Engine and Cloud Run

Addresses deployment challenges outlined in issue #327 where async MCP toolset creation is no longer supported in deployment environments.

Fixes #327

- Add detailed deployment section for ADK agents with MCP tools
- Document critical synchronous agent definition requirement for deployment
- Cover three deployment patterns: self-contained stdio, remote SSE/HTTP, and sidecar containers
- Include production deployment checklist with security, monitoring, and scalability considerations
- Provide environment-specific configurations for Cloud Run, GKE, and Vertex AI Agent Engine
- Add troubleshooting guide for common MCP deployment issues
- Include quick deployment commands for Agent Engine and Cloud Run

Addresses deployment challenges outlined in issue #327 where async MCP toolset creation
is no longer supported in deployment environments.

Fixes #327
```python
# deploy_mcp_server.py - Separate Cloud Run service
from mcp.server import Server
from mcp.server.stdio import stdio_server
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done! PTAL... if you have time!

- Update Pattern 2 deployment example to use StreamableHTTPSessionManager
- Replace basic SSE server code with comprehensive Streamable HTTP implementation
- Add production-ready features: stateless mode, ASGI compliance, lifecycle management
- Include Starlette application setup with proper error handling
@sfkrieger sfkrieger enabled auto-merge (squash) August 13, 2025 23:16
@sfkrieger sfkrieger requested a review from seanzhou1023 August 13, 2025 23:19
@sfkrieger sfkrieger merged commit 8e4f503 into main Aug 14, 2025
4 checks passed
@sfkrieger sfkrieger deleted the mcp-deploy-327 branch August 14, 2025 18:13
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.

Add docs for deploying agents with MCPs
3 participants