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

ArgoCD doesn't support helm k8s lookup and thus qdrant existing secret lookup fails

Open pschwager opened this issue 1 year ago • 3 comments

I just tried your latest changes with a ArgoCD deployment, by specifing a apiKeys via an existing secret like

apiKey:
  valueFrom:
     secretKeyRef:
       name: "secretname"
       key: "apikeyname"

This doesn't work, as ArgoCD doesn't support the helm lookup function: https://github.com/argoproj/argo-cd/issues/5202

During debugging the issue I found out, that the qdrant helm chart tries to read the existing k8s secret and creates a new secret.

I'm looking for some alternative solution to specify the api keys outside the qdrant helm chart, so that I don't have to put the secerts as cleartext into our deployment repos... any suggestion?

pschwager avatar Sep 30 '24 12:09 pschwager

Also hitting this. I think, instead of having the the template lookup the secret and copy it to a new secret, the secret could just be mounted. It needs to reside in the same namespace anyway.

justbert avatar Oct 24 '24 17:10 justbert

Also facing this issue. ~I still haven't tested it but~ my current workaround is to pass these values to the environment variables of the StatefulSet.

  env:
    - name: QDRANT__SERVICE__API_KEY
      valueFrom:
        secretKeyRef:
          name: "secretname"
          key: "apikeyname"

joseprsm avatar Nov 27 '24 13:11 joseprsm

+1 to this, thank you @joseprsm for the workaround fix!

cdxker avatar Jul 16 '25 17:07 cdxker