Skip to content

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

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

Conversation

chyundunovDatamonsters
Copy link
Contributor

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.

  • [*] New feature (non-breaking change which adds new functionality)

Dependencies

Tests

Chingis Yundunov and others added 17 commits February 13, 2025 10:02
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>
Copy link

github-actions bot commented Apr 11, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

@yinghu5 yinghu5 requested a review from yongfengdu April 24, 2025 01:38
chyundunovDatamonsters and others added 6 commits April 24, 2025 20:33
…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>
…nt using Helm

Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
…nt using Helm

Signed-off-by: Chingis Yundunov <c.yundunov@datamonsters.com>
Copy link
Collaborator

@mkbhanda mkbhanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yinghu5 yinghu5 requested a review from lianhao May 7, 2025 06:22
@yinghu5
Copy link
Collaborator

yinghu5 commented May 7, 2025

Hi @yongfengdu @lianhao could you please help review the PR, thank you!

@xiguiw xiguiw requested a review from ashahba as a code owner May 16, 2025 01:24
supervisor:
llm_endpoint_url: http://{{ .Release.Name }}-tgi
llm_engine: tgi
model: "meta-llama/Meta-Llama-3-8B-Instruct"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

@lianhao lianhao left a 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.

@yinghu5
Copy link
Collaborator

yinghu5 commented May 23, 2025

Hi @chyundunovDatamonsters , the PR almost done, could you please help to address the left comments? thank you

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.

7 participants