Skip to content

azurerm_data_protection_backup_vault - support for immutability property#27859

Merged
mbfrahry merged 10 commits intomainfrom
kt/dp-backup-vault-immutability
Jan 15, 2025
Merged

azurerm_data_protection_backup_vault - support for immutability property#27859
mbfrahry merged 10 commits intomainfrom
kt/dp-backup-vault-immutability

Conversation

@katbyte
Copy link
Collaborator

@katbyte katbyte commented Nov 1, 2024

Change Log

Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.

  • azurerm_data_protection_backup_vault - support for immutability property [GH-00000]

This is a (please select all that apply):

  • Bug Fix
  • New Feature (ie adding a service, resource, or data source)
  • Enhancement
  • Breaking Change

Related Issue(s)

Fixes #22238

Note

If this PR changes meaningfully during the course of review please update the title and description as required.

@katbyte katbyte requested a review from a team as a code owner November 1, 2024 21:22
Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

Hey @katbyte, this looks good but I think we want to update some of the functionality to line up better with what the portal is doing!

merge main
Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

The CustomizeDiff is perfect but the way it's currently written is a breaking change if people move between versions. I've left comments for how to get it working without it being a breaking change

Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM once linting is good!

mbfrahry
mbfrahry previously approved these changes Nov 22, 2024
Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM

@stephybun stephybun self-assigned this Dec 12, 2024
@mennlo
Copy link

mennlo commented Dec 31, 2024

@mbfrahry can you re-review?

Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM

@mbfrahry mbfrahry changed the title azurerm_data_protection_backup_vault - support for immutability property azurerm_data_protection_backup_vault - support for immutability property Jan 15, 2025
@mbfrahry mbfrahry merged commit 758e738 into main Jan 15, 2025
2 checks passed
@mbfrahry mbfrahry deleted the kt/dp-backup-vault-immutability branch January 15, 2025 23:53
@github-actions github-actions bot added this to the v4.16.0 milestone Jan 15, 2025
mbfrahry added a commit that referenced this pull request Jan 15, 2025
jackofallops added a commit that referenced this pull request Jan 16, 2025
* update for #27680

* Update CHANGELOG.md for #28465

* Update CHANGELOG.md #27932

* Update CHANGELOG.md for #28505

* Update CHANGELOG.md for #28474

* Update CHANGELOG.md #28516

* Update CHANGELOG for #28456

* Update CHANGELOG.md for #28472

* Update CHANGELOG.md #28307

* Update CHANGELOG.md for #27859

* Update for #28519

* Update for #27791 #27528

* Update CHANGELOG.md for #28527

* update changelog links and generate provider schema

---------

Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com>
Co-authored-by: catriona-m <86247157+catriona-m@users.noreply.github.com>
Co-authored-by: sreallymatt <106555974+sreallymatt@users.noreply.github.com>
Co-authored-by: Matthew Frahry <mbfrahry@gmail.com>
Co-authored-by: jackofallops <ste@hashicorp.com>
NotTheEvilOne pushed a commit to b1-systems/terraform-provider-azurerm that referenced this pull request Jan 20, 2025
NotTheEvilOne pushed a commit to b1-systems/terraform-provider-azurerm that referenced this pull request Jan 20, 2025
* update for hashicorp#27680

* Update CHANGELOG.md for hashicorp#28465

* Update CHANGELOG.md hashicorp#27932

* Update CHANGELOG.md for hashicorp#28505

* Update CHANGELOG.md for hashicorp#28474

* Update CHANGELOG.md hashicorp#28516

* Update CHANGELOG for hashicorp#28456

* Update CHANGELOG.md for hashicorp#28472

* Update CHANGELOG.md hashicorp#28307

* Update CHANGELOG.md for hashicorp#27859

* Update for hashicorp#28519

* Update for hashicorp#27791 hashicorp#27528

* Update CHANGELOG.md for hashicorp#28527

* update changelog links and generate provider schema

---------

Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com>
Co-authored-by: catriona-m <86247157+catriona-m@users.noreply.github.com>
Co-authored-by: sreallymatt <106555974+sreallymatt@users.noreply.github.com>
Co-authored-by: Matthew Frahry <mbfrahry@gmail.com>
Co-authored-by: jackofallops <ste@hashicorp.com>
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for Immutable vaults in Azure Backup Vault

5 participants