support MCP session management by zane-neo · Pull Request #3803 · opensearch-project/ml-commons

zane-neo · 2025-04-29T13:59:05Z

Description

This PR is to support MCP session management to fix the issue in cluster case, the session could lose if client sends message to load balancer or random coordinators.

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

New functionality includes testing.
New functionality has been documented.
API changes companion pull request created.
Commits are signed per the DCO using --signoff.
Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: zane-neo <zaniu@amazon.com>

ylwu-amzn · 2025-04-29T19:59:48Z

plugin/src/main/java/org/opensearch/ml/action/mcpserver/TransportMcpMessageAction.java

+
+                        @Override
+                        public void handleException(TransportException e) {
+                            System.out.println("got exception:" + e.getMessage());


remove debugging code ?

ylwu-amzn · 2025-04-29T20:01:00Z

plugin/src/main/java/org/opensearch/ml/action/mcpserver/TransportMcpMessageAction.java

+                        @Override
+                        public void handleException(TransportException e) {
+                            System.out.println("got exception:" + e.getMessage());
+                            log.error("got exception: ", e);


got exception is too simple, Make error message more readable

ylwu-amzn · 2025-04-29T20:04:47Z

.../src/main/java/org/opensearch/ml/rest/mcpserver/RestMcpConnectionMessageStreamingAction.java

+                                channel.sendResponse(new BytesRestResponse(channel, e));
                            } catch (IOException ex) {
-                                log.error("Failed to send exception response to client during message handling due to IOException");
+                                log.error("Failed to get the session management index result with sessionId: {}", sessionId);


Error message is misleading. Here the exception should be some failure when send exception response to client. I think the original exception make sense.

This error message should be added after link 235.

This is the error message when getIndex request failed, so we need to change the message here

ylwu-amzn · 2025-04-29T20:21:05Z

plugin/src/main/java/org/opensearch/ml/action/mcpserver/TransportMcpMessageAction.java

+
+        try (ThreadContext.StoredContext context = client.threadPool().getThreadContext().stashContext()) {
+            Map<String, DiscoveryNode> nodes = new HashMap<>(clusterService.state().nodes().getNodes());
+            nodes.remove(clusterService.localNode().getId());


Why remove local node ? I see this transport action will be used in

if (clusterService.localNode().getId().equals(nodeId)) { client .execute( MLMcpMessageAction.INSTANCE, new MLMcpMessageRequest(sessionId, requestBody), actionListener );

Can you add some comments ?

Curious about the same, I see that we are picking the first node all the time:
clusterService.state().nodes().getNodes().get(nodes.keySet().stream().findFirst().get()

These are test code, removing

ylwu-amzn · 2025-04-29T20:23:36Z

...in/src/main/java/org/opensearch/ml/action/mcpserver/TransportMcpMessageDispatchedAction.java

+    @Override
+    protected void doExecute(Task task, ActionRequest request, ActionListener<AcknowledgedResponse> listener) {
+        MLMcpMessageRequest mlMcpMessageRequest = MLMcpMessageRequest.fromActionRequest(request);
+        final StreamingRestChannel channel = McpAsyncServerHolder.CHANNELS.get(mlMcpMessageRequest.getSessionId());


Why use REST channel in transport action? Is this ok ?

rest channels here represents persistent connections, it's ok to use them anywhere, but we need somewhere to close them, will add this logic.

ylwu-amzn · 2025-04-29T20:28:45Z

plugin/src/main/java/org/opensearch/ml/action/mcpserver/McpAsyncServerHolder.java

        new ObjectMapper()
    );

+    public static Map<String, StreamingRestChannel> CHANNELS = new ConcurrentHashMap<>();


When the session will be removed ? I don't see remove logic

When/should we close the StreamingRestChannel ?

Based on the lifecycle of MCP protocol, we should close the channel when client disconnected, but I'm not sure if we have an approach to listen to this event. This doesn't have big impact for now so we can deprioritize this.

ylwu-amzn · 2025-04-29T20:30:26Z

Have you addressed all comments from last PR #3781 ?

#3781 (comment), I don't see setting to enable/disable this experimental feature

Suggest add your comment on last PR if the comment already addressed in this PR

ylwu-amzn · 2025-04-29T20:52:13Z

.../src/main/java/org/opensearch/ml/rest/mcpserver/RestMcpConnectionMessageStreamingAction.java

+                                        }
+                                    );
+                                    if (clusterService.localNode().getId().equals(nodeId)) {
+                                        client


Do we need this ? I see same code in else block

I didn't see the same in else.

If you check the first commit, you can see such same code

client .execute( MLMcpMessageAction.INSTANCE, new MLMcpMessageRequest(sessionId, requestBody), actionListener );

I see you removed such code in the second commit for (clusterService.localNode().getId().equals(nodeId)). I think such code not needed for local node case, right ?

At first I thought you mean the clusterService.localNode().getId().equals(nodeId) in else. I removed the part you mentioned, it's not needed.

pyek-bot · 2025-04-29T22:30:12Z

...n/src/main/java/org/opensearch/ml/action/mcpserver/OpenSearchMcpServerTransportProvider.java

-            String result = String.format("/sse/message?sessionId=%s", sessionId);
-            return Mono.just(createHttpChunk(ENDPOINT_EVENT_TYPE, result));
+            ActionListener<IndexResponse> actionListener = ActionListener.wrap(r -> {
+                if (r != null && r.status() == RestStatus.CREATED) {


what happens if the status is not CREATED here?

good catch, added else logic.

pyek-bot · 2025-04-29T22:31:18Z

common/src/main/resources/index-mappings/ml_mcp_session_management.json

+    "node_id": {
+      "type": "keyword"
+    },
+    "status": {


What are the expected values for status? I see active and created being used.

Values are: active and inactive. Now code has only active, created is the rest response status.

pyek-bot · 2025-04-29T22:42:02Z

plugin/src/main/java/org/opensearch/ml/action/mcpserver/TransportMcpMessageAction.java

+
+                        @Override
+                        public String executor() {
+                            return SAME;


curious, what does this achieve?

Which thread will be use to handle the transport response.

pyek-bot · 2025-04-29T22:42:50Z

...in/src/main/java/org/opensearch/ml/action/mcpserver/TransportMcpMessageDispatchedAction.java

+                    channel.sendResponse(new BytesRestResponse(channel, new Exception(e)));
+                    listener.onFailure(new Exception(e));
+                } catch (IOException ex) {
+                    log.error("Failed to send exception response to client during message handling due to IOException");


log the exception

it's logged.

sorry, i meant to log the actual exception log.error("...", ex)

anyways, it's a nit, can handle this later

O, I mean I'll add this log, and I've logged in the comment addressing commit.

pyek-bot · 2025-04-29T22:48:30Z

.../src/main/java/org/opensearch/ml/rest/mcpserver/RestMcpConnectionMessageStreamingAction.java

+                                String nodeId = String.valueOf(r.getSourceAsMap().get("node_id"));
+                                DiscoveryNode node = clusterService.state().getNodes().getNodes().get(nodeId);
+                                if (node == null) {
+                                    log.error("The node:{} is no longer in the current cluster, can not handle the mcp request", r.getId());


should this be nodeId and not r.getId()

pyek-bot · 2025-04-29T22:49:52Z

.../src/main/java/org/opensearch/ml/rest/mcpserver/RestMcpConnectionMessageStreamingAction.java

+                                            log.debug("MCP request has been dispatched to corresponding node and handled successfully!");
+                                            if (requestBody.contains("notifications/initialized")) {
+                                                log
+                                                    .debug(
+                                                        "Starting to send OK response for notifications/initialized request in coordinator node"
+                                                    );
+                                                channel.sendChunk(createInitializedNotificationRes());
+                                            }
+                                        }
+                                    },
+                                        e -> {
+                                            log
+                                                .error(
+                                                    "MCP request has been dispatched to corresponding node but peer node failed to handle it",
+                                                    e


let's log the nodeId here?

ylwu-amzn

Approved to unblock RC2. Please address all comments in new PR

ylwu-amzn · 2025-04-30T02:00:08Z

CI failed

Suite: Test class org.opensearch.ml.rest.mcpserver.RestMLRegisterMcpToolsActionTests
  2> REPRODUCE WITH: ./gradlew ':opensearch-ml-plugin:test' --tests "org.opensearch.ml.rest.mcpserver.RestMLRegisterMcpToolsActionTests.test_routes" -Dtests.seed=9611987A44BAA22D -Dtests.security.manager=false -Dtests.locale=fr-DZ -Dtests.timezone=SystemV/EST5EDT -Druntime.java=23
  2> java.lang.AssertionError: expected:<[POST /_plugins/_ml/mcp_tools/_register]> but was:<[POST /_plugins/_ml/mcp/tools/_register]>
        at __randomizedtesting.SeedInfo.seed([9611987A44BAA22D:DA045D84C754C8A3]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:120)
        at org.junit.Assert.assertEquals(Assert.java:146)
        at org.opensearch.ml.rest.mcpserver.RestMLRegisterMcpToolsActionTests.test_routes(RestMLRegisterMcpToolsActionTests.java:172)

Signed-off-by: zane-neo <zaniu@amazon.com>

zane-neo · 2025-04-30T02:59:47Z

Let's try to take care of these comment since we cannot modify them later (will keep adding more as I see them):

initial commit for MCP server in OpenSearch #3781 (comment)

https://github.com/opensearch-project/ml-commons/pull/3781/files#r2064552594

https://github.com/opensearch-project/ml-commons/pull/3781/files#r2064559936

https://github.com/opensearch-project/ml-commons/pull/3781/files#r2064666837

https://github.com/opensearch-project/ml-commons/pull/3781/files#r2064870075

Additionally, for the comments that we don't address, can you create issues to track them?

These comments are been address in this PR.

pyek-bot · 2025-04-30T03:16:37Z

...on/src/main/java/org/opensearch/ml/common/transport/mcpserver/requests/register/McpTool.java

@@ -92,26 +93,26 @@ public static McpTool parse(XContentParser parser) throws IOException {
                    break;


should these fields also be changed?

Which fields? Not following.

ylwu-amzn

approve to unblock 3.0 RC2

* support MCP session management Signed-off-by: zane-neo <zaniu@amazon.com> * Addressing comments Signed-off-by: zane-neo <zaniu@amazon.com> * add feature flag for mcp server and renaming mcp connector feature flag Signed-off-by: zane-neo <zaniu@amazon.com> * Address critical comments in #3781 Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> (cherry picked from commit 7c05295)

* support MCP session management * Addressing comments * add feature flag for mcp server and renaming mcp connector feature flag * Address critical comments in #3781 --------- (cherry picked from commit 7c05295) Signed-off-by: zane-neo <zaniu@amazon.com> Co-authored-by: zane-neo <zaniu@amazon.com>

* [BUG] Agent Framework: Handle model response when toolUse is not accompanied by text (#3755) * fix: handle model response when toolUse is not accompanied by text Signed-off-by: Pavan Yekbote <pybot@amazon.com> * feat: add test case for parseLLMOutput Signed-off-by: Pavan Yekbote <pybot@amazon.com> --------- Signed-off-by: Pavan Yekbote <pybot@amazon.com> * [BUG] Allow user to control react agent max_interations value to prevent empty response (#3756) * fix: expose max_iteration for react Signed-off-by: Pavan Yekbote <pybot@amazon.com> * fix: defaults for agent execution and differentiate between step and step result Signed-off-by: Pavan Yekbote <pybot@amazon.com> * fix: return react agent id in agent response to expose more details Signed-off-by: Pavan Yekbote <pybot@amazon.com> * spotless Signed-off-by: Pavan Yekbote <pybot@amazon.com> * fix: remove test prompt from react system prompt Signed-off-by: Pavan Yekbote <pybot@amazon.com> * refactor: rename parameters exposed to user to executor Signed-off-by: Pavan Yekbote <pybot@amazon.com> * fix: give user complete control over planner system prompt Signed-off-by: Pavan Yekbote <pybot@amazon.com> --------- Signed-off-by: Pavan Yekbote <pybot@amazon.com> * Clean up JSM from MCP (#3773) Signed-off-by: rithin-pullela-aws <rithinp@amazon.com> * [Bug] ListTools call does not return tool attributes (#3785) * initial commit for MCP server in OpenSearch (#3781) * initial commit for MCP server in OpenSearch Signed-off-by: zane-neo <zaniu@amazon.com> * Make change to support register or remove tools across cluster Signed-off-by: zane-neo <zaniu@amazon.com> * format code Signed-off-by: zane-neo <zaniu@amazon.com> * fix UT failure caused by code change Signed-off-by: zane-neo <zaniu@amazon.com> * format code Signed-off-by: zane-neo <zaniu@amazon.com> * format code Signed-off-by: zane-neo <zaniu@amazon.com> * add license header Signed-off-by: zane-neo <zaniu@amazon.com> * fix notifications initialized not respond issue Signed-off-by: zane-neo <zaniu@amazon.com> * fix minor issues and add UTs Signed-off-by: zane-neo <zaniu@amazon.com> * Add more UTs Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> * Remove beta1 qualifier (#3794) (#3795) (cherry picked from commit 3f503f1) Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> Co-authored-by: Peter Zhu <zhujiaxi@amazon.com> * [AUTO] Increment version to 3.1.0-SNAPSHOT (#3789) * Increment version to 3.1.0-SNAPSHOT Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> * Update build.gradle Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> --------- Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: Peter Zhu <zhujiaxi@amazon.com> * add release note for 3.0 (#3792) Signed-off-by: Mingshi Liu <mingshl@amazon.com> * support MCP session management (#3803) * support MCP session management Signed-off-by: zane-neo <zaniu@amazon.com> * Addressing comments Signed-off-by: zane-neo <zaniu@amazon.com> * add feature flag for mcp server and renaming mcp connector feature flag Signed-off-by: zane-neo <zaniu@amazon.com> * Address critical comments in #3781 Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> * upgrade http client to version align with core (#3809) * upgrade http client to versoin align with core Signed-off-by: zane-neo <zaniu@amazon.com> * upgrade httpclient-h2 to correct versiono Signed-off-by: zane-neo <zaniu@amazon.com> * use placeholder approach Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> * support customized message endpoint and addressing comments (#3810) * support customized message endpoint and addressing comments Signed-off-by: zane-neo <zaniu@amazon.com> * fix UT failures Signed-off-by: zane-neo <zaniu@amazon.com> * add files to jacoco exception Signed-off-by: zane-neo <zaniu@amazon.com> * fix tool name issue and optimize register tool api Signed-off-by: zane-neo <zaniu@amazon.com> * fix schema not parsed correctly issue and NPE when parameters is null Signed-off-by: zane-neo <zaniu@amazon.com> * fix failure UT Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> * excluding circuit breaker for Agent (#3814) Signed-off-by: Dhrubo Saha <dhrubo@amazon.com> * change release note (#3811) * change release note Signed-off-by: zane-neo <zaniu@amazon.com> * Update opensearch-ml-common.release-notes-3.0.0.0.md * Update opensearch-ml-common.release-notes-3.0.0.0.md * Update opensearch-ml-common.release-notes-3.0.0.0.md --------- Signed-off-by: zane-neo <zaniu@amazon.com> Co-authored-by: Peter Zhu <zhujiaxi@amazon.com> * Downgrade MCP version to 0.9 (#3821) Signed-off-by: rithin-pullela-aws <rithinp@amazon.com> * remove libs folder (#3824) Signed-off-by: Yaliang Wu <ylwu@amazon.com> * add more logging to deploy/undeploy flows for better debugging (#3825) * add more logging to deploy/undeploy flows for better debugging Signed-off-by: Bhavana Goud Ramaram <rbhavna@amazon.com> * Fix python client not able to connect to MCP server issue (#3822) Signed-off-by: zane-neo <zaniu@amazon.com> Co-authored-by: Dhrubo Saha <dhrubo@amazon.com> * exclude trusted connector check for hidden model (#3838) Signed-off-by: Dhrubo Saha <dhrubo@amazon.com> * adding tenantId to the connector executor when this is inline connector (#3837) * adding tenantId to the connector executor when this is inline connector Signed-off-by: Dhrubo Saha <dhrubo@amazon.com> * added more unit tests Signed-off-by: Dhrubo Saha <dhrubo@amazon.com> --------- Signed-off-by: Dhrubo Saha <dhrubo@amazon.com> --------- Signed-off-by: Pavan Yekbote <pybot@amazon.com> Signed-off-by: rithin-pullela-aws <rithinp@amazon.com> Signed-off-by: zane-neo <zaniu@amazon.com> Signed-off-by: Peter Zhu <zhujiaxi@amazon.com> Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com> Signed-off-by: Mingshi Liu <mingshl@amazon.com> Signed-off-by: Dhrubo Saha <dhrubo@amazon.com> Signed-off-by: Yaliang Wu <ylwu@amazon.com> Signed-off-by: Bhavana Goud Ramaram <rbhavna@amazon.com> Co-authored-by: Pavan Yekbote <pybot@amazon.com> Co-authored-by: Rithin Pullela <rithinp@amazon.com> Co-authored-by: zane-neo <zaniu@amazon.com> Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com> Co-authored-by: Peter Zhu <zhujiaxi@amazon.com> Co-authored-by: opensearch-ci-bot <opensearch-infra@amazon.com> Co-authored-by: Mingshi Liu <mingshl@amazon.com> Co-authored-by: Yaliang Wu <ylwu@amazon.com> Co-authored-by: Bhavana Goud Ramaram <rbhavna@amazon.com>

* support MCP session management Signed-off-by: zane-neo <zaniu@amazon.com> * Addressing comments Signed-off-by: zane-neo <zaniu@amazon.com> * add feature flag for mcp server and renaming mcp connector feature flag Signed-off-by: zane-neo <zaniu@amazon.com> * Address critical comments in opensearch-project#3781 Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com>

* initial commit for MCP server in OpenSearch (#3781) * initial commit for MCP server in OpenSearch Signed-off-by: zane-neo <zaniu@amazon.com> * Make change to support register or remove tools across cluster Signed-off-by: zane-neo <zaniu@amazon.com> * format code Signed-off-by: zane-neo <zaniu@amazon.com> * fix UT failure caused by code change Signed-off-by: zane-neo <zaniu@amazon.com> * format code Signed-off-by: zane-neo <zaniu@amazon.com> * format code Signed-off-by: zane-neo <zaniu@amazon.com> * add license header Signed-off-by: zane-neo <zaniu@amazon.com> * fix notifications initialized not respond issue Signed-off-by: zane-neo <zaniu@amazon.com> * fix minor issues and add UTs Signed-off-by: zane-neo <zaniu@amazon.com> * Add more UTs Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> * support MCP session management (#3803) * support MCP session management Signed-off-by: zane-neo <zaniu@amazon.com> * Addressing comments Signed-off-by: zane-neo <zaniu@amazon.com> * add feature flag for mcp server and renaming mcp connector feature flag Signed-off-by: zane-neo <zaniu@amazon.com> * Address critical comments in #3781 Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> * Create prompt API & System Index for prompt Signed-off-by: seungwon cho <rycho@amazon.com> * spotless is applied & disabled wildcard import Signed-off-by: seungwon cho <rycho@amazon.com> * javadoc added & header added & minor code errors fixed Signed-off-by: seungwon cho <rycho@amazon.com> * Apply spotless Signed-off-by: seungwon cho <rycho@amazon.com> * apply spotless Signed-off-by: seungwon cho <rycho@amazon.com> * addressed comments Signed-off-by: seungwon cho <rycho@amazon.com> * apply spotless Signed-off-by: seungwon cho <rycho@amazon.com> * addresses comments Signed-off-by: seungwon cho <rycho@amazon.com> * apply spotless Signed-off-by: seungwon cho <rycho@amazon.com> * addressed comments Signed-off-by: seungwon cho <rycho@amazon.com> * solve gradle build issue Signed-off-by: seungwon cho <rycho@amazon.com> * add test cases for create-api Signed-off-by: seungwon cho <rycho@amazon.com> * fix javadoc test case failure Signed-off-by: seungwon cho <rycho@amazon.com> * fix javadoc error Signed-off-by: seungwon cho <rycho@amazon.com> * fix guava noclass issue Signed-off-by: seungwon cho <rycho@amazon.com> * fix jacocoTestCoverageVerification fail Signed-off-by: seungwon cho <rycho@amazon.com> * improve test coverages Signed-off-by: seungwon cho <rycho@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> Signed-off-by: seungwon cho <rycho@amazon.com> Co-authored-by: zane-neo <zaniu@amazon.com>

* support MCP session management Signed-off-by: zane-neo <zaniu@amazon.com> * Addressing comments Signed-off-by: zane-neo <zaniu@amazon.com> * add feature flag for mcp server and renaming mcp connector feature flag Signed-off-by: zane-neo <zaniu@amazon.com> * Address critical comments in opensearch-project#3781 Signed-off-by: zane-neo <zaniu@amazon.com> --------- Signed-off-by: zane-neo <zaniu@amazon.com> Signed-off-by: Abdul Muneer Kolarkunnu <muneer.kolarkunnu@netapp.com>

support MCP session management

a6529fa

Signed-off-by: zane-neo <zaniu@amazon.com>

zane-neo requested review from HenryL27, Zhangxunmt, austintlee, b4sjoo, dhrubo-os, jngz-es, mingshl, model-collapse, rbhavna, xinyual and ylwu-amzn as code owners April 29, 2025 13:59

zane-neo had a problem deploying to ml-commons-cicd-env April 29, 2025 14:00 — with GitHub Actions Failure

zane-neo had a problem deploying to ml-commons-cicd-env April 29, 2025 14:00 — with GitHub Actions Error

zane-neo had a problem deploying to ml-commons-cicd-env April 29, 2025 14:00 — with GitHub Actions Failure

ylwu-amzn reviewed Apr 29, 2025

View reviewed changes

pyek-bot reviewed Apr 29, 2025

View reviewed changes

ylwu-amzn previously approved these changes Apr 30, 2025

View reviewed changes

add feature flag for mcp server and renaming mcp connector feature flag

c0cf540

Signed-off-by: zane-neo <zaniu@amazon.com>

zane-neo dismissed ylwu-amzn’s stale review via c0cf540 April 30, 2025 02:06

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:07 — with GitHub Actions Failure

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:07 — with GitHub Actions Error

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:07 — with GitHub Actions Failure

ylwu-amzn previously approved these changes Apr 30, 2025

View reviewed changes

Address critical comments in opensearch-project#3781

103de42

Signed-off-by: zane-neo <zaniu@amazon.com>

zane-neo dismissed ylwu-amzn’s stale review via 103de42 April 30, 2025 02:51

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:52 — with GitHub Actions Error

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:53 — with GitHub Actions Failure

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:53 — with GitHub Actions Error

zane-neo had a problem deploying to ml-commons-cicd-env April 30, 2025 02:53 — with GitHub Actions Failure

peterzhuamazon mentioned this pull request Apr 30, 2025

[RELEASE] Release version 3.0.0 opensearch-project/opensearch-build#3747

Closed

66 tasks

pyek-bot reviewed Apr 30, 2025

View reviewed changes

ylwu-amzn approved these changes Apr 30, 2025

View reviewed changes

xinyual approved these changes Apr 30, 2025

View reviewed changes

zane-neo merged commit 7c05295 into opensearch-project:main Apr 30, 2025
6 of 10 checks passed

zane-neo added the backport 3.0 label Apr 30, 2025

opensearch-trigger-bot bot mentioned this pull request Apr 30, 2025

[Backport 3.0] support MCP session management #3808

Merged

zane-neo mentioned this pull request Apr 30, 2025

initial commit for MCP server in OpenSearch #3781

Merged

5 tasks

		@@ -92,26 +93,26 @@ public static McpTool parse(XContentParser parser) throws IOException {
		break;

Conversation

zane-neo commented Apr 29, 2025

Description

Related Issues

Check List

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ylwu-amzn commented Apr 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ylwu-amzn Apr 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ylwu-amzn left a comment

Choose a reason for hiding this comment

Uh oh!

ylwu-amzn commented Apr 30, 2025

Uh oh!

ylwu-amzn commented Apr 29, 2025 •

edited

Loading

ylwu-amzn Apr 30, 2025 •

edited

Loading