Skip to content

Document Advisor recommendation apply resource values - #3274

Draft
Connie Yau (conniey) with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-docs-missing-resource-values
Draft

Document Advisor recommendation apply resource values#3274
Connie Yau (conniey) with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-docs-missing-resource-values

Conversation

Copilot AI commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

azmcp advisor recommendation apply accepts a constrained set of non-obvious --resource identifiers. The command reference previously omitted those accepted values.

  • Command reference
    • Lists all 20 supported --resource identifiers beside the command invocation.
azmcp advisor recommendation apply --resource storage_storageaccounts

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: conniey <10136526+conniey@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix documentation for azmcp advisor recommendation apply --resource values Document Advisor recommendation apply resource values Aug 12, 2026

# Apply Advisor recommendation to create or modify IaaC files (like ARM, Terraform) for Azure resources
# ❌ Destructive | ✅ Idempotent | ❌ OpenWorld | ✅ ReadOnly | ❌ Secret | ❌ LocalRequired
<!-- cspell:ignore domainservices virtualmachinescalesets containerregistry containerservice managedclusters flexibleservers databaseaccounts connectedclusters kubernetesconfiguration applicationgatewaywebapplicationfirewallpolicies expressrouteports frontdoorwebapplicationfirewallpolicies managedinstances staticsites -->

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copilot the cspell ignores should be put into https://github.com/microsoft/mcp/blob/main/servers/Azure.Mcp.Server/cspell.yaml rather than inline here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Moved the 14 cspell ignores into servers/Azure.Mcp.Server/cspell.yaml and removed the inline directive in cb9243e.

Co-authored-by: conniey <10136526+conniey@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

None yet

Development

Successfully merging this pull request may close these issues.

[Doc Gap] azmcp advisor recommendation apply - missing valid --resource values documentation

2 participants