Skip to content

Redis sentinel sentinels k8s-service #441

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

Open
klml wants to merge 5 commits into
base: main
Choose a base branch
from
+2 −6
Open

Redis sentinel sentinels k8s-service #441

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
febf2a4
redis sentinel: remove vanilla redis k8s-service
klml Jun 17, 2026
397f837
redis sentinel: set k8s-service setinel as sentinels
klml Jun 17, 2026
71420a2
version bump 16.4.2
klml Jun 17, 2026
ed30374
Merge branch 'main' into redis-sentinel-sentinels-svc
klml Jun 24, 2026
0b04e11
versionbump: 16.4.3
klml Jun 24, 2026
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
2 changes: 1 addition & 1 deletion zammad/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2
name: zammad
version: 16.4.2
version: 16.4.3
appVersion: 7.1.0-0017
description: Zammad is a web based open source helpdesk/customer support system with many features to manage customer communication via several channels like telephone, facebook, twitter, chat and e-mails.
home: https://zammad.org
Expand Down
4 changes: 0 additions & 4 deletions zammad/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,11 +204,7 @@ Redis Variables
# sentinel
{{- if .Values.zammadConfig.redis.sentinel.enabled }}
- name: REDIS_SENTINELS
{{- if .Values.zammadConfig.redis.enabled }}
value: "{{ .Release.Name }}-redis"

@mgruner mgruner Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This cannot get dropped, instead you can just change the name that is used here to point to another service.

@mgruner mgruner Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Probably value: "{{ .Release.Name }}-redis-sentinel"

{{- else }}
value: "{{ join "," .Values.zammadConfig.redis.sentinel.sentinels }}"
{{- end }}
- name: REDIS_SENTINEL_NAME
value: "{{ .Values.zammadConfig.redis.sentinel.masterName | default "mymaster" }}"
{{- if .Values.zammadConfig.redis.sentinel.username }}
Expand Down
2 changes: 1 addition & 1 deletion zammad/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,7 @@ zammadConfig:
sentinel:
enabled: false # set to true to enable Redis Sentinel
sentinels:
- zammad-redis:26379
- zammad-redis-sentinel:26379
masterName: mymaster
# leave empty if no username is required
username:
Expand Down
Loading