qdrant-helm
qdrant-helm copied to clipboard
fix(chart): add service.enable_tls to values
When trying to instal this chart with ArgoCD I was getting the following error:
Error: template: qdrant/charts/qdrant/templates/tests/test-db-interaction.yaml:84:48: executing "qdrant/charts/qdrant/templates/tests/test-db-interaction.yaml" at <$root.Values.config.service.enable_tls>: nil pointer evaluating interface {}.enable_tls
The test-db-interaction test was failing because it was checking for config.service.enable_tls which did not exist in the values.yaml file.
This commit adds the service.enable_tls key to the config section of the values.yaml file and sets its default value to false. This allows the test to correctly determine whether to use http or https when connecting to the qdrant service.
@bashofmann Can you take a look at this when you get a chance. Thanks!
Which version of ArgoCD/Helm are you using? I can't reproduce this with plain Helm. It is correctly not evaluating the second part of the if
{{- if and $root.Values.config.service $root.Values.config.service.enable_tls }}
And both helm install and helm template work correctly.