danswer-ai / danswer

Gen-AI Chat for Teams - Think ChatGPT if it had access to your team's unique knowledge.
https://docs.danswer.dev/
Other
9.73k stars 1.08k forks source link

Remove unused Helm chart config #1726

Open sd109 opened 3 days ago

sd109 commented 3 days ago

These config options are no longer used in the Helm chart since https://github.com/danswer-ai/danswer/pull/1552 so removing them will reduce confusion for end users (e.g. I just tried to modify the Vespa volume size on my own deployment using these values and then discovered that they have no effect).

vercel[bot] commented 3 days ago

Someone is attempting to deploy a commit to the Danswer Team on Vercel.

A member of the Team first needs to authorize it.

sd109 commented 3 days ago

In case it's useful for anyone else, the correct way to adjust the Vespa volume size in the current chart version is to pass the config through to the correct place in the underlying Vespa chart with

vespa:
  volumeClaimTemplates:
    - metadata:
        name: vespa-storage
      spec:
        accessModes:
          - ReadWriteOnce
        resources:
          requests:
            storage: <chosen-volume-size>Gi