Support for active geo-replication in `azurerm_redis_enterprise_database`

[MrDrMcCoy]

Is there an existing issue for this?

• [X] I have searched the existing issues

Community Note

• Please vote on this issue by adding a :thumbsup: 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

Please add support for active geo-replication in the Enterprise Azure Cache for Redis. The only other way to automate this is via Bicep, which we do not wish to introduce into our Terraform environment.

New or Affected Resource(s)/Data Source(s)

azurerm_redis_enterprise_database

Potential Terraform Configuration

resource "azurerm_redis_enterprise_database" "example" {
  name                = "default"
  resource_group_name = azurerm_resource_group.example.name
  cluster_id = azurerm_redis_enterprise_cluster.example.id
  geo_replication {
    name = "groupNickname"
    linked_databases = ["id", ...]
  }
}

References

Docs:

• https://docs.microsoft.com/en-us/azure/azure-cache-for-redis/cache-how-to-active-geo-replication
• https://docs.microsoft.com/en-us/azure/templates/microsoft.cache/redisenterprise/databases?tabs=bicep#databaseproperties

[xiaxyi]

The feature is merged:https://github.com/hashicorp/terraform-provider-azurerm/pull/16045

[MrDrMcCoy]

@xiaxyi This is very exciting, but I am not able to get it to work. When attempting to create the following resources:

resource "azurerm_redis_enterprise_cluster" "geo" {
  for_each = var.config.aks_regions
  location = each.key
  minimum_tls_version = "1.2"
  name = "anark${var.resource_group.name}${each.key}geo"
  resource_group_name = var.resource_group.name
  sku_name = var.config.redis_geo.sku_name
}

resource "azurerm_redis_enterprise_database" "geo" {
  name = "default"
  client_protocol = "Encrypted"
  cluster_id = azurerm_redis_enterprise_cluster.geo[keys(var.config.aks_regions)[0]].id
  clustering_policy = "EnterpriseCluster"
  linked_database_id = [
    for _, value in azurerm_redis_enterprise_cluster.geo :
      "${value.id}/databases/default"
  ]
}

It results in the following error:

│ Error: evictionPolicy must be set to NoEviction when using RediSearch module
│ 
│   with module.az_resource_group["dev2"].azurerm_redis_enterprise_database.geo,
│   on az_resource_group/redis.tf line 37, in resource "azurerm_redis_enterprise_database" "geo":
│   37: resource "azurerm_redis_enterprise_database" "geo" {

We are not interested in the RediSearch module, nor are we interested in disabling eviction. From the available documentation that I have seen, there is no way to exclude these add-on modules. Should I open a new issue for this?

[xiaxyi]

@MrDrMcCoy Let me check the code, besides, can you be more specific about the there is no way to exclude these add-on modules?

no need to open another issue, we can do the follow-up in the thread.

[MrDrMcCoy]

can you be more specific about the there is no way to exclude these add-on modules?

Sure. The module {} block only allows you to specify name and args, but not enabled. The fact that I am getting the error above despite not requesting any modules to be installed suggests that these modules are getting enabled by default when creating a geo-replicated database. Short of a mechanism to exclude these modules, the default behavior of this resource creation would need to be changed to not include them.

[xiaxyi]

@MrDrMcCoy May I know if any of the databases which are involved in the active-geo linking process are created before?

[MrDrMcCoy]

@xiaxyi it is my understanding that, given the example I posted earlier, these databases are not created until the clusters they reference are available. Neither the databases nor the clusters existed previously, as they are created from nothing by Terraform. Is that what you were looking for?

[xiaxyi]

@MrDrMcCoy yes, Thanks for the clarification. I will perform more tests to have the issue reproduced from my side

[MrDrMcCoy]

@xiaxyi any news on this issue? The ability to create and manage Azure Redis Enterprise with geo-replication via Terraform is very important for my company and we are blocked by this.

[xiaxyi]

https://github.com/hashicorp/terraform-provider-azurerm/pull/16700