search

fluxcd / helm-operator

Successor: https://github.com/fluxcd/helm-controller — The Flux Helm Operator, once upon a time a solution for declarative Helming.
https://docs.fluxcd.io/projects/helm-operator/
Apache License 2.0
649 stars 262 forks source link

Upgrade helm lib to fix null value overrides #602

Closed stevo-f3 closed 3 years ago

stevo-f3 commented 3 years ago

Describe the bug

A clear and concise description of what the bug is.

To Reproduce

Steps to reproduce the behaviour:

  1. Provide the Helm Operator install instructions
  2. Provide a HelmRelease example
  3. Post the HelmRelease status, you can get this by running kubectl describe helmrelease <name>

Expected behavior

A clear and concise description of what you expected to happen.

helm-operator uses outdated helm with a bug https://github.com/fluxcd/helm-operator/blob/master/go.mod#L32 The bug was fixed in helm https://github.com/helm/helm/pull/7743

Logs

If applicable, please provide logs. In a standard stand-alone installation, you'd get this by running kubectl logs deploy/flux-helm-operator -n fluxcd.

Additional context

stefanprodan commented 3 years ago

We can't update the helm packages as they come with breaking changes due to Kubernetes client-go. Please consider moving to Flux v2 as helm-operator is deprecated and it will be archived soon, see https://fluxcd.io/docs/migration/helm-operator-migration/