Skip to content

[Azure MCP] New command: Eventhubs namespace get#105

Merged
jairmyree merged 36 commits intomicrosoft:mainfrom
jairmyree:eventhubs_namespaces-list
Sep 29, 2025
Merged

[Azure MCP] New command: Eventhubs namespace get#105
jairmyree merged 36 commits intomicrosoft:mainfrom
jairmyree:eventhubs_namespaces-list

Conversation

@jairmyree
Copy link
Copy Markdown
Member

@jairmyree jairmyree commented Sep 2, 2025

This PR onboards EventHubs with its first MCP tool: eventhubs namespaces get

Closes #60

Intended Agent Scenarios
This tool returns a list containing the name, id, and resource group of each namespace. This tool is backed by the management library Azure.ResourceManager.EventHubs. This tool has the following required parameters: reource-group. Suggested prompt: "List all EventHubs Namespaces available in resource-group"

Tool Name: azmcp-eventhubs-namespace-get

Tool Description: Lists all available namespaces. This tool returns a list containing the name, id, and resource group of each namespace. This tool is backed by the management library Azure.ResourceManager.EventHubs. This tool has the following required parameters: reource-group. Suggested prompt: "List all EventHubs Namespaces available in resource-group"

Tool Parameters:

  • resourceGroupName

Tool Return: A JSON array of all available namespaces (their names, id, and resource groups)

Backing Library: Management

Copy link
Copy Markdown
Contributor

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 the EventHubs toolset with its first MCP tool for listing EventHubs namespaces within a resource group. The implementation follows the established toolset pattern and provides both unit and integration test coverage.

  • Adds EventHubs namespace listing functionality with comprehensive error handling
  • Implements full test infrastructure including Bicep templates for live testing
  • Updates documentation to reflect the transition from "area" to "toolset" terminology

Reviewed Changes

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

Show a summary per file
File Description
tools/Azure.Mcp.Tools.EventHubs/src/ Core EventHubs toolset implementation with command, service, and model classes
tools/Azure.Mcp.Tools.EventHubs/tests/ Unit and integration tests with Bicep test infrastructure
servers/Azure.Mcp.Server/src/Program.cs Registers EventHubs toolset in the server
docs/new-command.md Updates documentation from "area" to "toolset" terminology
Directory.Packages.props Adds Azure.ResourceManager.EventHubs package reference
AzureMcp.sln Adds EventHubs projects to solution

Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceListCommand.cs Outdated
@jairmyree jairmyree force-pushed the eventhubs_namespaces-list branch from c009fac to 4afbc57 Compare September 2, 2025 18:28
feiskyer pushed a commit to feiskyer/microsoft-mcp that referenced this pull request Sep 8, 2025
* Add AADSTS500200 error in troubleshooting

* Add helpful links
Comment thread docs/new-command.md
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceListCommand.cs Outdated
@jairmyree jairmyree force-pushed the eventhubs_namespaces-list branch from b3fba49 to 2afd247 Compare September 15, 2025 19:06
@jairmyree jairmyree requested a review from a team as a code owner September 15, 2025 19:06
@jairmyree jairmyree force-pushed the eventhubs_namespaces-list branch 2 times, most recently from b7bf4be to aacb0b7 Compare September 17, 2025 15:15
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceListCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceListCommand.cs Outdated
@jairmyree jairmyree force-pushed the eventhubs_namespaces-list branch 2 times, most recently from 8c679fc to d6660f2 Compare September 18, 2025 16:03
@jairmyree jairmyree moved this from Untriaged to In Progress in Azure MCP Server Sep 18, 2025
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceGetCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceGetCommand.cs Outdated
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Options/EventHubsOptionDefinitions.cs Outdated
Comment thread tools/Azure.Mcp.Tools.EventHubs/src/Options/EventHubsOptionDefinitions.cs Outdated
@jairmyree jairmyree requested a review from jongio September 18, 2025 20:17
@jairmyree jairmyree force-pushed the eventhubs_namespaces-list branch from bebcd01 to 9bf6cff Compare September 22, 2025 17:26
jairmyree and others added 23 commits September 29, 2025 14:49
Co-authored-by: Connie Yau <conniey@microsoft.com>
…ceGetCommand.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
…ceGetCommand.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
…ceGetCommand.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
…s.LiveTests/EventHubsCommandTests.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
…s.UnitTests/Namespace/NamespaceGetCommandTests.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
…ns.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
…s.LiveTests/EventHubsCommandTests.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jairmyree jairmyree force-pushed the eventhubs_namespaces-list branch from 9e899bc to 3e12ced Compare September 29, 2025 21:50
Copy link
Copy Markdown
Member

@conniey conniey left a comment

Choose a reason for hiding this comment

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

Lgtm

Copy link
Copy Markdown
Member

@conniey conniey left a comment

Choose a reason for hiding this comment

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

Lgtm

@jairmyree jairmyree dismissed jongio’s stale review September 29, 2025 23:07

Approved by Connie

@jairmyree jairmyree merged commit 2eb90fc into microsoft:main Sep 29, 2025
27 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Sep 29, 2025
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Dec 8, 2025
* Rebasing EventHubs Namespace List Command

* Live tests passing

* Updating test-resources-post and adding e2e prompt

* Updating  Namespace List to Namespace Get

* updating test-resources-post

* Updating test to reflect tool consolidation

* Updates to PR based on feedback

* EventHubs namespace get command update. Now supports getting metadata on specific resource

* Update namespace get command to not take id parameter

* Correcting EventHubs to Event Hubs in documentation

* Additional Updates

* Remove unused imports

* Changes to EventHubsService

* dotnet format

* Remove the dependency on Azure.ResourceManager.EventHubs due to use ARG instead

* Update tools/Azure.Mcp.Tools.EventHubs/src/EventHubsSetup.cs

Co-authored-by: Connie Yau <conniey@microsoft.com>

* Updates from PR comments

* Updates from PR comments

* Updates from PR comments

* Updates from PR comments

* Fixing test error

* Fixing live test error

* Update tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceGetCommand.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* Update tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceGetCommand.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* Update tools/Azure.Mcp.Tools.EventHubs/src/Commands/Namespace/NamespaceGetCommand.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* Update tools/Azure.Mcp.Tools.EventHubs/src/Services/EventHubsService.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* Update tools/Azure.Mcp.Tools.EventHubs/tests/Azure.Mcp.Tools.EventHubs.LiveTests/EventHubsCommandTests.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* Update tools/Azure.Mcp.Tools.EventHubs/tests/Azure.Mcp.Tools.EventHubs.UnitTests/Namespace/NamespaceGetCommandTests.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* Updates to PR based on feedback fron Alan and Connie

* Update tools/Azure.Mcp.Tools.EventHubs/src/Options/BaseEventHubsOptions.cs

Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>

* additional PR updates

* Updates to fix test failures

* Update tools/Azure.Mcp.Tools.EventHubs/tests/Azure.Mcp.Tools.EventHubs.LiveTests/EventHubsCommandTests.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update tools/Azure.Mcp.Tools.EventHubs/src/Services/EventHubsService.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Rmoving empty file

* Update validation pattern

---------

Co-authored-by: Arthur Ma <arthurma@microsoft.com>
Co-authored-by: Connie Yau <conniey@microsoft.com>
Co-authored-by: Alan Zimmer <48699787+alzimmermsft@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Azure MCP] Add command: azmcp eventhubs namespace get

6 participants