azurerm_redis_cache - review optional and computed

[catriona-m]

Community Note

• Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
• Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

This PR removes instances of optional + computed in azurerm_redis_cache or adds a note explaining the usage.

It also deprecates enable_non_ssl_port in favour of non_ssl_port_enabled and deprecates redis_configuration. enable_authentication in favour of redis_configuration. authentication_enabled

Testing

• My submission includes Test coverage as described in the Contribution Guide and the tests pass. (if this is not possible for any reason, please include details of why you did or could not add test coverage)

Tests running in TC

[jackofallops]

Thanks @catriona-m - Just one comment to take a look at, but otherwise LGTM 👍

[jackofallops]

can we wrap the schema in the beta flag here to make it easier to find and remove later?

[catriona-m]

Thanks @jackofallops - I spotted another TODO that I had missed, if you wouldn't mind having another quick look!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.