Ignore flaky RestChatAgentWithMcpConnectorIT.testChatAgentWithMcpStreamableHttpConnector

[rithinpullela]

Ignores testChatAgentWithMcpStreamableHttpConnector, which fails intermittently on the multi-node CI leg. Root cause and a proper fix are being tracked separately.

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🧪 No relevant tests

🔒 No security concerns identified

✅ No TODO sections

🔀 No multiple PR themes

⚡ No major issues detected

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Ensure ignored test is actually skipped The @Ignore annotation on JUnit test methods only takes effect when the method is also annotated with @Test. Since this test method appears to rely on JUnit 3-style naming conventions (method prefixed with test) inherited from the OpenSearch test framework, @Ignore may not be respected and the test could still run. Add an explicit @Test annotation or use an Assume.assumeTrue(false) / early return to reliably skip the test. plugin/src/test/java/org/opensearch/ml/rest/RestChatAgentWithMcpConnectorIT.java [139-140] @Ignore("Flaky on multi-node CI: MCP loopback tool call intermittently times out under load") public void testChatAgentWithMcpStreamableHttpConnector() throws IOException { + Assume.assumeTrue("Skipping flaky MCP streamable HTTP connector test", false); Suggestion importance[1-10]: 7 __ Why: Valid concern: OpenSearch test framework extends JUnit 3-style OpenSearchTestCase, where @Ignore may not reliably skip tests without @Test. Adding Assume.assumeTrue(false) ensures the test is skipped, which is a meaningful correctness improvement for CI stability.	Medium