Skip to content
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.

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

Investigate the necessity of nameOverride #185

Closed
disrupted opened this issue Aug 30, 2023 · 3 comments
Closed

Investigate the necessity of nameOverride #185

disrupted opened this issue Aug 30, 2023 · 3 comments

Comments

@disrupted
Copy link
Member

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
Copy link
Contributor

raminqaf commented Nov 20, 2023

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:

  • If no nameOverride and fullNameOverride is set -> Deployment name = release-name-<chart-name
  • if nameOverride is set -> Deployment name = release-name-<name-override>
  • if fullnameOverride is set -> Deployment name = <full-name-override>

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
Copy link
Member

Can this be closed? See discussion on #193

@raminqaf
Copy link
Contributor

Closing this

Can this be closed? See discussion on #193

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants