Powershell STR cmdlet updates for PITR Immutability by invincible-akshay · Pull Request #30215 · Azure/azure-powershell

Akshay Nehe (invincible-akshay) · 2026-09-28T23:22:14Z

🤖 PR Validation — ❌ Action needed

Tests
❌ 58/64

️✔️Az.Accounts

️✔️Az.Compute

️✔️Az.EventHub

️✔️Az.KeyVault

️✔️Az.ManagedServiceIdentity

️✔️Az.Monitor

️✔️Az.Network

️✔️Az.OperationalInsights

️✔️Az.PrivateDns

❌Az.Sql

️✔️Az.Storage

️✔️Az.TrafficManager

Description

Added short-term backup immutability support to Set-AzSqlDatabaseBackupShortTermRetentionPolicy and Set-AzSqlInstanceDatabaseBackupShortTermRetentionPolicy.
Added the LockImmutability parameter and ImmutabilityStatus output property.
Restricted LockImmutability to live Managed Instance databases.
Added an irreversible-action confirmation prompt when locking immutability, which can be bypassed with -Force.
Added an override for DiffBackupIntervalInHours datatype as an AutoRest directive to force int32 instead of double, related to Azure/typespec-azure#919

Mandatory Checklist

Please choose the target release of Azure PowerShell. (⚠️Target release is a different concept from API readiness. Please click below links for details.)
- General release
- Public preview
- Private preview
- Engineering build
- No need for a release
Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

SHOULD update ChangeLog.md file(s) appropriately
- Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
  - A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
- Should not change ChangeLog.md if no new release is required, such as fixing test case only.
SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
SHOULD have proper test coverage for changes in pull request.
SHOULD NOT adjust version of module manually in pull request

… property for SqlDb and MI

azure-pipelines · 2026-09-28T23:22:24Z

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

Copilot

Copilot review overview

🟡 Changes recommended

The SQL Database cmdlet currently rejects valid lock-only updates unless another retention setting is supplied.

Review effort: Balanced
Findings: 1

Open (1)

Allow lock-only invocation for retention policy updates · New

What changed in this PR

Adds point-in-time backup immutability support to Azure SQL Database and Managed Instance short-term retention cmdlets.

Changes:

Adds LockImmutability, Force, and ImmutabilityStatus.
Updates the SQL SDK to API version 2026-08-01-preview.
Adds help, changelog entries, and scenario coverage.

File	Description
`src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupAdapter.cs`	Forwards managed-database immutability settings.
`src/Sql/Sql/ManagedDatabase Backup/Model/AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel.cs`	Exposes immutability status.
`src/Sql/Sql/ManagedDatabase Backup/Cmdlet/SetAzureSqlManagedDatabaseBackupShortTermRetentionPolicy.cs`	Adds locking, validation, and confirmation.
`src/Sql/Sql/help/Set-AzSqlInstanceDatabaseBackupShortTermRetentionPolicy.md`	Documents Managed Instance locking.
`src/Sql/Sql/help/Set-AzSqlDatabaseBackupShortTermRetentionPolicy.md`	Documents SQL Database locking.
`src/Sql/Sql/help/Get-AzSqlInstanceDatabaseBackupShortTermRetentionPolicy.md`	Documents Managed Instance status output.
`src/Sql/Sql/help/Get-AzSqlDatabaseBackupShortTermRetentionPolicy.md`	Documents SQL Database status output.
`src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupAdapter.cs`	Forwards SQL Database lock settings.
`src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs`	Adds immutability model properties.
`src/Sql/Sql/Database Backup/Cmdlet/SetAzureSqlDatabaseBackupShortTermRetentionPolicy.cs`	Adds locking and confirmation behavior.
`src/Sql/Sql/ChangeLog.md`	Records the user-facing feature.
`src/Sql/Sql.Test/ScenarioTests/ManagedDatabaseBackupTests.ps1`	Tests Managed Instance immutability.
`src/Sql/Sql.Test/ScenarioTests/ManagedDatabaseBackupTests.cs`	Registers the Managed Instance scenario.
`src/Sql/Sql.Test/ScenarioTests/DatabaseBackupTests.ps1`	Tests SQL Database immutability.
`src/Sql/Sql.Test/ScenarioTests/DatabaseBackupTests.cs`	Registers the SQL Database scenario.
`src/Sql/Sql.Management.Sdk/README.md`	Updates SDK inputs and type transformation.
`src/Sql/Sql.Management.Sdk/Generated/Models/ManagedRestorableDroppedDatabaseBackupShortTermRetentionPoliciesUpdateHeaders.cs`	Adds generated update headers.
`src/Sql/Sql.Management.Sdk/Generated/Models/ManagedRestorableDroppedDatabaseBackupShortTermRetentionPoliciesCreateOrUpdateHeaders.cs`	Adds generated create/update headers.
`src/Sql/Sql.Management.Sdk/Generated/Models/ManagedBackupShortTermRetentionPolicyProperties.cs`	Adds managed immutability properties.
`src/Sql/Sql.Management.Sdk/Generated/Models/ManagedBackupShortTermRetentionPolicy.cs`	Updates the managed policy resource.
`src/Sql/Sql.Management.Sdk/Generated/Models/ManagedBackupShortTermRetentionPoliciesUpdateHeaders.cs`	Adds managed update headers.
`src/Sql/Sql.Management.Sdk/Generated/Models/ManagedBackupShortTermRetentionPoliciesCreateOrUpdateHeaders.cs`	Adds managed create/update headers.
`src/Sql/Sql.Management.Sdk/Generated/Models/ImmutabilityStatus.cs`	Defines generated status constants.
`src/Sql/Sql.Management.Sdk/Generated/Models/BackupShortTermRetentionPolicyProperties.cs`	Adds database immutability properties.
`src/Sql/Sql.Management.Sdk/Generated/Models/BackupShortTermRetentionPolicy.cs`	Updates the database policy resource.
`src/Sql/Sql.Management.Sdk/Generated/Models/BackupShortTermRetentionPoliciesUpdateHeaders.cs`	Adds database update headers.
`src/Sql/Sql.Management.Sdk/Generated/Models/BackupShortTermRetentionPoliciesCreateOrUpdateHeaders.cs`	Adds database create/update headers.
`src/Sql/Sql.Management.Sdk/Generated/ManagedRestorableDroppedDatabaseBackupShortTermRetentionPoliciesOperationsExtensions.cs`	Regenerates dropped-database extensions.
`src/Sql/Sql.Management.Sdk/Generated/ManagedRestorableDroppedDatabaseBackupShortTermRetentionPoliciesOperations.cs`	Regenerates dropped-database operations.
`src/Sql/Sql.Management.Sdk/Generated/ManagedBackupShortTermRetentionPoliciesOperationsExtensions.cs`	Regenerates managed policy extensions.
`src/Sql/Sql.Management.Sdk/Generated/ManagedBackupShortTermRetentionPoliciesOperations.cs`	Regenerates managed policy operations.
`src/Sql/Sql.Management.Sdk/Generated/IManagedRestorableDroppedDatabaseBackupShortTermRetentionPoliciesOperations.cs`	Updates dropped-policy SDK contracts.
`src/Sql/Sql.Management.Sdk/Generated/IManagedBackupShortTermRetentionPoliciesOperations.cs`	Updates managed-policy SDK contracts.
`src/Sql/Sql.Management.Sdk/Generated/IBackupShortTermRetentionPoliciesOperations.cs`	Updates database-policy SDK contracts.
`src/Sql/Sql.Management.Sdk/Generated/BackupShortTermRetentionPoliciesOperationsExtensions.cs`	Regenerates database policy extensions.
`src/Sql/Sql.Management.Sdk/Generated/BackupShortTermRetentionPoliciesOperations.cs`	Regenerates database policy operations.

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

Aditya Pujara (a0x1ab) · 2026-09-28T23:33:35Z

/azp run

azure-pipelines · 2026-09-28T23:33:52Z

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

Copilot

🔵 Needs a closer look

The irreversible locking workflow and preview API migration require human validation of service-rollout compatibility.

0 open findings

1 resolved since last review

Allow lock-only invocation for retention policy updates

Previously missed (1)

In code that hasn't changed since last review

Preserve database identity and deletion date when returning adapter results

src/Sql/Sql/ManagedDatabase Backup/Cmdlet/SetAzureSqlManagedDatabaseBackupShortTermRetentionPolicy.cs:187

Returning the adapter result changes the output for dropped databases: UpsertDeletedManagedDatabaseRetentionPolicy passes the combined databaseName,timestamp identifier to the model converter and omits deletionDate (AzureSqlManagedDatabaseBackupAdapter.cs:147-149). The returned DatabaseName therefore gains the timestamp suffix and DeletionDate becomes null. Previously, returning entity preserved both fields. Restore the original identity fields while retaining the service's updated policy values.

🧠 Review effort: Balanced

Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Aditya Pujara (a0x1ab) · 2026-10-08T03:45:25Z

/azp run

azure-pipelines · 2026-10-08T03:45:32Z

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

NoriZC · 2026-10-08T04:31:35Z

/azp run

azure-pipelines · 2026-10-08T04:31:52Z

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

Copilot

🔵 Needs a closer look

Irreversible locking and broad preview-API regeneration require human validation beyond the mocked-only scenarios.

2 open findings

Add confirmation and WhatIf tests for retention locking · New
Restore identity fields on dropped database retention models · New

🧠 Review effort: Balanced

Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Aditya Pujara (a0x1ab) · 2026-10-08T04:41:28Z

/azp run

azure-pipelines · 2026-10-08T04:41:45Z

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

Copilot

🔵 Needs a closer look

The broad generated SDK migration and irreversible locking behavior warrant final human validation.

2 open findings

Restore identity fields on dropped database retention models
Add confirmation and WhatIf tests for retention locking

🧠 Review effort: Balanced

Aditya Pujara (a0x1ab) · 2026-10-09T08:35:43Z

/azp run

azure-pipelines · 2026-10-09T08:36:01Z

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

Copilot

🔵 Needs a closer look

Irreversible locking behavior and extensive generated SDK contract changes warrant final human validation.

2 open findings

🧠 Review effort: Balanced

NoriZC · 2026-10-13T05:33:44Z

/azp run

azure-pipelines · 2026-10-13T05:33:59Z

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

Adding STR LockImmutability input param and ImmutabilityStatus output…

c7551f6

… property for SqlDb and MI

Copilot AI balanced review requested due to automatic review settings September 28, 2026 23:22

Copilot started reviewing on behalf of Akshay Nehe (invincible-akshay) September 28, 2026 23:22 View session

Akshay Nehe (invincible-akshay) mentioned this pull request Sep 28, 2026

Powershell STR cmdlet updates for PITR Immutability #30187

Closed

2 of 6 tasks

Copilot AI reviewed Sep 28, 2026

View reviewed changes

Comment thread src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs

DanielMicrosoft added this to the Az 16.5.0 (2026-11-03) milestone Sep 29, 2026

svmicano approved these changes Oct 1, 2026

View reviewed changes

Update ChangeLog.md

850d77e

Copilot AI balanced review requested due to automatic review settings October 8, 2026 03:28

Copilot started reviewing on behalf of Akshay Nehe (invincible-akshay) October 8, 2026 03:29 View session

Copilot AI reviewed Oct 8, 2026

View reviewed changes

NoriZC previously approved these changes Oct 8, 2026

View reviewed changes

Merge branch 'main' into dev/akshaynehe/str_api_powershell

44c67ee

NoriZC dismissed their stale review via 44c67ee October 8, 2026 04:31

Copilot AI balanced review requested due to automatic review settings October 8, 2026 04:31

Copilot started reviewing on behalf of Akshay Nehe (invincible-akshay) October 8, 2026 04:31 View session

Copilot AI reviewed Oct 8, 2026

View reviewed changes

Comment thread src/Sql/Sql/Database Backup/Cmdlet/SetAzureSqlDatabaseBackupShortTermRetentionPolicy.cs

Comment thread ...ql/ManagedDatabase Backup/Cmdlet/SetAzureSqlManagedDatabaseBackupShortTermRetentionPolicy.cs

pixia reviewed Oct 8, 2026

View reviewed changes

Comment thread src/Sql/Sql.Test/ScenarioTests/DatabaseBackupTests.ps1 Outdated

pixia reviewed Oct 8, 2026

View reviewed changes

Comment thread src/Sql/Sql.Test/ScenarioTests/DatabaseBackupTests.ps1

pixia reviewed Oct 8, 2026

View reviewed changes

Comment thread src/Sql/Sql.Test/ScenarioTests/ManagedDatabaseBackupTests.ps1

Copilot AI balanced review requested due to automatic review settings October 9, 2026 08:28

Copilot started reviewing on behalf of Akshay Nehe (invincible-akshay) October 9, 2026 08:29 View session

Copilot AI reviewed Oct 9, 2026

View reviewed changes

svmicano mentioned this pull request Oct 9, 2026

[SQL] Add short-term backup retention immutability support Azure/azure-cli#34207

Open

3 tasks done

Updated tests and AzureSqlManagedDatabaseBackupAdapter

24e0bda

Akshay Nehe (invincible-akshay) force-pushed the dev/akshaynehe/str_api_powershell branch from 371c477 to 24e0bda Compare October 9, 2026 18:24

Copilot AI balanced review requested due to automatic review settings October 9, 2026 18:24

Copilot started reviewing on behalf of Akshay Nehe (invincible-akshay) October 9, 2026 18:25 View session

Copilot AI reviewed Oct 9, 2026

View reviewed changes

NoriZC approved these changes Oct 13, 2026

View reviewed changes

NoriZC self-assigned this Oct 13, 2026

Conversation

Akshay Nehe (invincible-akshay) commented Sep 28, 2026 • edited by x-engineering-agent Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 PR Validation — ❌ Action needed

Description

Mandatory Checklist

Uh oh!

azure-pipelines Bot commented Sep 28, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Aditya Pujara (a0x1ab) commented Sep 28, 2026

Uh oh!

azure-pipelines Bot commented Sep 28, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Aditya Pujara (a0x1ab) commented Oct 8, 2026

Uh oh!

azure-pipelines Bot commented Oct 8, 2026

Uh oh!

NoriZC commented Oct 8, 2026

Uh oh!

azure-pipelines Bot commented Oct 8, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Uh oh!

Uh oh!

Aditya Pujara (a0x1ab) commented Oct 8, 2026

Uh oh!

azure-pipelines Bot commented Oct 8, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Aditya Pujara (a0x1ab) commented Oct 9, 2026

Uh oh!

azure-pipelines Bot commented Oct 9, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

NoriZC commented Oct 13, 2026

Uh oh!

azure-pipelines Bot commented Oct 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

Akshay Nehe (invincible-akshay) commented Sep 28, 2026 •

edited by x-engineering-agent Bot

Loading