Skip to content

Commit 7fa1f42

Browse files
committed
fix(sentry/sentry/helper): Redis URL not set when using externalRedis.existingSecret or redis.auth.existingSecret
1 parent 480a2a3 commit 7fa1f42

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

charts/sentry/templates/sentry/_helper-sentry.tpl

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,9 @@ sentry.conf.py: |-
208208
BROKER_URL = os.environ.get("BROKER_URL", "amqp://{{ .Values.rabbitmq.auth.username }}:{{ .Values.rabbitmq.auth.password }}@{{ template "sentry.rabbitmq.host" . }}:5672/{{ .Values.rabbitmq.vhost }}")
209209
{{- else if $redisPass }}
210210
BROKER_URL = os.environ.get("BROKER_URL", "{{ $redisProto }}://:{{ $redisPass }}@{{ $redisHost }}:{{ $redisPort }}/{{ $redisDb }}")
211-
{{- else if and (not .Values.externalRedis.existingSecret) (not .Values.redis.auth.existingSecret)}}
211+
{{- else if or (.Values.externalRedis.existingSecret) (.Values.redis.auth.existingSecret) }}
212+
BROKER_URL = os.environ.get("BROKER_URL", f"{{ $redisProto }}://:{os.environ.get("REDIS_PASSWORD")}@{{ $redisHost }}:{{ $redisPort }}/{{ $redisDb }}")
213+
{{- else }}
212214
BROKER_URL = os.environ.get("BROKER_URL", "{{ $redisProto }}://{{ $redisHost }}:{{ $redisPort }}/{{ $redisDb }}")
213215
{{- end }}
214216

@@ -447,7 +449,7 @@ sentry.conf.py: |-
447449
"organizations:starfish-mobile-appstart",
448450
"projects:span-metrics-extraction",
449451
"projects:span-metrics-extraction-addons",
450-
452+
451453
# flags added in this chart
452454
"organizations:trace-view-load-more",
453455
"organizations:trace-tabs-ui",
@@ -460,7 +462,7 @@ sentry.conf.py: |-
460462
+ (
461463
# Session Replay related flags
462464
"organizations:session-replay",
463-
465+
464466
# flags added in this chart
465467
"organizations:session-replay-ui",
466468
"organizations:session-replay-issue-emails",

0 commit comments

Comments
 (0)