hashicorp / terraform-provider-azurerm

Terraform provider for Azure Resource Manager
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs
Mozilla Public License 2.0
4.6k stars 4.65k forks source link

[3.114.0][azurerm_redis_cache] Changing enable_authentication to authentication_enabled. #27146

Open veerasuraj opened 2 months ago

veerasuraj commented 2 months ago

Is there an existing issue for this?

Community Note

Terraform Version

1.3.10

AzureRM Provider Version

3.114.0

Affected Resource(s)/Data Source(s)

azurerm_redis_cache

Terraform Configuration Files

resource "azurerm_redis_cache" "main" {
  name                 = "idpe-${var.cluster_name}"
  location             = var.location
  resource_group_name  = local.resource_group
  capacity             = var.capacity
  family               = "P"
  sku_name             = "Premium"
  non_ssl_port_enabled = true
  minimum_tls_version  = "1.2"
  subnet_id            = azurerm_subnet.main.id
  redis_configuration {
    authentication_enabled = false

  }
}

Debug Output/Panic Output

N/A

Expected Behaviour

With the provider in version v3.114.0, changing enable_authentication to authentication_enabled should remove the depreciation warning as per https://github.com/hashicorp/terraform-provider-azurerm/releases/tag/v3.114.0 and #26608

Actual Behaviour

  ~ redis_configuration {
      ~ enable_authentication                   = false -> true
        # (12 unchanged attributes hidden)
    }
}

Pls note that even after replacing this argument with authentication_enabled., terraform plan is throwing a warning pointing out still to enable_authentication and applying a change in value, even after not doing so.

 Also, we did update the azurerm version to 3.116.0 to test, but the issue remains the same. Hope to get a fix for this at your availability.

Screenshot 2024-08-22 at 9 45 15 AM

Steps to Reproduce

No response

Important Factoids

No response

References

No response

xuzhang3 commented 2 months ago

@veerasuraj this change should be fix in v4 and enable_authentication will also be removed in v4

harshavmb commented 2 months ago

This issue looks similar to #26943 & caused by single commit although attributes are different, symptoms are the same.