azurerm_storage_account: set default for min_tls_version
#8152
Conversation
For those storage account created using old API version (at least 2018-02-01, before provider v1.29.0), the GET call of new API version will not return the `minimumTLSVersion` in response body. So those users will get a diff. This PR sets the default value for `min_tls_version` in case it is missing in response, using the same value as the default value defined in schema.
|
Due to the the particular conditions required to reproduce the issue fixed by this PR I am skipping the normally required full test pass of the resource and running only a single test case on the PR to verify the fix does not cause a regression to the resource. I have been assured by the PR author that this has been manually tested and works as expected. |
|
|
|
This has been released in version 2.24.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example: provider "azurerm" {
version = "~> 2.24.0"
}
# ... other configuration ... |
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
ghost
locked and limited conversation to collaborators
For those storage account created using old API version (at least
2018-02-01, before provider v1.29.0), the GET call of new API version
will not return the
minimumTLSVersionin response body. So those userswill get a diff.
This PR sets the default value for
min_tls_versionin case it ismissing in response, using the same value as the default value defined
in schema.
At least fixes the issue encountered by this user comment.