Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 30, 2026

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.sh
  • actions/setup/sh/verify_mcp_gateway_health.sh

Progress visibility:

  • Logs attempt count every 10 retries with elapsed time
  • Shows success message with attempt number and timing
  • Reports total attempts in final summary

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: 35

Previously, 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

  • Type: patch
  • Description: Add progress logging to the MCP gateway health check script so retries report elapsed time and total attempts.

Ahoy! This treasure was crafted by 🏴‍☠️ Changeset Generator

Copilot AI and others added 2 commits January 30, 2026 18:52
- 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>
Copilot AI changed the title [WIP] Add logging for health check phase in start_mcp_gateway.sh Add retry progress logging to MCP gateway health checks Jan 30, 2026
@pelikhan pelikhan marked this pull request as ready for review January 30, 2026 19:12
@pelikhan pelikhan added the smoke label Jan 30, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

🎉 Yo ho ho! Changeset Generator found the treasure and completed successfully! ⚓💰

@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions
Copy link
Contributor

📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing...

@github-actions
Copy link
Contributor

Agent Container Tool Check

Tool Status Version
bash 5.2.21
sh not found
git 2.52.0
jq 1.7
yq 4.50.1
curl 8.5.0
gh 2.86.0
node 24.13.0
python3 3.13.11
go 1.24.12
java not found
dotnet not found

Result: 9/12 tools available ❌ (3 missing)

Missing tools: sh, java, dotnet

AI generated by Agent Container Smoke Test

@lpcox lpcox merged commit 1659ea6 into main Jan 30, 2026
83 checks passed
@lpcox lpcox deleted the copilot/add-health-check-logging branch January 30, 2026 19:17
@github-actions
Copy link
Contributor

🤖 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 🎯

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

Smoke Test Results ✅

Last 2 Merged PRs:

Test Status:
✅ GitHub MCP | ✅ Safe Inputs | ✅ Serena | ✅ Playwright | ✅ File Writing | ✅ Bash | ✅ Discussion | ✅ Build

Overall: PASS

cc @Copilot @lpcox

AI generated by Smoke Copilot

@github-actions
Copy link
Contributor

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

PR titles:
Fix: Add temporary ID resolution to update_project handler
Add timing instrumentation to MCP gateway setup scripts
1 MCP merged PRs ✅ | 2 safeinputs-gh ✅ | 3 Serena ✅
4 Playwright ✅ | 5 Tavily ✅ | 6 File write ✅
7 Bash verify ✅ | 8 Discussion comment ✅ | 9 Build ✅
Overall status: PASS

AI generated by Smoke Codex

@github-actions
Copy link
Contributor

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

AI generated by Smoke Claude

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants