fix: kafka env if disabled (#43)

Author: Mokto

sentry/templates/_helper.tpl

@@ -191,18 +191,18 @@ Set Kafka Confluent host
 {{- if .Values.kafka.enabled -}}
     {{- template "sentry.kafka.fullname" . -}}
 {{- else -}}
-    "kafka-confluent"
+    kafka-confluent
 {{- end -}}
 {{- end -}}
 
 {{/*
 Set Kafka Confluent port
 */}}
 {{- define "sentry.kafka.port" -}}
-{{- if .Values.kafka.enabled -}}
-    {{- default "9092" .Values.kafka.service.port }}
+{{- if and (.Values.kafka.enabled) (.Values.kafka.service.port) -}}
+    {{- .Values.kafka.service.port }}
 {{- else -}}
-    "9092"
+    9092
 {{- end -}}
 {{- end -}}
 

sentry/templates/configmap-snuba.yaml

@@ -17,4 +17,6 @@ data:
   {{- if $redisPass }}
   REDIS_PASSWORD: "{{ $redisPass }}"
   {{- end }}
+  {{- if .Values.kafka.enabled }}
   DEFAULT_BROKERS: "{{ printf "%s:%s" (include "sentry.kafka.host" .) (include "sentry.kafka.port" .) }}"
+  {{- end }}