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

🧨 install Chart is NOT idempotent 🧨 #69

Open
eformat opened this issue Dec 14, 2021 · 1 comment
Open

🧨 install Chart is NOT idempotent 🧨 #69

eformat opened this issue Dec 14, 2021 · 1 comment

Comments

@eformat
Copy link
Member

eformat commented Dec 14, 2021

you can install, uninstall this chart.

but if you have a failed install, then try to reinstall i.e. run this multiple times:

helm upgrade --install do500 . --namespace do500 --create-namespace --timeout=15m --set group_name=lodestar-developers

then gitlab deployments DOES NOT redeploy properly, making a right royal mess. you have to delete do500-gitlab namespace, let that install from fresh.

also, the stackrox and crw operators finalizers are still dodgy ... causing you to have to patch finalizers and remove objects when trying to cleanly uninstall.

@burigolucas
Copy link

burigolucas commented Nov 12, 2024

I experienced this issue with the reinstall after a failure to deploy gitlab. The cause of the problem is that the chart generates new passwords, see

{{- define "gitlab.root_password" -}}
{{- print (randAlphaNum 10) -}}
{{- end -}}
{{- define "gitlab.postgres.user" -}}
{{- print (randAlphaNum 10) -}}
{{- end -}}
{{- define "gitlab.postgres.password" -}}
{{- print (randAlphaNum 10) -}}
{{- end -}}
{{- define "gitlab.postgres.admin_password" -}}
{{- print (randAlphaNum 10) -}}
{{- end -}}

One way to solve this is to store the passwords in a secret and use a lookup to retrieve the passwords when reinstalling (upgrading) the chart.

The PR #172 implements this approach.

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

No branches or pull requests

2 participants