-
Notifications
You must be signed in to change notification settings - Fork 43
Add retry progress logging to MCP gateway health checks #12808
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
Conversation
- Replace curl --retry with custom retry loop for better visibility - Show progress every 10 attempts with attempt count and elapsed time - Display total attempts made when health check completes - Add clear section markers for health check progress - Improve debugging with final status summary Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
- Apply same custom retry loop with progress logging - Show progress every 10 attempts with elapsed time - Display total attempts made - Consistent logging format with start_mcp_gateway.sh Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰 |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
Agent Container Tool Check
Result: 9/12 tools available ❌ (3 missing) Missing tools: sh, java, dotnet
|
|
🤖 Smoke test agent reporting! Just passing through to validate all systems are operational. Running tests, checking integrations, making sure everything works as expected. All green! 🚀 Beep boop - your friendly neighborhood testing bot 🎯
|
Smoke Test Results ✅Last 2 Merged PRs:
Test Status: Overall: PASS cc
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
|
PR titles:
|
|
Claude Smoke Test Results
Test Results: ✅ GitHub MCP | ✅ Safe Inputs GH CLI | ✅ Serena MCP | ✅ Make Build | ✅ Playwright | ✅ Tavily | ✅ File Writing | ✅ Bash | ✅ Discussion Overall Status: PASS
|
MCP gateway health checks can take 40-120 seconds during startup with no progress indication. Operators cannot distinguish between a hung process and a slow-starting gateway.
Changes
Replaced silent curl retry with custom retry loop in:
actions/setup/sh/start_mcp_gateway.shactions/setup/sh/verify_mcp_gateway_health.shProgress visibility:
Example Output
=== Health Check Progress === Attempt 1/120 (0s elapsed)... Attempt 11/120 (10s elapsed)... Attempt 21/120 (20s elapsed)... Attempt 31/120 (30s elapsed)... ✓ Health check succeeded on attempt 35 (34s elapsed) === End Health Check Progress === Final curl exit code: 0 Final HTTP code: 200 Total attempts: 35Previously, the same scenario showed no output between "Retrying up to 120 times..." and the final success message 34 seconds later.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.
Changeset