Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃獤 feat: Helm Chart #2339

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

iamNoah1
Copy link
Contributor

@iamNoah1 iamNoah1 commented Apr 6, 2024

Pull Request Template

Summary

Add HelmChart to deploy Librechat on Kubernetes.

Change Type

Please delete any irrelevant options.

  • New feature/chore (non-breaking change which adds functionality)
  • This change requires a documentation update
  • Documentation update

Testing

Everything I tested is pretty much described in the documentation section.

Test Configuration:

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have made pertinent documentation changes
  • New documents have been locally validated with mkdocs

Signed-off-by: Noah Ispas (iamNoah1) <[email protected]>
@danny-avila
Copy link
Owner

danny-avila commented Apr 8, 2024

Thank @iamNoah1

When I get the chance I will try following your instructions and merge if I can successfully install it.

Also @idachev may want to take a look as he has a chart that works and was thinking of adding it too.

EDIT: just waking up, also remembered there is already an open PR: #2276

@iamNoah1
Copy link
Contributor Author

@danny-avila @idachev what do you suggest on how to proceed?

@bnsd55
Copy link

bnsd55 commented Apr 16, 2024

Is there a reason for not creating helm for both MongoDB and Redis?

@iamNoah1
Copy link
Contributor Author

@bnsd55 I wanna to keep it as simple as possible. For dev reasons having mongodb and redis also included as chart could come in handy, but in my use case I would want to have them externalized.

@bnsd55
Copy link

bnsd55 commented Apr 17, 2024

@iamNoah1 okay, I tried to run it and got error:

`

[email protected] backend
cross-env NODE_ENV=production node api/server/index.js
node:buffer:325
throw new ERR_INVALID_ARG_TYPE(
^
TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received undefined
at new NodeError (node:internal/errors:405:5)
at Function.from (node:buffer:325:9)
at Object. (/app/api/server/utils/crypto.js:4:20)
at Module._compile (node:internal/modules/cjs/loader:1364:14)
at Module._extensions..js (node:internal/modules/cjs/loader:1422:10)
at Module.load (node:internal/modules/cjs/loader:1203:32)
at Module._load (node:internal/modules/cjs/loader:1019:12)
at Module.require (node:internal/modules/cjs/loader:1231:19)
at require (node:internal/modules/helpers:177:18)
at Object. (/app/api/server/utils/index.js:5:21) {
code: 'ERR_INVALID_ARG_TYPE'
}
Node.js v18.20.0
npm notice
npm notice New patch version of npm available! 10.5.0 -> 10.5.2
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.5.2
npm notice Run npm install -g [email protected] to update!
npm notice `

any idea why?

@iamNoah1
Copy link
Contributor Author

@bnsd55 have a look at this: #651

Seems you haven't configured it correctly.

@iamNoah1
Copy link
Contributor Author

@danny-avila can we maybe follow up on this? native support of helm chart would be very nice :) and @idachev did not respond to my question about how they think we should proceed with their PR which is on draft by the way. My suggestion is to go with my PR and adjust as needed.

@danny-avila
Copy link
Owner

Yes, there鈥檚 a couple helm charts in open PRs. I don鈥檛 have much experience with them otherwise I鈥檇 compare, or I can get to studying this when I can

@danny-avila
Copy link
Owner

@danny-avila can we maybe follow up on this? native support of helm chart would be very nice :) and @idachev did not respond to my question about how they think we should proceed with their PR which is on draft by the way. My suggestion is to go with my PR and adjust as needed.

actually I re-read your comment and I agree with you. we can start off with this one and adjust

@danny-avila danny-avila changed the title chore: adding helm chart for librechat 馃獤 feat: Helm Chart Jun 21, 2024
@danny-avila
Copy link
Owner

I was thinking before merging this, is it better to have it on a separate repo in case others have helm charts on the repo already?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants