azurerm_recovery_services_vault - introduce soft_delete_status #27593
+117
−21
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
introduce soft_delete_status for azurerm_recovery_services_vault
Supersede soft_delete_state (boolean) by soft_delete_status (String)
Community Note
Description
I am trying to fix #23557 , Azure did a bad thing here, it changed a previous boolean value to a something with 4 states. So I think this needs to be addressed in a backwards compatible way by introducing a new attribute for it.
I do have a problem that I have not yet solved, I think within
https:/Klaas-/terraform-provider-azurerm/blob/3227a9bbd9dee25c8b41cfeb10452d5327b69a5a/internal/services/recoveryservices/recovery_services_vault_resource.go#L351-L356
I need to tell tf that the value of soft_delete_status is not the default true but false.
Also I was unsure if I should update all tests to use the new attribute (which I currently did) or if I should write separate tests. Looking for guidance here. I have not yet run the full tests suite because I noticed the problem described above in my manual tests.
PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Testing
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
azurerm_recovery_services_vault
- add soft_delete_status property [GH-00000]This is a (please select all that apply):
Related Issue(s)
Fixes #23557
Note
If this PR changes meaningfully during the course of review please update the title and description as required.