Merge Kubernetes get and list tools#591
Merged
alzimmermsft merged 11 commits intomicrosoft:mainfrom Oct 7, 2025
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR merges Azure Kubernetes Service (AKS) get and list tools to consolidate functionality and reduce the number of commands. Instead of having separate get and list commands, the get commands now perform both operations based on optional parameters.
Key changes:
- Merged
azmcp_aks_cluster_listintoazmcp_aks_cluster_get- now returns all clusters when no cluster name is provided - Merged
azmcp_aks_nodepool_listintoazmcp_aks_nodepool_get- now returns all node pools when no nodepool name is provided
Reviewed Changes
Copilot reviewed 25 out of 26 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/Azure.Mcp.Tools.Quota/src/Services/Util/Usage/NetworkUsageChecker.cs | Removes unnecessary parentheses around cast operations |
| tools/Azure.Mcp.Tools.Quota/src/Services/Util/AzureUsageChecker.cs | Modernizes collection initialization syntax |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.UnitTests/Nodepool/NodepoolListCommandTests.cs | Removes obsolete test file for list command |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.UnitTests/Nodepool/NodepoolGetCommandTests.cs | Updates tests to use merged get functionality |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.UnitTests/Cluster/ClusterListCommandTests.cs | Removes obsolete test file for list command |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.UnitTests/Cluster/ClusterGetCommandTests.cs | Updates tests to use merged get functionality |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.LiveTests/NodepoolGetCommandTests.cs | Updates live tests for merged functionality |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.LiveTests/NodepoolCommandTests.cs | Updates live tests to use get command |
| tools/Azure.Mcp.Tools.Aks/tests/Azure.Mcp.Tools.Aks.LiveTests/AksCommandTests.cs | Updates live tests to use get command |
| tools/Azure.Mcp.Tools.Aks/src/Services/IAksService.cs | Merges service interfaces for get and list operations |
| tools/Azure.Mcp.Tools.Aks/src/Services/AksService.cs | Implements merged functionality in service layer |
| tools/Azure.Mcp.Tools.Aks/src/Options/Nodepool/NodepoolListOptions.cs | Removes obsolete options class |
| tools/Azure.Mcp.Tools.Aks/src/Options/Cluster/ClusterListOptions.cs | Removes obsolete options class |
| tools/Azure.Mcp.Tools.Aks/src/Options/AksOptionDefinitions.cs | Makes options non-required to support merged functionality |
| tools/Azure.Mcp.Tools.Aks/src/Commands/Nodepool/NodepoolListCommand.cs | Removes obsolete command file |
| tools/Azure.Mcp.Tools.Aks/src/Commands/Nodepool/NodepoolGetCommand.cs | Updates to handle both get and list operations |
| tools/Azure.Mcp.Tools.Aks/src/Commands/Cluster/ClusterListCommand.cs | Removes obsolete command file |
| tools/Azure.Mcp.Tools.Aks/src/Commands/Cluster/ClusterGetCommand.cs | Updates to handle both get and list operations |
| tools/Azure.Mcp.Tools.Aks/src/Commands/AksJsonContext.cs | Removes obsolete serialization contexts |
| tools/Azure.Mcp.Tools.Aks/src/AksSetup.cs | Updates service registration to remove list commands |
| servers/Azure.Mcp.Server/CHANGELOG.md | Documents breaking changes |
| eng/tools/ToolDescriptionEvaluator/tools.json | Updates tool descriptions and removes list commands |
| eng/tools/ToolDescriptionEvaluator/prompts.json | Updates test prompts for merged commands |
| docs/e2eTestPrompts.md | Updates end-to-end test prompts |
| docs/azmcp-commands.md | Updates documentation for merged commands |
jongio
approved these changes
Oct 7, 2025
colbytimm
pushed a commit
to colbytimm/microsoft-mcp
that referenced
this pull request
Dec 8, 2025
* Initial update to merge tools * Update CHANGELOG * Descriptio tweaks * Fix linting and tests * Checkpoint changes * Fix resource group requiredness when listing * Finalize test fixes
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What does this PR do?
Merges Azure Kubernetes Service get and list tools.
GitHub issue number?
Pre-merge Checklist
servers/Azure.Mcp.Server/CHANGELOG.mdand/orservers/Fabric.Mcp.Server/CHANGELOG.mdfor product changes (features, bug fixes, UI/UX, updated dependencies)servers/Azure.Mcp.Server/README.mdand/orservers/Fabric.Mcp.Server/README.mddocumentation/docs/azmcp-commands.mdand/or/docs/fabric-commands.mdToolDescriptionEvaluatorand obtained a score of0.4or more and a top 3 ranking for all related test prompts/docs/e2eTestPrompts.mdcrypto mining, spam, data exfiltration, etc.)/azp run mcp - pullrequest - liveto run Live Test Pipeline