Skip to content

feat: adding fcp service and fc interface tools#94

Open
Hardikl wants to merge 7 commits intomainfrom
hl_fcp
Open

feat: adding fcp service and fc interface tools#94
Hardikl wants to merge 7 commits intomainfrom
hl_fcp

Conversation

@Hardikl
Copy link
Copy Markdown
Contributor

@Hardikl Hardikl commented Apr 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 14:09
@cla-bot cla-bot bot added the cla-signed label Apr 3, 2026
@Hardikl Hardikl linked an issue Apr 3, 2026 that may be closed by this pull request
Copy link
Copy Markdown

Copilot AI left a comment

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 introduces MCP tools and REST client support for managing ONTAP SAN objects: FCP services and FC interfaces, including integration tests and documentation examples.

Changes:

  • Added new tool parameter schemas for FCP service and FC interface operations.
  • Implemented server handlers and REST client methods for create/update/delete of FCP services and FC interfaces.
  • Added integration coverage and updated docs/examples and tool descriptions for the new capabilities.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tool/tool.go Adds input parameter structs for FCP service and FC interface tools.
server/server.go Registers new MCP tools for FCP service and FC interface operations.
server/fcp.go Implements server-side tool handlers and request builders for FCP/FC operations.
rest/fcp.go Adds REST client methods to call ONTAP endpoints for FCP services and FC interfaces.
ontap/ontap.go Introduces ONTAP model structs for FCP service and FC interface payloads.
integration/test/fcp_test.go Adds integration tests covering the new prompts/tools.
docs/examples.md Documents example prompts for managing FCP and FC interfaces.
descriptions/descriptions.go Adds tool description strings for the new tools.

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

Copilot AI review requested due to automatic review settings April 6, 2026 13:23
Copy link
Copy Markdown

Copilot AI left a comment

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 8 out of 8 changed files in this pull request and generated 2 comments.


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

Copilot AI review requested due to automatic review settings April 8, 2026 15:47
Copy link
Copy Markdown

Copilot AI left a comment

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 8 out of 8 changed files in this pull request and generated 2 comments.


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

# Conflicts:
#	descriptions/descriptions.go
#	docs/examples.md
#	ontap/ontap.go
#	server/server.go
#	tool/tool.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.

Add FCP service/interface management tools

3 participants