[BUG] azurerm_redis_cache backup requires connection string

[cemckenzie]

When creating a redis-cache resource, the rdb-storage-connection-string in the redis_configuration is required is rdb_backup_enabled is true.

Currently, an error is thrown during the create:

redis.Client#Create: Failure sending request: StatusCode=400 -- Original Error: Code="InvalidRequestBody" Message="The value of the parameter 'properties.redisConfiguration.rdb-storage-connection-string' is invalid."

This change updates the error to:

Error: Error parsing Redis Configuration: The rdb_storage_connection_string property must be set when rdb_backup_enabled is true on <PATH>/main.tf line 23

It also updates the docs.

[katbyte]

Thanks for this fix @cemckenzie ! aside from 2 comments this looks good

[katbyte]

thanks @cemckenzie! LGTM 👍

[ghost]

This has been released in version 2.9.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.9.0"
}
# ... other configuration ...

[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 👉 hashibot-feedback@hashicorp.com. Thanks!