Skip to content

{SQL} az sql midb ltr-policy/ltr-backup: Remove preview status - #34029

Merged
Ethan Yang (necusjz) merged 1 commit into
Azure:devfrom
svmicano:svmicano/remove-midb-ltr-preview
Sep 4, 2026
Merged

{SQL} az sql midb ltr-policy/ltr-backup: Remove preview status#34029
Ethan Yang (necusjz) merged 1 commit into
Azure:devfrom
svmicano:svmicano/remove-midb-ltr-preview

Conversation

@svmicano

@svmicano svmicano commented Sep 3, 2026

Copy link
Copy Markdown
Member

🤖 PR Validation — ⚠️ Review suggested

Breaking Changes Tests
⚠️ None ️✔️ 130/130
⚠️AzureCLI-BreakingChangeTest
⚠️sql
rule cmd_name rule_message suggest_message
⚠️ 1004 - CmdPropRemove sql midb ltr-backup delete cmd sql midb ltr-backup delete removed property is_preview
⚠️ 1004 - CmdPropRemove sql midb ltr-backup list cmd sql midb ltr-backup list removed property is_preview
⚠️ 1004 - CmdPropRemove sql midb ltr-backup restore cmd sql midb ltr-backup restore removed property is_preview
⚠️ 1004 - CmdPropRemove sql midb ltr-backup show cmd sql midb ltr-backup show removed property is_preview
⚠️ 1004 - CmdPropRemove sql midb ltr-policy set cmd sql midb ltr-policy set removed property is_preview
⚠️ 1004 - CmdPropRemove sql midb ltr-policy show cmd sql midb ltr-policy show removed property is_preview

Related command

az sql midb ltr-policy set
az sql midb ltr-policy show
az sql midb ltr-backup show
az sql midb ltr-backup list
az sql midb ltr-backup delete
az sql midb ltr-backup restore

Description

Remove the preview status from Azure SQL Managed Instance long-term retention policy and backup commands. These commands no longer display the preview warning when invoked.

Related work item: https://msdata.visualstudio.com/Database%20Systems/_workitems/edit/3093099/

Testing Guide

Passed local no-build validation in the PR checkout:

  • Used a PowerShell source-metadata check to verify stable command registrations for:
    • az sql midb ltr-policy set
    • az sql midb ltr-policy show
    • az sql midb ltr-backup show
    • az sql midb ltr-backup list
    • az sql midb ltr-backup delete
    • az sql midb ltr-backup restore
  • Confirmed no is_preview metadata remains in the Managed Instance LTR command registration region.
  • git diff --check HEAD^ HEAD passed with no whitespace errors.

Not run: package build or live invocation using a locally installed development CLI.


@svmicano
svmicano requested a review from a team as a code owner September 3, 2026 14:37
Copilot AI lite review requested due to automatic review settings September 3, 2026 14:37
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

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

The change is limited to removing preview metadata on existing command registrations with no functional logic changes.

Pull request overview

This PR updates Azure SQL Managed Instance long-term retention (LTR) command registrations so the az sql midb ltr-policy and az sql midb ltr-backup commands are no longer marked as preview, removing the preview warning shown when these commands are invoked.

Changes:

  • Removed is_preview=True from az sql midb ltr-policy set/show command registrations.
  • Removed is_preview=True from az sql midb ltr-backup show/list/delete/restore command registrations.
File summaries
File Description
src/azure-cli/azure/cli/command_modules/sql/commands.py Removes preview metadata from MI LTR policy/backup command registrations so the CLI no longer emits preview warnings for these commands.
Review details
  • Files reviewed: 1/1 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.

@a0x1ab

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).

@svmicano svmicano changed the title [SQL] az sql midb ltr-policy/ltr-backup: Remove preview status [SQL] az sql midb ltr-policy/ltr-backup: Remove preview status Sep 3, 2026
@svmicano
svmicano force-pushed the svmicano/remove-midb-ltr-preview branch from eac4426 to 6f226cb Compare September 3, 2026 15:26
@necusjz Ethan Yang (necusjz) changed the title [SQL] az sql midb ltr-policy/ltr-backup: Remove preview status {SQL} az sql midb ltr-policy/ltr-backup: Remove preview status Sep 4, 2026
@necusjz

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 3 pipeline(s).

@necusjz
Ethan Yang (necusjz) merged commit ae54f66 into Azure:dev Sep 4, 2026
50 checks passed
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.

4 participants