Support for TLS minimal version - azurerm_sql_server

[HarleyB123]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Much like the solution for mssql_server released in version 2.31.0, I would like that same implementation for the azurerm_sql_server resource, as currently the TLS option is greyed out in the portal and has to be manually set there.

• azurerm_2.31.0

References

• https://docs.microsoft.com/en-us/azure/azure-sql/database/connectivity-settings#minimal-tls-version
• mssql_server: add minimal_tls_version property #8361

[HarleyB123]

Just been made aware of the discussion in #8338, could the documentation be updated to reflect this?

[SebSa]

Just investigating this, there is a comment in the azurerm_sql_server resource file declaring an intent to deprecate with azurerm 4.0.

terraform-provider-azurerm/internal/services/sql/sql_server_resource.go

Line 36 in 56d7347

DeprecationMessage: "The `azurerm_sql_server` resource is deprecated and will be removed in version 4.0 of the AzureRM provider. Please use the `azurerm_mssql_server` resource instead.",

Perhaps this issue should be closed.

[github-actions]

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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.