qdrant-helm icon indicating copy to clipboard operation
qdrant-helm copied to clipboard

fix(chart): add service.enable_tls to values

Open knkarthik opened this issue 2 months ago • 2 comments

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.

knkarthik avatar Sep 30 '25 21:09 knkarthik

@bashofmann Can you take a look at this when you get a chance. Thanks!

knkarthik avatar Sep 30 '25 21:09 knkarthik

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.

bashofmann avatar Nov 24 '25 18:11 bashofmann