Skip to content

Consolidate LoadTesting tools#1711

Merged
g2vinay merged 10 commits intomicrosoft:mainfrom
g2vinay:consolidate-loadtesting-tools
Feb 12, 2026
Merged

Consolidate LoadTesting tools#1711
g2vinay merged 10 commits intomicrosoft:mainfrom
g2vinay:consolidate-loadtesting-tools

Conversation

@g2vinay
Copy link
Copy Markdown
Contributor

@g2vinay g2vinay commented Feb 12, 2026

Consolidates Get/List and Create/Update tools.

@g2vinay g2vinay changed the title Consolidate LoadTesting commands from 4 to 2 Consolidate LoadTesting tools Feb 12, 2026
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 consolidates Load Testing tools by merging similar commands to reduce redundancy and simplify the API surface. Specifically, it combines testrun get + testrun list into a single testrun get command, and testrun create + testrun update into a single testrun createorupdate command. The consolidation maintains all existing functionality while making the command structure more intuitive.

Changes:

  • Merged testrun get and testrun list into a single get command that accepts either --testrun (for single run) or --test (for listing all runs)
  • Merged testrun create and testrun update into a single createorupdate command
  • Updated service registrations, serialization contexts, and test files to reflect the consolidation
  • Added comprehensive test guide documenting the consolidated commands

Reviewed changes

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

Show a summary per file
File Description
TestRunGetCommand.cs Consolidated command supporting both get-by-id and list-by-test-id operations
TestRunCreateOrUpdateCommand.cs Consolidated command handling both create and update operations
TestRunGetCommandTests.cs Updated tests to cover both single-run and list scenarios
TestRunCreateOrUpdateCommandTests.cs Renamed and updated tests for consolidated create/update command
LoadTestingSetup.cs Updated service registration to use consolidated commands
LoadTestingJsonContext.cs Updated serialization types to match new command result structures
LoadTestRunGetOptions.cs Added TestId property to support list functionality
LoadTestRunCreateOrUpdateOptions.cs Renamed from TestRunCreateOptions
CONSOLIDATION_TEST_GUIDE.md New test guide documenting consolidated command usage
azmcp-commands.md Removed metadata comments from old commands (needs full update)
prompts.json Updated prompts to use new consolidated command names
consolidated-tools.json Updated tool mappings to reflect consolidated commands
Comments suppressed due to low confidence (1)

servers/Azure.Mcp.Server/docs/azmcp-commands.md:1473

  • The documentation still lists the old separate commands (testrun create, testrun list, testrun update) that are being removed in this PR. These should be updated to reflect the consolidated commands: "testrun get" (which now handles both get and list) and "testrun createorupdate" (which now handles both create and update).
# Create load test run
azmcp loadtesting testrun create --subscription <subscription> \
                                 --resource-group <resource-group> \
                                 --test-resource-name <test-resource-name> \
                                 --test-id <test-id> \
                                 --testrun-id <testrun-id> \
                                 --display-name <display-name> \
                                 --description <description> \
                                 --old-testrun-id <old-testrun-id>

# Get load test run
# ❌ Destructive | ✅ Idempotent | ❌ OpenWorld | ✅ ReadOnly | ❌ Secret | ❌ LocalRequired
azmcp loadtesting testrun get --subscription <subscription> \
                              --resource-group <resource-group> \
                              --test-resource-name <test-resource-name> \
                              --testrun-id <testrun-id>

# List load test run
azmcp loadtesting testrun list --subscription <subscription> \
                               --resource-group <resource-group> \
                               --test-resource-name <test-resource-name> \
                               --test-id <test-id>

# Update load test run
azmcp loadtesting testrun update --subscription <subscription> \
                                 --resource-group <resource-group> \
                                 --test-resource-name <test-resource-name> \
                                 --test-id <test-id> \
                                 --testrun-id <testrun-id> \
                                 --display-name <display-name> \
                                 --description <description>

Comment thread tools/Azure.Mcp.Tools.LoadTesting/CONSOLIDATION_TEST_GUIDE.md Outdated
Copy link
Copy Markdown
Member

@anuchandy anuchandy left a comment

Choose a reason for hiding this comment

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

thanks Vinay!

Comment thread servers/Azure.Mcp.Server/docs/azmcp-commands.md Outdated
- Synchronized command metadata using Update-AzCommandsMetadata.ps1
- Updated 263 metadata lines across 227 tools
- Fixes CommandMetadataSyncTests.AzCommandsMetadata_Should_Be_Synchronized
@g2vinay g2vinay merged commit 31f9122 into microsoft:main Feb 12, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Untriaged to Done in Azure MCP Server Feb 12, 2026
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants