Skip to content

feat: Add defaults and levels for certain fields of Redis Cluster. #318

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

feat: Add defaults and levels for certain fields of Redis Cluster. #318

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
598e493
add altDefaults and level
Jul 22, 2025
07e1020
update altdefaults and levels
Jul 23, 2025
bc0eb0c
update altdefaults
Jul 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 41 additions & 1 deletion modules/redis-cluster/metadata.display.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,25 @@ spec:
authorization_mode:
name: authorization_mode
title: Authorization Mode
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value: AUTH_MODE_IAM_AUTH
cluster_role:
name: cluster_role
title: Cluster Role
deletion_protection_enabled:
name: deletion_protection_enabled
title: Deletion Protection Enabled
altDefaults:
- type: ALTERNATE_TYPE_DC
value: true
enable_apis:
name: enable_apis
title: Enable Apis
altDefaults:
- type: ALTERNATE_TYPE_DC
value: true
kms_key:
name: kms_key
title: Kms Key
Expand All @@ -49,12 +59,25 @@ spec:
network:
name: network
title: Network
altDefaults:
- type: ALTERNATE_TYPE_DC
value: projects/${project_id}/global/networks/default
Copy link
Collaborator

@q2w q2w Jul 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

value should not have any variable? What if there is no default set for this? Or simply default won't work?

node_type:
name: node_type
title: Node Type
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value: REDIS_HIGHMEM_MEDIUM
persistence_config:
name: persistence_config
title: Persistence Config
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value:
mode: AOF
append_fsync: EVERYSEC
primary_cluster:
name: primary_cluster
title: Primary Cluster
Expand All @@ -73,6 +96,10 @@ spec:
replica_count:
name: replica_count
title: Replica Count
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value: 1
secondary_clusters:
name: secondary_clusters
title: Secondary Clusters
Expand All @@ -82,15 +109,28 @@ spec:
shard_count:
name: shard_count
title: Shard Count
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value: 10
transit_encryption_mode:
name: transit_encryption_mode
title: Transit Encryption Mode
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value: TRANSIT_ENCRYPTION_MODE_DISABLED
weekly_maintenance_window:
name: weekly_maintenance_window
title: Weekly Maintenance Window
title: Maintenance Window
level: 1
zone_distribution_config_mode:
name: zone_distribution_config_mode
title: Zone Distribution Config Mode
level: 1
altDefaults:
- type: ALTERNATE_TYPE_DC
value: MULTI_ZONE
zone_distribution_config_zone:
name: zone_distribution_config_zone
title: Zone Distribution Config Zone