-
Notifications
You must be signed in to change notification settings - Fork 287
AgentQnA - Adding files to deploy an application in the K8S environment using Helm #1793
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
AgentQnA - Adding files to deploy an application in the K8S environment using Helm #1793
Conversation
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
…nt using Helm Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>
Dependency Review✅ No vulnerabilities or license issues found.Scanned FilesNone |
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
for more information, see https://pre-commit.ci
…into feature/AgentQnA_k8s
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
for more information, see https://pre-commit.ci
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
for more information, see https://pre-commit.ci
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Hi @yongfengdu @lianhao could you please help review the PR, thank you! |
supervisor: | ||
llm_endpoint_url: http://{{ .Release.Name }}-tgi | ||
llm_engine: tgi | ||
model: "meta-llama/Meta-Llama-3-8B-Instruct" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's align the values with GenAIInfra, https://github.com/opea-project/GenAIInfra/pull/975/files#diff-8c465c490b6932f29f8e7222851ed7ec3cf5b183394d47d4ae16e6747ae3a5acR57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
agreed with @chensuyue. The helm value files here should be a simple copy from the corresponding value files from GenAIInfra repo.
Hi @chyundunovDatamonsters , the PR almost done, could you please help to address the left comments? thank you |
…nt using Helm Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
Description
AgentQnA - Adding files to deploy an application in the K8S environment using Helm
Issues
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
Tests