You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add a new tool for querying OpenSearch. The tool is disabled by default and needs to be enabled using a flag.
I am using the /api/ds/query endpoint instead of /api/datasources/proxy/uid/{uid}/_msearch to support SigV4 signature for AWS.
Note
Medium Risk
Adds a new datasource-querying tool and wires it into the CLI/tool enablement path, plus expands the integration/e2e environment with an OpenSearch container and seeded data. Moderate risk due to new HTTP query path via Grafana /api/ds/query and additional CI infrastructure that could affect test stability.
Overview
Adds OpenSearch support via a new query_opensearch tool that queries OpenSearch through Grafana’s /api/ds/query (plugin backend) with time-range and limit handling, and registers it under a new opensearch tool category.
Updates server/CLI wiring and docs to expose --disable-opensearch and include OpenSearch in enabled-tools lists and capability descriptions.
Extends test infrastructure to provision an OpenSearch datasource, run an OpenSearch container with a seed script, and adds Go integration tests plus Python e2e tests that validate query_opensearch usage; CI workflows are updated accordingly.
Written by Cursor Bugbot for commit 2375189. This will update automatically on new commits. Configure here.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You have signed the CLA already but the status is still pending? Let us recheck it.
Hello @LucasHild! Thanks for putting this together! We’re trying to avoid growing the MCP tool list when the use case is this close to what we already have. OpenSearch and Elasticsearch are similar under the hood (Lucene, comparable query styles), so query_opensearch and query_elasticsearch end up overlapping a lot, and we’d be maintaining two parallel paths long term.
Maybe we could consider extending query_elasticsearch with an opensearch path. Would you be open to try and extend query_elasticsearch tool instead? Shared bits (auth, limits, time range, document shape) can stay in one place and the request/response handling is the one that needs to diverge.
Thanks again! Happy to discuss more or have another look if you think this is something that you'd like to explore.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a new tool for querying OpenSearch. The tool is disabled by default and needs to be enabled using a flag.
I am using the
/api/ds/queryendpoint instead of/api/datasources/proxy/uid/{uid}/_msearchto support SigV4 signature for AWS.Note
Medium Risk
Adds a new datasource-querying tool and wires it into the CLI/tool enablement path, plus expands the integration/e2e environment with an OpenSearch container and seeded data. Moderate risk due to new HTTP query path via Grafana
/api/ds/queryand additional CI infrastructure that could affect test stability.Overview
Adds OpenSearch support via a new
query_opensearchtool that queries OpenSearch through Grafana’s/api/ds/query(plugin backend) with time-range and limit handling, and registers it under a newopensearchtool category.Updates server/CLI wiring and docs to expose
--disable-opensearchand include OpenSearch in enabled-tools lists and capability descriptions.Extends test infrastructure to provision an OpenSearch datasource, run an OpenSearch container with a seed script, and adds Go integration tests plus Python e2e tests that validate
query_opensearchusage; CI workflows are updated accordingly.Written by Cursor Bugbot for commit 2375189. This will update automatically on new commits. Configure here.