Unable to update tags (OR the lack of tags support) for the azurerm_redis_cache resource

[hashibot]

This issue was originally opened by @horsey as hashicorp/terraform#13892. It was migrated here as part of the provider split. The original body of the issue is below.

---

Terraform Version

v0.9.2

Affected Resource(s)

azurerm_redis_cache

Terraform Configuration Files

resource "azurerm_redis_cache" "redis" {
    name                = "redis1"
    location            = "East US"
    resource_group_name = "${azurerm_resource_group.production.name}"
    capacity            = 1
    family              = "C"
    sku_name            = "Standard"
    enable_non_ssl_port = false

    redis_configuration {
        maxclients = "1000"
    }

    tags {
        role = "redis"
        environment = "production"
    }
}

Debug Output

https://gist.github.com/horsey/355b4caa85cb10b3135d828fc554c63b

Panic Output

None

Expected Behavior

The tags updated inside the .tf file should reflect on the resource.

Actual Behavior

Terraform converges successfully saying that the tags were updated, but one can't see the tags on the portal.

Steps to Reproduce

1. Create a azurerm_redis_cache resource with empty tags.
2. Converge the resource successfully.
3. Update tags as shown in the snippet above.
4. terraform apply

Important Factoids

When the azurerm_redis_cache resource was converged for the first time, the resource did not have any tag field absolutely. It was not checked if the tags were added to the resource for the first time.

References

None.

[tombuildsstuff]

Hey @horsey

To provide an update here - the Swagger is wrong for the Update function and should allow Tags to be specified on the top level object, rather than within the Properties object - which is why this is not working.

Once the Swagger/SDK has been updated - we can progress with this

Thanks!

[tombuildsstuff]

Support for Tags in the azurerm_redis_cache resource has been added in #1086 which will ship as part of v1.3.3 of the AzureRM Provider which should be shipping soon; as such I'm going to close this issue for the moment.

[ghost]

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!