fix(ci): start Redis before the MCP server; fail fast when MCP never comes up - #3056
Open
gewenyu99 wants to merge 1 commit into
Open
fix(ci): start Redis before the MCP server; fail fast when MCP never comes up#3056gewenyu99 wants to merge 1 commit into
gewenyu99 wants to merge 1 commit into
Conversation
…comes up posthog services/mcp (hono) hard-exits without Redis at localhost:6379, so every wizard-ci cell lost its MCP and reported an agent failure per app. Start Redis first (docker/brew/apt per runner OS) and turn the 2-minute MCP readiness warning into a hard error so infra outages fail one step early instead of burning a full agent run per matrix cell. Generated-By: PostHog Code Task-Id: 2d43c175-646e-4d73-a974-760280c8c95b
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Every wizard-ci cell has been failing since posthog master's MCP hono server began hard-exiting without Redis (
[MCP] Failed to connect to Redis … process.exit(1)at services/mcp/src/hono/index.ts). The workflow never started one, the readiness loop only warned, and each matrix cell then ran a full agent that failed at the dashboard task — ~40 wasted LLM runs per trigger reporting an infra outage as wizard failures.Two changes in the Run Wizard CI step: start Redis before the MCP server (docker on ubuntu, brew fallback on the macos/swift runner, apt as last resort), and make the 2-minute MCP readiness timeout a hard
::error+ exit 1.Created with PostHog Code