CodeGen/CodeTrans - Adding files to deploy an application in the K8S environment using Helm #1792

chyundunovDatamonsters · 2025-04-11T04:32:43Z

Description

CodeGen/CodeTrans - 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

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

…environment using Helm Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

github-actions · 2025-04-11T04:32:55Z

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

for more information, see https://pre-commit.ci

…into feature/CodeGen_CodeTrans_k8s

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

…nto feature/CodeGen_CodeTrans_k8s

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

for more information, see https://pre-commit.ci

…into feature/CodeGen_CodeTrans_k8s

…nto feature/CodeGen_CodeTrans_k8s

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

for more information, see https://pre-commit.ci

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

…nto feature/CodeGen_CodeTrans_k8s # Conflicts: # CodeGen/kubernetes/helm/rocm-tgi-values.yaml # CodeTrans/kubernetes/helm/rocm-tgi-values.yaml

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

for more information, see https://pre-commit.ci

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

Copilot

Pull Request Overview

This PR adds new Helm chart files and documentation to deploy an application in a Kubernetes environment using Helm with AMD ROCm support. Key changes include:

Adding separate values YAML files for ROCm deployments with vLLM and TGI configurations.
Updating README documentation to provide detailed deployment instructions for both CodeTrans and CodeGen.
Including system variable setup instructions and command examples for both binary repository and Git repository based deployments.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
CodeTrans/kubernetes/helm/rocm-values.yaml	New Helm values file for configuring ROCm vLLM deployment.
CodeTrans/kubernetes/helm/rocm-tgi-values.yaml	New Helm values file for configuring ROCm TGI deployment.
CodeTrans/kubernetes/helm/README.md	Updated deployment instructions for CodeTrans.
CodeGen/kubernetes/helm/rocm-values.yaml	New Helm values file for configuring ROCm vLLM deployment.
CodeGen/kubernetes/helm/rocm-tgi-values.yaml	New Helm values file for configuring ROCm TGI deployment.
CodeGen/kubernetes/helm/README.md	Updated deployment instructions for CodeGen with a minor typo.

CodeGen/kubernetes/helm/README.md

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

mkbhanda

Curious why not running as non-root for greater security.
LGTM

mkbhanda · 2025-04-25T12:11:31Z

CodeTrans/kubernetes/helm/README.md

+### Creating working dirs
+
+```bash
+mkdir ~/codetrans-k8s-install && cd ~/codetrans-k8s-install


Once you created the directory, cd to it. All the other paths then get shorter, do not need to reference ~/codetrans-k8s-install everywhere

chensuyue · 2025-05-09T06:52:40Z

CodeGen/kubernetes/helm/rocm-tgi-values.yaml

+      memory: 16Gi
+  securityContext:
+    readOnlyRootFilesystem: false
+    runAsNonRoot: false


So here it's ok to keep run with root? Why chatqna is special? https://github.com/opea-project/GenAIInfra/pull/949/files/180f16fb65570968a44663d0490c42ed539862b0#diff-f93551169c7cda08f51cb91abe0a36eb96356b53ace54c5fd940d24d5d4264acR29

Do you have a PR to GenAIInfra?

https://github.com/opea-project/GenAIInfra/blob/main/helm-charts/codetrans/rocm-values.yaml The CodeGen/CodeTrans PRs for GenAIInfra had been merged.

So here it's ok to keep run with root? Why chatqna is special? https://github.com/opea-project/GenAIInfra/pull/949/files/180f16fb65570968a44663d0490c42ed539862b0#diff-f93551169c7cda08f51cb91abe0a36eb96356b53ace54c5fd940d24d5d4264acR29

Changes to the launch from an unprivileged user will be made after this PR is completed - opea-project/GenAIComps#1638

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

for more information, see https://pre-commit.ci

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

…nto feature/CodeGen_CodeTrans_k8s

…into feature/CodeGen_CodeTrans_k8s

CICD-at-OPEA · 2025-06-29T22:42:23Z

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

CICD-at-OPEA · 2025-07-06T22:43:22Z

This PR was closed because it has been stalled for 7 days with no activity.

* add native pii detection guardrail Signed-off-by: daniel-de-leon-user293 <daniel.de.leon@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fix port name env var. and typo Signed-off-by: daniel-de-leon-user293 <daniel.de.leon@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fix license dates Signed-off-by: daniel-de-leon-user293 <daniel.de.leon@intel.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Signed-off-by: daniel-de-leon-user293 <daniel.de.leon@intel.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Abolfazl Shahbazi <12436063+ashahba@users.noreply.github.com>

Chingis Yundunov and others added 17 commits February 13, 2025 10:02

DocSum - add files for deploy app with ROCm vLLM

cf60682

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

DocSum - fix main

1fd1de1

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

DocSum - add files for deploy app with ROCm vLLM

bd2d47e

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

DocSum - fix main

2459ecb

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

Merge remote-tracking branch 'origin/main'

4d35065

DocSum - add files for deploy app with ROCm vLLM

6d5049d

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

DocSum - fix main

9dfbdc5

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

DocSum - add files for deploy app with ROCm vLLM

a8857ae

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

DocSum - fix main

5a38b26

Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

Merge remote-tracking branch 'origin/main'

0e2ef94

Merge branch 'main' of https://github.com/opea-project/GenAIExamples

30071db

Merge branch 'opea-project:main' into main

0757dec

Merge branch 'main' of https://github.com/opea-project/GenAIExamples

9aaf378

Merge branch 'main' of https://github.com/opea-project/GenAIExamples

9cf4b6e

Merge branch 'main' of https://github.com/opea-project/GenAIExamples

8e89787

Merge branch 'main' of https://github.com/opea-project/GenAIExamples

a117c69

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

82e675c

…environment using Helm Signed-off-by: Chingis Yundunov <YundunovCN@sibedge.com>

chyundunovDatamonsters requested review from Spycsh, XinyaoWa and lvliang-intel as code owners April 11, 2025 04:32

pre-commit-ci bot and others added 9 commits April 11, 2025 04:33

[pre-commit.ci] auto fixes from pre-commit.com hooks

d2717ae

for more information, see https://pre-commit.ci

Merge branch 'main' of https://github.com/opea-project/GenAIExamples …

cf9b048

…into feature/CodeGen_CodeTrans_k8s

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

584f4fd

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

Merge remote-tracking branch 'origin/feature/CodeGen_CodeTrans_k8s' i…

742acd6

…nto feature/CodeGen_CodeTrans_k8s

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

9cd726e

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

[pre-commit.ci] auto fixes from pre-commit.com hooks

8b46bf4

for more information, see https://pre-commit.ci

Merge branch 'main' into feature/CodeGen_CodeTrans_k8s

e93bd62

Merge branch 'main' of https://github.com/opea-project/GenAIExamples …

07e838e

…into feature/CodeGen_CodeTrans_k8s

Merge remote-tracking branch 'origin/feature/CodeGen_CodeTrans_k8s' i…

adbb079

…nto feature/CodeGen_CodeTrans_k8s

chyundunovDatamonsters and others added 9 commits April 24, 2025 09:51

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

2b02f6a

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

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

061a646

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

[pre-commit.ci] auto fixes from pre-commit.com hooks

5116ecb

for more information, see https://pre-commit.ci

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

849d8a1

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

Merge remote-tracking branch 'origin/feature/CodeGen_CodeTrans_k8s' i…

b61b824

…nto feature/CodeGen_CodeTrans_k8s # Conflicts: # CodeGen/kubernetes/helm/rocm-tgi-values.yaml # CodeTrans/kubernetes/helm/rocm-tgi-values.yaml

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

34382be

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

[pre-commit.ci] auto fixes from pre-commit.com hooks

87a0169

for more information, see https://pre-commit.ci

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

4bb240f

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

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

c38d6e3

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

yinghu5 requested review from Copilot and yongfengdu April 25, 2025 01:40

Copilot AI reviewed Apr 25, 2025

View reviewed changes

CodeGen/kubernetes/helm/README.md Outdated Show resolved Hide resolved

chyundunovDatamonsters added 2 commits April 25, 2025 09:02

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

f34ac3b

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

Merge branch 'main' into feature/CodeGen_CodeTrans_k8s

03b12e3

mkbhanda approved these changes Apr 25, 2025

View reviewed changes

chensuyue reviewed May 9, 2025

View reviewed changes

chyundunovDatamonsters and others added 4 commits May 27, 2025 12:11

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

e56fac1

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

[pre-commit.ci] auto fixes from pre-commit.com hooks

3c46038

for more information, see https://pre-commit.ci

CodeGen/CodeTrans - Adding files to deploy an application in the K8S …

14f2c1d

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

Merge remote-tracking branch 'origin/feature/CodeGen_CodeTrans_k8s' i…

d93f017

…nto feature/CodeGen_CodeTrans_k8s

chyundunovDatamonsters requested a review from letonghan as a code owner May 27, 2025 05:13

chyundunovDatamonsters and others added 3 commits May 27, 2025 12:13

Merge branch 'main' of https://github.com/opea-project/GenAIExamples …

1e654ee

…into feature/CodeGen_CodeTrans_k8s

Merge branch 'main' into feature/CodeGen_CodeTrans_k8s

4d8db18

Merge branch 'main' into feature/CodeGen_CodeTrans_k8s

c628cf5

CICD-at-OPEA added the Stale label Jun 29, 2025

CICD-at-OPEA closed this Jul 6, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CodeGen/CodeTrans - Adding files to deploy an application in the K8S environment using Helm #1792

CodeGen/CodeTrans - Adding files to deploy an application in the K8S environment using Helm #1792

Uh oh!

chyundunovDatamonsters commented Apr 11, 2025

Uh oh!

github-actions bot commented Apr 11, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

mkbhanda left a comment

Uh oh!

mkbhanda Apr 25, 2025

Uh oh!

chensuyue May 9, 2025

Uh oh!

chensuyue May 9, 2025

Uh oh!

yongfengdu May 9, 2025

Uh oh!

chyundunovDatamonsters May 12, 2025

Uh oh!

CICD-at-OPEA commented Jun 29, 2025

Uh oh!

CICD-at-OPEA commented Jul 6, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

CodeGen/CodeTrans - Adding files to deploy an application in the K8S environment using Helm #1792

CodeGen/CodeTrans - Adding files to deploy an application in the K8S environment using Helm #1792

Uh oh!

Conversation

chyundunovDatamonsters commented Apr 11, 2025

Description

Issues

Type of change

Dependencies

Tests

Uh oh!

github-actions bot commented Apr 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Dependency Review

Scanned Files

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

mkbhanda left a comment

Choose a reason for hiding this comment

Uh oh!

mkbhanda Apr 25, 2025

Choose a reason for hiding this comment

Uh oh!

chensuyue May 9, 2025

Choose a reason for hiding this comment

Uh oh!

chensuyue May 9, 2025

Choose a reason for hiding this comment

Uh oh!

yongfengdu May 9, 2025

Choose a reason for hiding this comment

Uh oh!

chyundunovDatamonsters May 12, 2025

Choose a reason for hiding this comment

Uh oh!

CICD-at-OPEA commented Jun 29, 2025

Uh oh!

CICD-at-OPEA commented Jul 6, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

github-actions bot commented Apr 11, 2025 •

edited

Loading