bakdata / streams-bootstrap

Utility functions and base classes for Kafka Streams applications
MIT License
29 stars 3 forks source link

Investigate the necessity of `nameOverride` #185

Closed disrupted closed 5 months ago

disrupted commented 1 year ago

Having to set the nameOverride for every deployment seems like a workaround. We should look into setting the correct Helm release name automatically without overriding.

raminqaf commented 1 year ago

Let's step back and see the differences between nameOverride and fullnameOverride. By default, whenever you create a Helm chart, the template file defines nameOverride and fullnameOverride like this:

{{- define "<CHARTNAME>.fullname" -}}
{{- if .Values.fullnameOverride }}
{{- .Values.fullnameOverride | trunc 63 | trimSuffix "-" }}
{{- else }}
{{- $name := default .Chart.Name .Values.nameOverride }}
{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" }}
{{- end }}
{{- end }}

The template above can be broken down as follows:

However, in streams bootstrap, we have altered this code in this commit. Now the behavior of nameOverride and fullnameOverride is the same. Maybe we should consider changing this parameter to kafka-app-name, which is not Helm-specific or keep either nameOverride or fullnameOverride.

philipp94831 commented 5 months ago

Can this be closed? See discussion on #193

raminqaf commented 5 months ago

Closing this

Can this be closed? See discussion on #193