wdaan/hoppscotch-helm

Helm Chart for hoppscotch.io

hoppscotch-helm's Issues

Many issues

Hello, thank you for your work but I can't install your chart, I have several issues.
If I install it with helm install hoppscotch hoppscotch/hoppscotch -f values.yaml the global.env are not taken:

coalesce.go:223 : warning : destination for hoppscotch.admin.env is a table. Ignore values that are not tables ([])
coalesce.go:223 : warning: destination for hoppscotch.backend.env is a table. Ignore non-table value ([])
coalesce.go:223 : warning : destination for hoppscotch.web.env is a table. Ignore non-table value ([])

I also have a issue with ingress:
Error: template: hoppscotch/templates/web/ingress.yaml:3:23: executing "hoppscotch/templates/web/ingress.yaml" at <.Values.service.httpPort>: nil pointer evaluating interface {}.httpPort"

I cloned your repo, and the global.env works, but the containers are in crashloopbackoff
standard_init_linux.go:228: exec user process caused: operation not permitted and same issue with ingress.

I doesn't understand why, thanks for your help.

Expose the chart on Artifactub.io

Hello,

Thank you for this (nearly) up to date chart
To facilitate the usage of your great chart a good idea would be to put it on the site https://artifacthub.io/

indentation for backend-envs

Indentation should be 12 here:

hoppscotch-helm/charts/hoppscotch/templates/backend/deployment.yaml

Line 45 in e5ca496

{{- toYaml . | nindent 10 }}

Provide a postgresql by default

Hi thank for you work it could interersting to have by default a postgresql database like in the docker-compose what do you think ?

improvement: add init container which runs the database migration

The database migrations aren't applied. Maybe we could add a init container which applies the migration before booting the backend container.

What do you think?

hoppscotch.env_vars - cannot handle strings

It seems as the hoppscotch.env_vars helper function doesn't support env values containing string.

I guess you could remove the if-statement altogether, but I might miss something.

{{/* Format env vars */}}
{{- define "hoppscotch.env_vars" -}}
  {{- with . -}}
    {{- $result := list -}}
    {{- range $name, $value := . -}}
        {{- if or (kindIs "float64" $value) (kindIs "bool" $value) -}}
          {{- $result = append $result (dict "name" $name "value" ($value | toString)) -}}
        {{- else -}}
          {{- $result = append $result (dict "name" $name "value" $value) -}}
        {{- end -}}
      {{- end -}}
    {{- toYaml $result | nindent 0 -}}
  {{- end -}}
{{- end -}}

global:
  env:
    DATABASE_URL: postgresql://postgres:testpass@hoppscotch-db:5432/hoppscotch

Error details

Error: INSTALLATION FAILED: template: hoppscotch/templates/web/deployment.yaml:44:17: executing "hoppscotch/templates/web/deployment.yaml" at <include "hoppscotch.env_vars" (deepCopy .Values.web.env | merge .Values.global.env)>: error calling include: template: hoppscotch/templates/_helpers.tpl:104:67: executing "hoppscotch.env_vars" at <tpl $value $>: error calling tpl: cannot retrieve Template.Basepath from values inside tpl function: postgresql://postgres:[e-mail]:5432/hoppscotch: "BasePath" is not a value