Skip to content

[Key Vault] az keyvault ekm-connection: Remove preview status - #34095

Open
Yash (notyashhh) wants to merge 1 commit into
devfrom
yash/ekm-ga-remove-preview
Open

Yash (notyashhh) wants to merge 1 commit into
devfrom
yash/ekm-ga-remove-preview

Conversation

@notyashhh

@notyashhh Yash (notyashhh) commented Sep 17, 2026

Copy link
Copy Markdown
Member

🤖 PR Validation — ⚠️ Review suggested

Breaking Changes Tests
⚠️ None ️✔️ 130/130
⚠️AzureCLI-BreakingChangeTest
⚠️keyvault
rule cmd_name rule_message suggest_message
⚠️ 1004 - CmdPropRemove keyvault ekm-connection certificate show cmd keyvault ekm-connection certificate show removed property is_preview
⚠️ 1004 - CmdPropRemove keyvault ekm-connection check cmd keyvault ekm-connection check removed property is_preview
⚠️ 1004 - CmdPropRemove keyvault ekm-connection create cmd keyvault ekm-connection create removed property is_preview
⚠️ 1004 - CmdPropRemove keyvault ekm-connection delete cmd keyvault ekm-connection delete removed property is_preview
⚠️ 1004 - CmdPropRemove keyvault ekm-connection show cmd keyvault ekm-connection show removed property is_preview
⚠️ 1004 - CmdPropRemove keyvault ekm-connection update cmd keyvault ekm-connection update removed property is_preview

Related command
az keyvault ekm-connection
az keyvault key create --external-key-id

Description
Removes the preview labels from the EKM connection command group, its certificate subgroup, and --external-key-id for EKM GA. No SDK, API version, or command behavior changes.

Related to Azure/CLIPS#605 (CLI portion only).

Testing Guide

  • Verified 10 help views covering both groups, all six commands, the parent listing, and --external-key-id. The relevant preview labels are gone.
  • Ran az keyvault ekm-connection show and az keyvault ekm-connection certificate show against an existing Managed HSM. Both returned HTTP 200, and the returned certificates parsed successfully.
  • Live testing was GET-only. No resources were changed. Write operations and documentation generation were not tested locally.

History Notes
[Key Vault] az keyvault ekm-connection: Remove preview status
[Key Vault] az keyvault key create: Remove preview status from --external-key-id


Copilot AI lite review requested due to automatic review settings September 17, 2026 00:18
@notyashhh
Yash (notyashhh) requested a review from a team as a code owner September 17, 2026 00:18
@yonzhan

Copy link
Copy Markdown
Collaborator

Thank you for your contribution! 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.

🟢 Approval recommended

No unresolved issues were identified that would block approval.

Pull request overview

Removes preview metadata from the Key Vault EKM connection commands and --external-key-id, promoting them to GA without changing behavior.

Changes:

  • Removes preview status from EKM command groups.
  • Removes preview status from --external-key-id.
File summaries
File Description
src/azure-cli/azure/cli/command_modules/keyvault/commands.py Updated as part of this pull request.
src/azure-cli/azure/cli/command_modules/keyvault/_params.py Updated as part of this pull request.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

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

Labels

act-identity-squad Auto-Assign Auto assign by bot KeyVault az keyvault

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants