Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated Enable-AzRecoveryServicesBackupProtection by removing warning about ensuring Enhanced Policy for Trusted Launch VMs when configuring protection for Azure VMs . #27284

Merged

Conversation

SourabhSomnathGholap
Copy link
Contributor

@SourabhSomnathGholap SourabhSomnathGholap commented Mar 6, 2025

Description

Updated Enable-AzRecoveryServicesBackupProtection by removing the warning about ensuring Enhanced Policy for Trusted Launch VMs when configuring protection for Azure VMs, as the service now supports protecting TVM with standard policy.

Along with this in this PR I have added breaking change announcement for Get-AzRecoveryServicesBackupSchedulePolicyObject that this command will return a Enhanced policy object by default for IaaSVM workload.

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • For SDK-based development mode, 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.
    • For autorest-based development mode, include the changelog in the PR description.
    • 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

Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@SourabhSomnathGholap SourabhSomnathGholap marked this pull request as ready for review March 6, 2025 10:33
@SourabhSomnathGholap SourabhSomnathGholap marked this pull request as draft March 17, 2025 03:33
@SourabhSomnathGholap SourabhSomnathGholap marked this pull request as ready for review March 18, 2025 10:22
@notyashhh
Copy link
Member

/azp run azure-powershell - security-tools

Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@notyashhh notyashhh self-assigned this Mar 19, 2025
@notyashhh notyashhh added this to the Az 13.4.0 (04/01/2025) milestone Mar 19, 2025
@notyashhh
Copy link
Member

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

Copy link
Member

@notyashhh notyashhh left a comment

Choose a reason for hiding this comment

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

Looks Good!

@notyashhh notyashhh merged commit 0c4fd7f into Azure:main Mar 20, 2025
12 checks passed
@notyashhh notyashhh added the Breaking Change Preannounce This PR adds preannouncement about upcoming breaking change label Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking Change Preannounce This PR adds preannouncement about upcoming breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants