k3s-io / helm-controller

Apache License 2.0
390 stars 85 forks source link

HelmChart delete leaves orphans #34

Open brandonkal opened 4 years ago

brandonkal commented 4 years ago

Earlier I had issues getting helm charts to delete. Now with a fresh install, I am experiencing the opposite issue. Too buggy for use now.

Here is the chart:

apiVersion: helm.cattle.io/v1
kind: HelmChart
metadata:
  name: openfaas
  namespace: openfaas
spec:
  chart: openfaas
  repo: 'https://openfaas.github.io/faas-netes'
  targetNamespace: openfaas
  valuesContent: |
    ---
    basic_auth: true
    faasIdler:
      dryRun: 'false'
    faasnetes:
      imagePullPolicy: IfNotPresent
      httpProbe: true
      readTimeout: 5m
      writeTimeout: 5m
    functionNamespace: openfaas-fn
    gateway:
      replicas: 2
      readTimeout: 15m
      scaleFromZero: true
      upstreamTimeout: 14m55s
      writeTimeout: 15m
    operator:
      create: true
    queueWorker:
      replicas: 2
      ackWait: 15m

Here is the log for the delete job. It exists very quickly and the job is never marked as completed.

2020-03-06T13:22:41.73077854Z + '[' helm_v3 == helm_v3 ']
2020-03-06T13:22:41.730808742Z + [[ openfaas == stable/* ]]
2020-03-06T13:22:41.730817716Z + '[' -n https://openfaas.github.io/faas-netes ']'
2020-03-06T13:22:41.730857079Z + helm_v3 repo add openfaas https://openfaas.github.io/faas-netes
2020-03-06T13:22:42.881154731Z "openfaas" has been added to your repositories
2020-03-06T13:22:42.884731895Z + helm_v3 repo update
2020-03-06T13:22:42.95460054Z Hang tight while we grab the latest from your chart repositories...
2020-03-06T13:22:43.420767524Z ...Successfully got an update from the "openfaas" chart repository
2020-03-06T13:22:43.420848117Z Update Complete. ⎈  Happy Helming!⎈
2020-03-06T13:22:43.424436598Z + helm_update delete
2020-03-06T13:22:43.424484672Z + '[' helm_v3 == helm_v3 ']'
2020-03-06T13:22:43.425375666Z ++ helm_v3 ls --all -f '^openfaas$' --output json
2020-03-06T13:22:43.425719158Z ++ jq -r '"\(.[0].app_version),\(.[0].status)"'
2020-03-06T13:22:43.425870562Z ++ tr '[:upper:]' '[:lower:]'
2020-03-06T13:22:43.557990486Z + LINE=,deployed
2020-03-06T13:22:43.558996474Z ++ echo ,deployed
2020-03-06T13:22:43.559189469Z ++ cut -f1 -d,
2020-03-06T13:22:43.56083381Z + INSTALLED_VERSION=
2020-03-06T13:22:43.561694435Z ++ echo ,deployed
2020-03-06T13:22:43.561859876Z ++ cut -f2 -d,
2020-03-06T13:22:43.563022109Z + STATUS=deployed
2020-03-06T13:22:43.563067569Z + '[' -e /config/values.yaml ']
2020-03-06T13:22:43.563080318Z + VALUES='--values /config/values.yaml'
2020-03-06T13:22:43.563095289Z + '[' delete = delete ']
2020-03-06T13:22:43.563126807Z + '[' -z '' ']
2020-03-06T13:22:43.563143487Z + exit
ibuildthecloud commented 4 years ago

Thanks for the feedback. We plan to do a major update of this project to address these reliability issues.