Skip to content

Commit b6a4ca4

Browse files
authored
Merge pull request #1563 from element-hq/bbz/rename-external-redis-properties-in-values-files
Replace usage of redis in components' CI values files with redisOrValkey
2 parents 4c32b48 + b03333d commit b6a4ca4

78 files changed

Lines changed: 2826 additions & 382 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/element-docs-words.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,5 +51,6 @@ templatable
5151
threepids
5252
towncrier
5353
Traefik
54+
Valkey
5455
Wifi
5556
zizmor

charts/matrix-stack/ci/example-default-enabled-components-checkov-values.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
#
44
# SPDX-License-Identifier: AGPL-3.0-only
55

6-
# source_fragments: deployment-markers-checkov.yaml deployment-markers-minimal.yaml element-admin-checkov.yaml element-admin-minimal.yaml element-web-checkov.yaml element-web-minimal.yaml haproxy-checkov.yaml init-secrets-checkov.yaml init-secrets-minimal.yaml matrix-authentication-service-checkov.yaml matrix-authentication-service-minimal.yaml matrix-rtc-checkov.yaml matrix-rtc-minimal.yaml postgres-checkov.yaml postgres-minimal.yaml redis-checkov.yaml server-name.yaml synapse-checkov.yaml synapse-minimal.yaml well-known-minimal.yaml
6+
# source_fragments: deployment-markers-checkov.yaml deployment-markers-minimal.yaml element-admin-checkov.yaml element-admin-minimal.yaml element-web-checkov.yaml element-web-minimal.yaml haproxy-checkov.yaml init-secrets-checkov.yaml init-secrets-minimal.yaml matrix-authentication-service-checkov.yaml matrix-authentication-service-minimal.yaml matrix-rtc-checkov.yaml matrix-rtc-minimal.yaml postgres-checkov.yaml postgres-minimal.yaml server-name.yaml synapse-checkov.yaml synapse-minimal.yaml valkey-checkov.yaml well-known-minimal.yaml
77
# DO NOT EDIT DIRECTLY. Edit the fragment files to add / modify / remove values
88

99
# wellKnownDelegation don't have any required properties to be set and defaults to enabled
@@ -62,11 +62,6 @@ postgres:
6262
checkov.io/skip1: CKV_K8S_11=We deliberately don't set CPU limits. Pod is BestEffort not Guaranteed
6363
checkov.io/skip2: CKV_K8S_43=No digests
6464
checkov.io/skip3: CKV2_K8S_6=No network policy yet
65-
redis:
66-
annotations:
67-
checkov.io/skip1: CKV_K8S_11=We deliberately don't set CPU limits. Pod is BestEffort not Guaranteed
68-
checkov.io/skip2: CKV_K8S_43=No digests
69-
checkov.io/skip3: CKV2_K8S_6=No network policy yet
7065
serverName: ess.localhost
7166
synapse:
7267
annotations:
@@ -80,3 +75,8 @@ synapse:
8075
checkov.io/skip3: CKV2_K8S_6=No network policy yet
8176
ingress:
8277
host: synapse.ess.localhost
78+
valkey:
79+
annotations:
80+
checkov.io/skip1: CKV_K8S_11=We deliberately don't set CPU limits. Pod is BestEffort not Guaranteed
81+
checkov.io/skip2: CKV_K8S_43=No digests
82+
checkov.io/skip3: CKV2_K8S_6=No network policy yet

charts/matrix-stack/ci/fragments/hookshot-redis-secrets-externally.yaml renamed to charts/matrix-stack/ci/fragments/hookshot-redis-valkey-secrets-externally.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
# SPDX-License-Identifier: AGPL-3.0-only
44

55
hookshot:
6-
redis:
6+
redisOrValkey:
77
password:
8-
secret: "{{ $.Release.Name }}-redis-credentials"
8+
secret: "{{ $.Release.Name }}-redis-valkey-credentials"
99
secretKey: "password"

charts/matrix-stack/ci/fragments/hookshot-redis-secrets-in-helm.yaml renamed to charts/matrix-stack/ci/fragments/hookshot-redis-valkey-secrets-in-helm.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
# SPDX-License-Identifier: AGPL-3.0-only
44

55
hookshot:
6-
redis:
6+
redisOrValkey:
77
password:
8-
value: "test-redis-password"
8+
value: "test-redis-valkey-password"

charts/matrix-stack/ci/fragments/hookshot-redis.yaml renamed to charts/matrix-stack/ci/fragments/hookshot-redis-valkey.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
# SPDX-License-Identifier: AGPL-3.0-only
44

55
hookshot:
6-
redis:
7-
host: "external-redis.example.com"
6+
redisOrValkey:
7+
host: "external-redis-or-valkey.example.com"
88
port: 6379

charts/matrix-stack/ci/fragments/matrix-rtc-redis-secrets-externally.yaml

Lines changed: 0 additions & 11 deletions
This file was deleted.

charts/matrix-stack/ci/fragments/matrix-rtc-redis-secrets-in-helm.yaml

Lines changed: 0 additions & 10 deletions
This file was deleted.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# Copyright 2026 Element Creations Ltd
2+
#
3+
# SPDX-License-Identifier: AGPL-3.0-only
4+
5+
matrixRTC:
6+
# The authoriser doesn't yet support an external Redis / Valkey
7+
sfu:
8+
redisOrValkey:
9+
password:
10+
secret: "{{ $.Release.Name }}-redis-valkey-credentials"
11+
secretKey: "password"
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Copyright 2026 Element Creations Ltd
2+
#
3+
# SPDX-License-Identifier: AGPL-3.0-only
4+
5+
matrixRTC:
6+
# The authoriser doesn't yet support an external Redis / Valkey
7+
sfu:
8+
redisOrValkey:
9+
password:
10+
value: "test-redis-valkey-password"
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Copyright 2026 Element Creations Ltd
2+
#
3+
# SPDX-License-Identifier: AGPL-3.0-only
4+
5+
matrixRTC:
6+
# The authoriser doesn't yet support an external Redis / Valkey
7+
sfu:
8+
redisOrValkey:
9+
host: "external-redis-or-valkey.example.com"
10+
port: 6379

0 commit comments

Comments
 (0)