azurerm_mssql_database / azurerm_mssql_server - Added fields for database-level TDE

[dkuzmenok]

Changes

• Fixed issue with setting transparent_data_encryption_enabled, when it is already set to same value. It was causing policy failures (looks like Setting Enabled to already Enabled was causing Disable -> Enable chain in new API, so added a check to prevent it).
• In data source azurerm_mssql_database new fields: identity, transparent_data_encryption_enabled, transparent_data_encryption_key_vault_key_id, auto_key_rotation_enabled
• In data source azurerm_mssql_server new fields: transparent_data_encryption_key_vault_key_id
• In resource azurerm_mssql_database new fields: identity, transparent_data_encryption_key_vault_key_id, auto_key_rotation_enabled

Basically we add support for setting TDE on a database level, that is supported with a newer API we now use.
Tests passed on my side.

Fixes #24316
Fixes #24253

[katbyte]

Thanks for the PR @dkuzmenok - but it looks like we have a test failure

------- Stdout: -------
=== RUN   TestAccMsSqlDatabase_complete
=== PAUSE TestAccMsSqlDatabase_complete
=== CONT  TestAccMsSqlDatabase_complete
    testcase.go:113: Step 3/4 error: Error running apply: exit status 1
        
        Error: unable to parse key: "": cannot parse azure key vault child ID: parse "": empty url
        
          with azurerm_mssql_database.test,
          on terraform_plugin_test.tf line 37, in resource "azurerm_mssql_database" "test":
          37: resource "azurerm_mssql_database" "test" {
        
--- FAIL: TestAccMsSqlDatabase_complete (564.94s)
FAIL

[dkuzmenok]

Thanks for the PR @dkuzmenok - but it looks like we have a test failure

------- Stdout: -------
=== RUN   TestAccMsSqlDatabase_complete
=== PAUSE TestAccMsSqlDatabase_complete
=== CONT  TestAccMsSqlDatabase_complete
    testcase.go:113: Step 3/4 error: Error running apply: exit status 1
        
        Error: unable to parse key: "": cannot parse azure key vault child ID: parse "": empty url
        
          with azurerm_mssql_database.test,
          on terraform_plugin_test.tf line 37, in resource "azurerm_mssql_database" "test":
          37: resource "azurerm_mssql_database" "test" {
        
--- FAIL: TestAccMsSqlDatabase_complete (564.94s)
FAIL

Please, make another run after my update to test cases.

[WodansSon]

The two failures are due to API internal server errors and the other failure is in VM's so not related to this PR.

[katbyte]

thanks for fixing the test @dkuzmenok - aside from one comment about a property name i think this looks good

[WodansSon]

Both test fail with internal server errors, API issue... LGTM! 🚀

[github-actions]

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.