r/redis_cache: Add support for Redis 6

[andrei-dascalu]

Community Note

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

Currently in Azure there is the possibility of choosing either Redis 4 or Redis 6 (preview). As of now azurerm_redis_cache doesn't seem to have an option to make a choice.

New or Affected Resource(s)

azurerm_redis_cache
azurerm_redis_linked_server (presumably?)

Potential Terraform Configuration

Not sure about where this could go, whether this would be a redis_configuration item or a simple version key

Thanks!

[maxkuzkin]

I would suggest changing the title of this issue to "Add support for Redis 6", because, practically speaking, it's not possible to use Redis 6 in Azure through Terraform without this feature today.

And even though Redis 6 Preview has been announced by Microsoft back in November (https://azure.microsoft.com/en-us/updates/redis-60-is-now-in-preview-for-azure-cache-for-redis/), all of us who automate our deployments (which is the target audience of services like Redis, most likely) are currently not able to use Redis 6 in our scenarios.

Combined with the fact that there is no Redis 5 support in Azure, this is an issue for usu which needs to be fixed with high priority, because we're stuck with Redis 4 only.

[tombuildsstuff]

👋

Taking a look through here it appears there's a couple of different things to call out here:

1. Support for Azure Redis Cache running v6 of Redis appears to be possible by setting "redisVersion": "6" within the properties block - however support for this isn't available in the Azure SDK at this point (only as a read-only value). As such this'd need Microsoft to support this within the Azure SDK for us to be able to support this in Terraform - @WodansSon would you mind running that down?
2. Support for Azure Redis Enterprise Cache - which does include support for Redis 6 in the SDK - however this looks to be an entirely separate service

Whilst this issue is tracking the former - and so at this point is blocked on the Azure SDK adding support for this - if you're looking for support for Redis Enterprise, please open a separate Feature Request so that we can track that specifically.

Thanks!

[NillsF]

@tombuildsstuff @WodansSon: I believe the issue is not (only) an SDK issue, but a raw API issue for Redis. Based on what I'm seeing in the API documentation, the redisVersion property is not available as a property in the body on the create request and only is available on the returned object.

[steph-moto]

Has there been any updates on this. I truly believe it would be beneficial to have this supported from terraform

[tomatwoodliquidx]

Any status update on this item?

[Poil]

Hi,

The SDK have been updated to support API 2020-1201.
We just need a patch to support the "version" parameter

Best regards

[Poil]

#12820

[tombuildsstuff]

Fixed via #12820

[github-actions]

This functionality has been released in v2.71.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

[github-actions]

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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.