Azure / terraform-azurerm-avm-res-documentdb-databaseaccount

https://registry.terraform.io/modules/Azure/avm-res-documentdb-databaseaccount/azurerm/latest
MIT License
2 stars 1 forks source link

[AVM Module Issue]: add support for Azure Provider v4.0 #54

Open chopeen opened 1 month ago

chopeen commented 1 month ago

Check for previous/existing GitHub issues

Issue Type?

Feature Request

Description

Modify the module to support azurerm v4 (see 4.0-upgrade-guide for change log and migration hints).

Related issues in other AVMs

Some of these have already been implemented and closed, so maybe they will be useful for reference.

chopeen commented 1 month ago

This may be one of required fixes:

│ Warning: Argument is deprecated
│ 
│   with module.avm_res_cosmos_db_account.azurerm_cosmosdb_sql_container.this["...],
│   on .terraform/modules/avm_res_cosmos_db_account/main.sql.tf line 34, in resource "azurerm_cosmosdb_sql_container" "this":
│   34:   partition_key_path     = each.value.container_params.partition_key_path
│ 
│ `partition_key_path` will be removed in favour of the property
│ `partition_key_paths` in version 4.0 of the AzureRM Provider.
bryansan-msft commented 1 month ago

I have this in mind but to be honest is a big effort task so it will take some time to have this done