feat: update ClickHouse templates to use headless service name and fix distributed cluster name formatting

Author: TartanLeGrand

charts/sentry/templates/_helper.tpl

@@ -333,7 +333,7 @@ Set ClickHouse host
 */}}
 {{- define "sentry.clickhouse.host" -}}
 {{- if .Values.clickhouse.enabled -}}
-{{- template "sentry.clickhouse.fullname" . -}}
+{{- include "clickhouse.headlessServiceName" $.Subcharts.clickhouse -}}
 {{- else -}}
 {{ required "A valid .Values.externalClickhouse.host is required" .Values.externalClickhouse.host }}
 {{- end -}}

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

@@ -81,7 +81,7 @@ settings.py: |
       {{- end }}
       {{- if or .Values.clickhouse.enabled (not .Values.externalClickhouse.singleNode) }}
       "cluster_name": {{ default "default" .Values.clickhouse.clusterName | quote }},
-      "distributed_cluster_name": {{ default "default" .Values.clickhouse.clusterName | printf "%s-clickhouse" }},
+      "distributed_cluster_name": {{ default "default" .Values.clickhouse.clusterName | quote }},
       {{- end }}
     },
   ]