Skip to content

Update references in CI values file from redis to valkey for the built-in Valkey - #1561

Merged
benbz merged 1 commit into
bbz/redis-as-statefulsetfrom
bbz/redis-to-valkey-within-ci-values-files
Sep 2, 2026
Merged

Update references in CI values file from redis to valkey for the built-in Valkey#1561
benbz merged 1 commit into
bbz/redis-as-statefulsetfrom
bbz/redis-to-valkey-within-ci-values-files

Conversation

@benbz

@benbz benbz commented Sep 2, 2026

Copy link
Copy Markdown
Member

#1559 deliberately did not touch the CI value files so as to prove the merging of values from redis -> valkey worked.

Now that this has been done, we do this rename. This should be a no-op and the end of the updating of things for the built-in Valkey.

Absent is any renaming of properties for external Redis/Valkey. The files in charts/matrix-stack/ci with redis in their name relate to an external Redis/Valkey and so should not be renamed yet

@benbz
benbz requested a review from a team as a code owner September 2, 2026 10:30
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

dyff of changes in rendered templates of CI manifests

No changes in rendered templates

@benbz
benbz force-pushed the bbz/redis-to-valkey-within-ci-values-files branch from b20081b to 3879323 Compare September 2, 2026 10:34
@benbz
benbz force-pushed the bbz/redis-to-valkey-within-ci-values-files branch from 3879323 to a0c7396 Compare September 2, 2026 10:47
@gaelgatelement

Copy link
Copy Markdown
Member
 grep -rnw redis: charts/matrix-stack/ci/fragments/
charts/matrix-stack/ci/fragments/matrix-rtc-redis-secrets-in-helm.yaml:8:    redis:
charts/matrix-stack/ci/fragments/synapse-redis.yaml:6:  redis:
charts/matrix-stack/ci/fragments/matrix-rtc-redis-secrets-externally.yaml:8:    redis:
charts/matrix-stack/ci/fragments/synapse-redis-secrets-in-helm.yaml:6:  redis:
charts/matrix-stack/ci/fragments/hookshot-redis-secrets-externally.yaml:6:  redis:
charts/matrix-stack/ci/fragments/matrix-rtc-redis.yaml:8:    redis:
charts/matrix-stack/ci/fragments/synapse-redis-secrets-externally.yaml:6:  redis:
charts/matrix-stack/ci/fragments/hookshot-redis.yaml:6:  redis:
charts/matrix-stack/ci/fragments/hookshot-redis-secrets-in-helm.yaml:6:  redis:

Is that on purpose?

@benbz

benbz commented Sep 2, 2026

Copy link
Copy Markdown
Member Author
 grep -rnw redis: charts/matrix-stack/ci/fragments/
charts/matrix-stack/ci/fragments/matrix-rtc-redis-secrets-in-helm.yaml:8:    redis:
charts/matrix-stack/ci/fragments/synapse-redis.yaml:6:  redis:
charts/matrix-stack/ci/fragments/matrix-rtc-redis-secrets-externally.yaml:8:    redis:
charts/matrix-stack/ci/fragments/synapse-redis-secrets-in-helm.yaml:6:  redis:
charts/matrix-stack/ci/fragments/hookshot-redis-secrets-externally.yaml:6:  redis:
charts/matrix-stack/ci/fragments/matrix-rtc-redis.yaml:8:    redis:
charts/matrix-stack/ci/fragments/synapse-redis-secrets-externally.yaml:6:  redis:
charts/matrix-stack/ci/fragments/hookshot-redis.yaml:6:  redis:
charts/matrix-stack/ci/fragments/hookshot-redis-secrets-in-helm.yaml:6:  redis:

Is that on purpose?

Yes as per the PR body:

Absent is any renaming of properties for external Redis/Valkey. The files in charts/matrix-stack/ci with redis in their name relate to an external Redis/Valkey and so should not be renamed yet

@benbz
benbz merged commit b6a4ca4 into main Sep 2, 2026
263 of 268 checks passed
@benbz
benbz deleted the bbz/redis-to-valkey-within-ci-values-files branch September 2, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants