search

InftyAI / llmaz

☸️ Easy, advanced inference platform for large language models on Kubernetes. 🌟 Star to support our work!
Apache License 2.0
30 stars 10 forks source link

feat:support apply llmaz to any ns #172

Closed qinguoyi closed 1 month ago

qinguoyi commented 1 month ago

What this PR does / why we need it

Support install llmaz at any namespace

Which issue(s) this PR fixes

Fixes #141

Special notes for your reviewer

i update the cert ns from hardcode to chart configuration, but there also has many "llamz-system" hardcode, such as : https://github.com/InftyAI/llmaz/blob/4c35e017d7f78600a5a5624571850374fd9a4b46/Makefile#L300-L303 https://github.com/InftyAI/llmaz/blob/4c35e017d7f78600a5a5624571850374fd9a4b46/docs/installation.md?plain=1#L10-L16 https://github.com/InftyAI/llmaz/blob/4c35e017d7f78600a5a5624571850374fd9a4b46/docs/installation.md?plain=1#L44-L49 https://github.com/InftyAI/llmaz/blob/4c35e017d7f78600a5a5624571850374fd9a4b46/config/default/kustomization.yaml#L1-L3 https://github.com/InftyAI/llmaz/blob/4c35e017d7f78600a5a5624571850374fd9a4b46/chart/crds/openmodel-crd.yaml#L7-L17

what can i do for these?

Does this PR introduce a user-facing change?

NONE

kerthcet commented 1 month ago

Thanks @qinguoyi for the help, let me take a try first.

qinguoyi commented 1 month ago

as suggested, i re-push the code. @kerthcet

kerthcet commented 1 month ago

Thanks @qinguoyi this makes sense to me. But we have some confusions on the document right now because we have two different ways to install lws, one with kustomize, another with helm chart, I'll update the document later to make sure people can participate in the project development easier. Thanks anyway.

/lgtm /approve /kind feature