Skip to content

feat: enhance InfrastructureHealth tool#4315

Open
rahulguptajss wants to merge 4 commits into
mainfrom
rg2-mcp-health
Open

feat: enhance InfrastructureHealth tool#4315
rahulguptajss wants to merge 4 commits into
mainfrom
rg2-mcp-health

Conversation

@rahulguptajss

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 10, 2026 11:14
@cla-bot cla-bot Bot added the cla-signed label Jun 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the infrastructure_health MCP tool to provide broader ONTAP health coverage and improved query filtering, so users can get a more complete operational snapshot (optionally scoped to a cluster/cluster regex).

Changes:

  • Expanded InfrastructureHealth health check set to include hardware/environmental, aggregate space, SnapMirror lag, network errors, licensing, SAN, and subsystem/SVM checks.
  • Improved cluster label injection to support PromQL selectors that already include labels (e.g., metric{...}).
  • Extended “details” output by extracting additional identifiers (svm/disk/shelf/lif/lun/etc.) and updated the tool description to match the new coverage.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
mcp/cmd/server/main.go Adds additional health checks/metric context, improves cluster filter injection for metric{...} queries, and expands identifier extraction for details output.
mcp/cmd/descriptions/descriptions.go Updates the infrastructure_health tool description to reflect the expanded checks and metric semantics.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread mcp/cmd/server/main.go Outdated
Comment thread mcp/cmd/descriptions/descriptions.go Outdated
Comment thread mcp/cmd/server/main.go
Copilot AI review requested due to automatic review settings June 10, 2026 13:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comment thread mcp/cmd/server/main_test.go
Comment thread mcp/cmd/descriptions/descriptions.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants