Skip to content

docs: remove subscription from resilience get examples - #3386

Open
Donna Gan (bleakbelladonnals) wants to merge 1 commit into
microsoft:mainfrom
bleakbelladonnals:codex/docs-resilience-get-3072
Open

docs: remove subscription from resilience get examples#3386
Donna Gan (bleakbelladonnals) wants to merge 1 commit into
microsoft:mainfrom
bleakbelladonnals:codex/docs-resilience-get-3072

Conversation

@bleakbelladonnals

Copy link
Copy Markdown

What does this PR do?

Removes the stale --subscription argument from seven Azure Resilience Management get-command examples:

  • goal template, assignment, and resource get
  • recovery plan and recovery plan resource get
  • recovery job and recovery job resource get

The corresponding option classes are tenant-scoped and no longer expose a subscription option. Subscription arguments remain documented for usage plan and usage plan enrollment commands, where they are still supported.

This is a documentation-only change; it does not change command behavior or tool metadata.

GitHub issue number?

Fixes #3072

Validation

  • reviewed all seven current option classes to confirm they do not expose Subscription
  • verified usage plan and usage plan enrollment examples still include --subscription
  • git diff --check
  • npx --yes cspell@^10.0.0 lint --config .vscode/cspell.json servers/Azure.Mcp.Server/docs/azmcp-commands.md

The PowerShell metadata script was not run because no generated tool metadata or tool implementation changed.

Pre-merge checklist

  • Read the contribution guidelines
  • PR title clearly describes the change
  • Commit history is clean with a single descriptive commit
  • Reviewed the source option classes for accuracy
  • Spelling check passes
  • Tests are not applicable to this documentation-only change
  • A changelog entry is not required for this documentation-only correction

@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.

@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution Donna Gan (@bleakbelladonnals)! We will review the pull request and get back to you soon.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Azure MCP Server command reference to remove outdated --subscription usage from Azure Resilience Management get command examples, aligning documentation with the current tenant-scoped option classes.

Changes:

  • Removed --subscription <subscription> from seven Resilience Management get-command example invocations (goal and recovery get variants).
  • Kept --subscription documented for usageplan and usageplan enrollment commands where it is still supported.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bleakbelladonnals

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc Gap] Resilience Management commands docs show removed --subscription parameter

2 participants