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.85k stars 1.11k forks source link

Fix Helm chart run failures due to low resources #1536

Closed artmatsak closed 1 month ago

artmatsak commented 1 month ago

Contrary to the comments in values.yaml claiming that specifying default resources is left out to the user, resources are being specified for some deployments. The resources for background, for example, are too low and cause process restarts in the container.

On the other hand, no resources are specified for vespa, and the Vespa Helm chart specifies resources that are too low.

This PR does the following:

  1. Comment out resource specifications for all deployments except vespa and fix code style of the commented-out blocks
  2. Override resource specifications for vespa to allow Vespa to run.
vercel[bot] commented 1 month ago

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

A member of the Team first needs to authorize it.