Skip to content

Conversation

@getsolaris
Copy link
Contributor

@getsolaris getsolaris commented Apr 4, 2025

Add get_shards tool for monitoring shard status

Screenshot 2025-04-05 at 18 00 33

Changes

  • Add get_shards tool to check shard distribution and status
  • Support optional index parameter for filtering
  • Return detailed shard metadata (state, docs, size, node info)

Using

Screenshot 2025-04-05 at 18 02 48 Screenshot 2025-04-05 at 18 01 59 Screenshot 2025-04-05 at 18 01 57

@getsolaris getsolaris requested a review from a team as a code owner April 4, 2025 07:20
@jedrazb jedrazb self-requested a review April 4, 2025 07:41
@jedrazb
Copy link
Member

jedrazb commented Apr 4, 2025

Hey @getsolaris, thank you for your contribution! This LGTM and I'm happy to merge. Could you please drop a screenshot from the MCP client showing that this tool works as intended?

@jedrazb
Copy link
Member

jedrazb commented Apr 4, 2025

buildkite test this

@getsolaris
Copy link
Contributor Author

@jedrazb
Thank you for the quick review!

I've attached three screenshots demonstrating the get_shards tool functionality:

  1. Claude Desktop MCP Tool interface showing the tool integration
  2. Complete cluster shard status overview using get_shards()
  3. Specific index shard distribution using get_shards({ index: "my-index" })

The screenshots verify that the tool:

  • Successfully integrates with the MCP client
  • Properly displays cluster-wide shard distribution
  • Correctly filters and shows shard information for specific indices
  • Maintains consistent response formatting for both use cases

Let me know if you need any additional information or clarification!

@getsolaris getsolaris changed the title feat: Add get_shards tool for monitoring shard status Add get_shards tool for monitoring shard status Apr 5, 2025
Copy link
Member

@jedrazb jedrazb left a comment

Choose a reason for hiding this comment

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

lgtm, thank you for contributing! :)

@jedrazb
Copy link
Member

jedrazb commented Apr 7, 2025

I'm resolving the issue with CI setup, I will merge this once it passes

@getsolaris
Copy link
Contributor Author

@jedrazb thank you :)

@jedrazb
Copy link
Member

jedrazb commented Apr 7, 2025

buildkite test this

@jedrazb
Copy link
Member

jedrazb commented Apr 7, 2025

buildkite test this

@jedrazb jedrazb merged commit 2abcaa6 into elastic:main Apr 7, 2025
2 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.

2 participants