zalf-rdm / geonode-k8s

A Kubernetes helm chart for the geospacial webapplication Geonode
https://geonode-k8s.readthedocs.io/en/latest/
GNU General Public License v2.0
12 stars 7 forks source link

Bug: disabling memcached still use memcached chart #117

Closed AlexGacon closed 7 months ago

AlexGacon commented 7 months ago

Bug Description

When trying to disable memcached by setting enabled to false, helm still evaluates the memcached charts which can raise incompatibilities in some case.

Reproduction Steps

Set the memcached enabled value to false and try to deploy in a chart geonode-k8s and the postgresql stack from bitnami. An error appears in memcached template, whereas it shouldn't be evaluated.

Behavior

If enabled is set to false, the chart should not be used at all.

Additional Information

Error due to a typo in condition: geonode.memcached.enaled. Should be condition: geonode.memcached.enabled