bug: Redis URL is not defined for built-in redis chart

mvisonneau / helm-charts

Personal Helm charts repository

Apache License 2.0

65 stars 58 forks source link

Closed lukasmrtvy closed 3 years ago

lukasmrtvy commented 3 years ago

Redis URL is not defined for built-in redis chart

Helm output:

<.Values.config.redis.url>: nil pointer evaluating interface {}.url

Thanks

mvisonneau commented 3 years ago

👋 hey @lukasmrtvy, thanks for raising this. It should be fixed with v0.2.1 👍

lukasmrtvy commented 3 years ago

@mvisonneau thanks 👍 any eta for publishing 0.2.1 chart? thanks

mvisonneau commented 3 years ago

looks like there was an issue with the CI, should be released now :+1:

lukasmrtvy commented 3 years ago

oh, nice. btw there is another bug, default value for https://github.com/mvisonneau/helm-charts/blob/main/charts/gitlab-ci-pipelines-exporter/templates/secret.yaml#L20 is not defined, fails to nil pointer evaluating interface

mvisonneau commented 3 years ago

indeed :facepalm:, just pushed a fix with 0.2.2 :bow:!

lukasmrtvy commented 3 years ago

one last thing. redis chart changed some options in values.yaml, these ones are correct for 14.6.1 chart:

redis:
  auth:
    enabled: false
  architecture: standalone

mvisonneau commented 3 years ago

ah cool, thanks for the info!