Skip to content

feat(kvstore): add storage_type support for Redis 7.0 cloud-disk inst…#9577

Open
cxx-12333 wants to merge 2 commits into
aliyun:masterfrom
cxx-12333:feat/kvstore-add-storage-type
Open

feat(kvstore): add storage_type support for Redis 7.0 cloud-disk inst…#9577
cxx-12333 wants to merge 2 commits into
aliyun:masterfrom
cxx-12333:feat/kvstore-add-storage-type

Conversation

@cxx-12333
Copy link
Copy Markdown

Summary

Add storage_type field to alicloud_kvstore_instance resource to support
Redis 7.0 cloud-disk instances.

Problem

Redis 7.0 requires cloud-disk instance classes and the StorageType parameter
must be passed to the CreateInstance API. Without this field, users get:
EngineVersion.NotSupportOnLocalDisk

Changes

  • Schema: add storage_type (Optional, ForceNew, Computed)
    Valid values: cloud_essd, cloud_essd_pl1, cloud_essd_pl2, cloud_essd_pl3
  • Create: pass StorageType to CreateInstance API
  • Read: populate storage_type from DescribeInstances response

Usage

resource "alicloud_kvstore_instance" "redis_7" {
  instance_type  = "Redis"
  engine_version = "7.0"
  instance_class = "redis.master.small.cloud.default"
  storage_type   = "cloud_essd"
  ...
}

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 16, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@cxx-12333 cxx-12333 force-pushed the feat/kvstore-add-storage-type branch from 3fa1955 to 7b41289 Compare March 16, 2026 13:53
@xuzhang3
Copy link
Copy Markdown
Collaborator

xuzhang3 commented Mar 18, 2026

@cxx-12333 Thank you for submit this PR. Current KV store resource is using the API:https://next.api.aliyun.com/api/R-kvstore/2015-01-01/CreateInstance while StorageType is being supported in https://next.api.aliyun.com/api/R-kvstore/2015-01-01/CreateTairInstance, there are two different API.
And also ensure the AccTest can pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(kvstore): add storage_type support to alicloud_kvstore_instance for Redis 7.0 cloud-disk instances

3 participants