Skip to content

[Doc Gap] Authorization, AzureBackup, FunctionApp, Grafana, and 13 more tools — azmcp-commands.md still implies --retry-* support after PR [Content truncated due to length] #3418

Description

@github-actions

Documentation Gap

Server: Azure.Mcp.Server
Tool directories: tools/Azure.Mcp.Tools.Authorization, tools/Azure.Mcp.Tools.AzureBackup, tools/Azure.Mcp.Tools.FunctionApp, tools/Azure.Mcp.Tools.Grafana, tools/Azure.Mcp.Tools.Insights, tools/Azure.Mcp.Tools.IoTHub, tools/Azure.Mcp.Tools.KeyVault, tools/Azure.Mcp.Tools.LoadTesting, tools/Azure.Mcp.Tools.ManagedLustre, tools/Azure.Mcp.Tools.Redis, tools/Azure.Mcp.Tools.ResilienceManagement, tools/Azure.Mcp.Tools.ServiceFabric, tools/Azure.Mcp.Tools.SignalR, tools/Azure.Mcp.Tools.Speech, tools/Azure.Mcp.Tools.Sql, tools/Azure.Mcp.Tools.VirtualDesktop, tools/Azure.Mcp.Tools.Workbooks
Triggered by: PR #3405 by @alzimmermsft, PR #3409 by @alzimmermsft, PR #3410 by @alzimmermsft, PR #3412 by @alzimmermsft, PR #3413 by @alzimmermsft, PR #3414 by @alzimmermsft
Changed files:

  • tools/Azure.Mcp.Tools.Authorization/src/Options/RoleAssignmentListOptions.cs
  • tools/Azure.Mcp.Tools.AzureBackup/src/Commands/Backup/BackupStatusCommand.cs (and others)
  • tools/Azure.Mcp.Tools.FunctionApp/src/Options/FunctionApp/FunctionAppGetOptions.cs
  • tools/Azure.Mcp.Tools.Grafana/src/Options/Workspace/WorkspaceListOptions.cs
  • tools/Azure.Mcp.Tools.Insights/src/Options/InsightsGetOptions.cs
  • tools/Azure.Mcp.Tools.IoTHub/src/Options/Device/IoTHubDeviceListOptions.cs
  • tools/Azure.Mcp.Tools.LoadTesting/src/Options/LoadTest/LoadTestCreateOptions.cs (and others)
  • tools/Azure.Mcp.Tools.ManagedLustre/src/Commands/FileSystem/AutoexportJob/AutoexportJobCancelCommand.cs (and others)
  • tools/Azure.Mcp.Tools.Redis/src/Options/ResourceCreateOptions.cs
  • tools/Azure.Mcp.Tools.ResilienceManagement/src/Commands/Drills/DrillCreateCommand.cs (and others)
  • tools/Azure.Mcp.Tools.ServiceFabric/src/Options/ManagedCluster/ManagedClusterNodeGetOptions.cs
  • tools/Azure.Mcp.Tools.SignalR/src/Options/Runtime/RuntimeGetOptions.cs
  • tools/Azure.Mcp.Tools.Speech/src/Options/BaseSpeechOptions.cs
  • tools/Azure.Mcp.Tools.VirtualDesktop/src/Options/Hostpool/BaseHostPoolOptions.cs
  • tools/Azure.Mcp.Tools.Workbooks/src/Options/Workbook/ListWorkbooksOptions.cs (and others)

What Changed

PRs #3405 (Part 2), #3409 (Part 5), #3410 (Part 6), #3412 (Part 7), #3413 (Part 8), and #3414 (Part 9) of the systematic RetryPolicyOptions removal removed the --retry-* CLI options from the above toolsets. None of these tools now accept --retry-max-retries, --retry-delay, --retry-max-delay, --retry-mode, or --retry-network-timeout.

Existing issues #3408 and #3416 cover Parts 1, 3, and 4 of this series. This issue covers the remaining uncovered parts (2, 5–9). With Parts 1–9 all merged, no RetryPolicyOptions remain in any file under tools/.

Gaps Found

Files to Update

  • servers/Azure.Mcp.Server/docs/azmcp-commands.md

Context

The global options table in azmcp-commands.md currently includes:

| `--retry-max-retries` | No | 3 | Maximum retry attempts for failed operations |
| `--retry-delay` | No | 2 | Delay between retry attempts (seconds) |
| `--retry-max-delay` | No | 10 | Maximum delay between retries (seconds) |
| `--retry-mode` | No | 'exponential' | Retry strategy ('fixed' or 'exponential') |
| `--retry-network-timeout` | No | 100 | Network operation timeout (seconds) |

With the removal now complete across all 9 parts, these options are no longer valid for any tool command.

📐 Implementation Guide

This section contains step-by-step instructions for a coding agent to implement the changes described above.

Step 1: Modify files

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

Since --retry-* options have been removed from all tools (Parts 1–9 are complete), the correct fix is:

  1. Remove the 5 --retry-* rows from the global options table at the top of the file (lines 15–19).

  2. Update the AKS note (line ~2954) from:

    The aks cluster get and aks nodepool get commands do not support --auth-method (the --retry-* options are still supported).

    To:

    The aks cluster get and aks nodepool get commands do not support --auth-method or any --retry-* options.

  3. Update the applens resource diagnose note (line ~573) — currently says it does not support --retry-*; this note can be removed since retry is no longer a global option.

  4. Update the cloudarchitect design note (line ~5004) similarly — remove the --retry-* exclusion since it's no longer global.

  5. Search the entire file for retry and clean up any remaining references.

Step 2: Verify documentation structure

servers/Azure.Mcp.Server/docs/azmcp-commands.md — global options table must include:

  • --subscription, --resource-group, --tenant, --auth-method rows
  • No --retry-* rows after this fix

Step 3: Validate

Run these commands in order. Each must succeed before proceeding to the next:

  1. dotnet build servers/Azure.Mcp.Server/ — confirms the server project compiles cleanly
  2. .\eng\common\spelling\Invoke-Cspell.ps1 — checks spelling in modified documentation

Next Steps

Tip

Ready for automated implementation? Assign this issue to @copilot to have Copilot coding agent implement the changes described in the Implementation Guide above

Generated by Documentation Updater · ● 13.1M ·

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions