Skip to content

Since we can use valkey or redis clusters, should the key be simply clusters instead of redis_clusters #83

Description

@nitrocode

Have a question? Please checkout our Slack Community or visit our Slack Archive.

Slack Community

Describe the Feature

The input is redis_clusters but since the engine_type can be either valkey or redis, then the input should simply be called clusters.

This can be done as a breaking change or a phased approach where var.redis_clusters is added to variables-deprecated.tf and var.clusters is added to variables.tf. Then the next major version can remove var.redis_clusters alongside any other deprecated vars.

Similar issue could plague the memcache sibling component.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions