tryretool / retool-helm

MIT License
45 stars 57 forks source link

support helm 3.7 #134

Closed avimoondra closed 8 months ago

avimoondra commented 8 months ago

Tested the same commands here: https://github.com/tryretool/retool-helm/pull/131 with helm 3.7

Repro of issues

# Download Helm v3.7.1 for Mac (amd64)
curl -LO https://get.helm.sh/helm-v3.7.1-darwin-amd64.tar.gz

# Extract the tarball
tar -zxvf helm-v3.7.1-darwin-amd64.tar.gz

# Move the helm binary to a directory in your PATH and set it as "helm-3-7"
sudo mv darwin-amd64/helm /usr/local/bin/helm-3-7

# Verify the installation
helm-3-7 version
# version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.16.9"}

helm-3-7 upgrade --install -f ./onprem-instances/values.base.yaml -f ./onprem-instances/values.joe-test-new.retool.dev.yaml joe-test-new retool/retool --version 6.0.9
Error: UPGRADE FAILED: template: retool/templates/deployment_jobs.yaml:1:7: executing "retool/templates/deployment_jobs.yaml" at <include "retool.jobRunner.enabled" .>: error calling include: template: retool/templates/_helpers.tpl:132:58: executing "retool.jobRunner.enabled" at <eq .Values.jobRunner.enabled true>: error calling eq: incompatible types for comparison

Tested as such:

helm-3-7 template -f ~/retool-helm/charts/retool/values.yaml foo ~/retool-helm/charts/retool --set config.encryptionKey="foo" --set image.tag="5.6.10" --set replicaCount=2  --debug | rg replicas -C 7

install.go:178: [debug] Original chart version: ""
install.go:199: [debug] CHART PATH: /Users/avimoondra/retool-helm/charts/retool

  name: foo-retool
  labels:
    helm.sh/chart: retool-6.0.9
    app.kubernetes.io/name: retool
    app.kubernetes.io/instance: foo
    app.kubernetes.io/managed-by: Helm
spec:
  replicas: 2
  selector:
    matchLabels:
      app.kubernetes.io/name: retool
      app.kubernetes.io/instance: foo
  revisionHistoryLimit: 3
  template:
    metadata:
--
  name: foo-retool-jobs-runner
  labels:
    helm.sh/chart: retool-6.0.9
    app.kubernetes.io/name: retool
    app.kubernetes.io/instance: foo
    app.kubernetes.io/managed-by: Helm