Skip to content

Move MCP gateway logs from /tmp/gh-aw/mcp-logs/gateway/ to /tmp/gh-aw/mcp-logs/#9617

Merged
pelikhan merged 4 commits into
mainfrom
copilot/update-mcp-logs-path
Jan 11, 2026
Merged

Move MCP gateway logs from /tmp/gh-aw/mcp-logs/gateway/ to /tmp/gh-aw/mcp-logs/#9617
pelikhan merged 4 commits into
mainfrom
copilot/update-mcp-logs-path

Conversation

Copilot AI commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

Flattens the MCP gateway logs directory structure by moving logs from /tmp/gh-aw/mcp-logs/gateway/ to /tmp/gh-aw/mcp-logs/, eliminating unnecessary nesting.

Changes

  • start_mcp_gateway.sh: Create /tmp/gh-aw/mcp-logs/ directly, write stderr to stderr.log instead of gateway/stderr.log, export MCP_GATEWAY_LOG_DIR=/tmp/gh-aw/mcp-logs/
  • parse_mcp_gateway_log.cjs: Update log paths from gateway/gateway.loggateway.log and gateway/stderr.logstderr.log
  • mcp_servers.go: Pass MCP_GATEWAY_LOG_DIR environment variable to gateway container via -e flag

Path Changes

- /tmp/gh-aw/mcp-logs/gateway/gateway.log
- /tmp/gh-aw/mcp-logs/gateway/stderr.log
+ /tmp/gh-aw/mcp-logs/gateway.log
+ /tmp/gh-aw/mcp-logs/stderr.log

The gateway container receives MCP_GATEWAY_LOG_DIR to configure its log output location.

Original prompt

Make the folder for MCP gateway logs /tmp/gh-aw/mcp-logs/, not /tmp/gh-aw/mcp-logs/gateway/

  • look for /tmp/gh-aw/mcp-logs/gateway/gateway.log and convert to /tmp/gh-aw/mcp-logs/gateway.log

  • setup MCP_GATEWAY_LOGS_DIR env var with /tmp/gh-aw/mcp-logs/ and pass env var to the MCP gateway


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot variable name should be MCP_GATEWAY_LOG_DIR (log singular)

Copilot AI and others added 2 commits January 11, 2026 13:48
…h-aw/mcp-logs/gateway/

- Updated start_mcp_gateway.sh to:
  - Create /tmp/gh-aw/mcp-logs/ instead of /tmp/gh-aw/mcp-logs/gateway/
  - Write stderr.log directly to /tmp/gh-aw/mcp-logs/stderr.log
  - Set MCP_GATEWAY_LOGS_DIR environment variable
  - Pass MCP_GATEWAY_LOGS_DIR to the gateway container
- Updated parse_mcp_gateway_log.cjs to use new log paths
- Updated mcp_servers.go to pass MCP_GATEWAY_LOGS_DIR to container
- Recompiled all workflow lock files with new paths

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update MCP gateway logs directory and configuration Move MCP gateway logs from /tmp/gh-aw/mcp-logs/gateway/ to /tmp/gh-aw/mcp-logs/ Jan 11, 2026
Copilot AI requested a review from pelikhan January 11, 2026 13:57
@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot use MCP_GATEWAY_LOG_DIR , not MCP_GATEWAY_LOGS_DIR (singular)

- Updated start_mcp_gateway.sh to use MCP_GATEWAY_LOG_DIR
- Updated mcp_servers.go to pass MCP_GATEWAY_LOG_DIR to container
- Recompiled all workflow lock files with new variable name

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Jan 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot variable name should be MCP_GATEWAY_LOG_DIR (log singular)

Done. Renamed to MCP_GATEWAY_LOG_DIR (singular) in dcf598b.

Copilot AI commented Jan 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot use MCP_GATEWAY_LOG_DIR , not MCP_GATEWAY_LOGS_DIR (singular)

Already done in dcf598b - the variable is now MCP_GATEWAY_LOG_DIR (singular) throughout the codebase.

@github-actions

github-actions Bot commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions

github-actions Bot commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test Results

Last 2 merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved merged PRs
  • ✅ Go Tool: go1.25.0 verified
  • ❌ Playwright: Page loaded but evaluation error
  • ✅ File Writing: Test file created successfully
  • ✅ Bash Tool: File verification passed

Overall: PARTIAL PASS (5/6 tests passed)

AI generated by Smoke Claude

@github-actions

Copy link
Copy Markdown
Contributor

Smoke test (Codex):
PR titles: unavailable (no GitHub auth)
GitHub MCP merged PRs: ❌
Serena go (go version): ❌
Playwright github.com title: ❌
safeinputs-gh issues list: ❌
File write/read: ✅ /tmp/gh-aw/agent/smoke-test-codex-20896552515.txt
Overall: FAIL

AI generated by Smoke Codex

@pelikhan pelikhan marked this pull request as ready for review January 11, 2026 14:32
@pelikhan pelikhan merged commit 73f6c6b into main Jan 11, 2026
36 of 37 checks passed
@pelikhan pelikhan deleted the copilot/update-mcp-logs-path branch January 11, 2026 14:32
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.

2 participants