-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
azurerm_data_protection_backup_vault
: support soft_delete
and retention_duration_in_days
properties
#24775
azurerm_data_protection_backup_vault
: support soft_delete
and retention_duration_in_days
properties
#24775
Changes from 3 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -47,6 +47,14 @@ The following arguments are supported: | |||||
|
||||||
* `identity` - (Optional) An `identity` block as defined below. | ||||||
|
||||||
* `retention_duration_in_days` - (Optional) The soft delete retention duration for this Backup Vault. Possible values are between `14` and `180`. | ||||||
|
||||||
-> **Note:** The `retention_duration_in_days` is the number of days for which deleted data is retained before being permanently deleted. Retention period till 14 days are free of cost, however, retention beyond 14 days may incur additional charges. The `retention_duration_in_days` is required when the `soft_delete_setting` is set to `On`. | ||||||
|
||||||
* `soft_delete_setting` - (Optional) The state of soft delete for this Backup Vault. Possible values are `AlwaysOn`, `Off` and `On`. Defaults to `On`. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i think setting is redundant? as it is implied?
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @katbyte thanks for your feedback. The code has been updated. Could you please take another look? Test results: https://hashicorp.teamcity.com/buildConfiguration/TF_AzureRM_AZURERM_SERVICE_PUBLIC_DATAPROTECTION/102824?buildTab=tests |
||||||
|
||||||
-> **Note:** Once the `soft_delete_setting` is set to `AlwaysOn`, the setting cannot be changed. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should add a customize diff that forces a new resource to be created when changing this value from true to false There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed. |
||||||
|
||||||
* `tags` - (Optional) A mapping of tags which should be assigned to the Backup Vault. | ||||||
|
||||||
--- | ||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The way this is implemented will break users as the default is on but
retention_duration_in_days
hasn't been added to the provider yet. So when people update to this version of the provider, Terraform will error with this message. We should probably defaultretention_duration_in_days
or remove this check.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @mbfrahry thanks a lot. I have updated the code. Could you please take another look?