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

Create helm-releases instead of helm template | kubectl apply -f - #153

Open
mxmxchere opened this issue May 16, 2024 · 1 comment
Open
Labels
Container Issues or pull requests relevant for Team 2: Container Infra and Tooling

Comments

@mxmxchere
Copy link

Helm should be used to create helm-releases. That is the way how helm is supposed to be used. The documentation of helm refers to the scenario where helm is used that way. helm template is useful for debugging but it is not supposed to be used to create the manifests that are then applied to the cluster. helm template lacks important features like capabilities, notes, upgrade and rollback, to just name a few.

In the worst case chart A version 1 deploys a configmap and in version 2 the configmap is changed to a secret. In the current approach the configmap will be stuck in the cluster forever. With helm upgrade, that configmap would be removed.

@janiskemper
Copy link
Member

we have that logic you mentioned in the example implemented in the cso. @batistein can maybe say in detail why we chose to not use helm install. I know that there are some shortcomings.

What do you miss as feature in detail, from user perspective? Or do you rather suggest a different way of implementing it, without any user-facing change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Container Issues or pull requests relevant for Team 2: Container Infra and Tooling
Projects
Status: Backlog
Development

No branches or pull requests

3 participants