Skip to content

fix(launchmybakery): fix Google Cloud MCP cold starts with explicit timeouts#31

Merged
jeffonelson merged 1 commit intogoogle:mainfrom
rajukandula9:fix/launchmybakery-mcp-timeouts
Mar 27, 2026
Merged

fix(launchmybakery): fix Google Cloud MCP cold starts with explicit timeouts#31
jeffonelson merged 1 commit intogoogle:mainfrom
rajukandula9:fix/launchmybakery-mcp-timeouts

Conversation

@rajukandula9
Copy link
Copy Markdown
Contributor

  • Added timeout and sse_read_timeout to StreamableHTTPConnectionParams for both Maps and BigQuery toolsets.
  • This prevents the underlying mcp-python-sdk from dropping the connection during an initial Google Cloud MCP server cold start.
  • Fixes the anyio.WouldBlock exception that occurs when the server takes longer than the default 5 seconds to wake up and return the tools/list.

- Added `timeout` and `sse_read_timeout` to StreamableHTTPConnectionParams for both Maps and BigQuery toolsets.
- This prevents the underlying mcp-python-sdk from dropping the connection during an initial Google Cloud MCP server cold start.
- Fixes the `anyio.WouldBlock` exception that occurs when the server takes longer than the default 5 seconds to wake up and return the tools/list.
@rajukandula9 rajukandula9 changed the title fix(launchmybakery): add connection timeouts for MCP endpoints fix(launchmybakery): fix Google Cloud MCP cold starts with explicit timeouts Mar 13, 2026
Copy link
Copy Markdown
Collaborator

@jeffonelson jeffonelson left a comment

Choose a reason for hiding this comment

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

LGTM - This prevents the client from prematurely dropping the request & allows the hosted MCPs to overcome cold starts.

@jeffonelson jeffonelson merged commit 657980b into google:main Mar 27, 2026
2 checks passed
Copy link
Copy Markdown

@thepointer1982 thepointer1982 left a comment

Choose a reason for hiding this comment

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

VH

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