Skip to content

Conversation

@ivyjeong13
Copy link
Contributor

Addresses #4182

@ivyjeong13 ivyjeong13 force-pushed the enhance/remove-auto-delete-and-see-logs-of-mcp-failure-on-setup branch from 6a8f607 to 4a22cdf Compare October 16, 2025 14:50
@ivyjeong13
Copy link
Contributor Author

Wanted to test this again before merging, but looks like /logs is no longer working for me when trying to run with the Tavily example case

After getting the 503 from /launch, trying to hit /logs gives me:
500 server running, but readiness check failed: server readiness check failed: MCP server is not healthy

Before, /logs would be successful and I could get the logs info showing that I am missing an environment variable. Example of what was happening before: https://www.loom.com/share/23d35958db214d99a48b99349b42e5e4

Wondering if there was a change with Tavily or on our side and holding off merge to see if I can try with another MCP server!

@ivyjeong13
Copy link
Contributor Author

Wanted to test this again before merging, but looks like /logs is no longer working for me when trying to run with the Tavily example case

After getting the 503 from /launch, trying to hit /logs gives me: 500 server running, but readiness check failed: server readiness check failed: MCP server is not healthy

Before, /logs would be successful and I could get the logs info showing that I am missing an environment variable. Example of what was happening before: https://www.loom.com/share/23d35958db214d99a48b99349b42e5e4

Wondering if there was a change with Tavily or on our side and holding off merge to see if I can try with another MCP server!

Was able to determine there was a backend change that made it so /logs will return a 500 if the deployment fails for any reason. Waiting on a backend PR to merge #4530 to verify that /logs can be hit if due to invalid/not provided environment variable.

@ivyjeong13 ivyjeong13 force-pushed the enhance/remove-auto-delete-and-see-logs-of-mcp-failure-on-setup branch from 5f858b5 to ce90e62 Compare October 22, 2025 20:07
@ivyjeong13 ivyjeong13 force-pushed the enhance/remove-auto-delete-and-see-logs-of-mcp-failure-on-setup branch from ce90e62 to 1f69607 Compare October 23, 2025 15:21
@ivyjeong13 ivyjeong13 force-pushed the enhance/remove-auto-delete-and-see-logs-of-mcp-failure-on-setup branch from 77a6fed to 0b718b7 Compare October 23, 2025 17:13
Copy link
Member

@g-linville g-linville left a comment

Choose a reason for hiding this comment

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

Backend lgtm

@ivyjeong13 ivyjeong13 merged commit d4612e3 into obot-platform:main Oct 23, 2025
5 of 6 checks passed
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.

4 participants