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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

code-gen-react-ui #417

Merged
merged 29 commits into from
Jul 24, 2024
Merged

Conversation

jaswanth8888
Copy link
Collaborator

Description

Added Code Gen React UI

Issues

n/a.

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)

Dependencies

  • mantine/core
  • mantine/hooks
  • microsoft/fetch-event-source
  • reduxjs/toolkit
  • tabler/icons-react
  • axios
  • react
  • react-dom
  • react-markdown
  • react-syntax-highlighter
  • remark-frontmatter
  • remark-gfm

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

hteeyeoh and others added 2 commits July 17, 2024 15:40
Include manifest to deploy CodeGen pipeline with ReactUI into
Kubernetes. Include as well README file for the steps.

Signed-off-by: Yeoh, Hoong Tee <hoong.tee.yeoh@intel.com>
Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
@chensuyue
Copy link
Collaborator

Add one line after this one to replace the registry,

sed -i "s#image: opea/codegen-ui:latest#image: opea/codegen-ui:${IMAGE_TAG}#g" docker_compose.yaml

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
@jaswanth8888
Copy link
Collaborator Author

sed -i "s#image: opea/codegen-ui:latest#image: opea/codegen-ui:${IMAGE_TAG}#g" docker_compose.yaml

added

@chensuyue chensuyue added this to the v0.8 milestone Jul 19, 2024
@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

If this works in your local test, I will do more test to identify the issue.

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
@yogeshmpandey
Copy link
Contributor

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

If this works in your local test, I will do more test to identify the issue.

@chensuyue

Tests works locally on Xeon.

image

CodeGen/docker/ui/react/README.md Outdated Show resolved Hide resolved
CodeGen/docker/ui/react/public/vite.svg Outdated Show resolved Hide resolved
CodeGen/docker/ui/react/src/vite-env.d.ts Outdated Show resolved Hide resolved
CodeGen/docker/xeon/docker_compose.yaml Show resolved Hide resolved
CodeGen/docker/gaudi/docker_compose.yaml Show resolved Hide resolved
@chensuyue
Copy link
Collaborator

docker compose -f docker_compose.yaml up -d failed with https://github.com/opea-project/GenAIExamples/actions/runs/10034513589/job/27729334956?pr=417#step:5:554 , does it work in your local test?

codegen-xeon-ui-server Pulled 
failed to solve: failed to read dockerfile: open /var/lib/docker/tmp/buildkit-mount3891912248/Dockerfile: no such file or directory
#1 [internal] load .dockerignore
#1 transferring context: 2B done
#1 DONE 0.0s

#2 [internal] load build definition from Dockerfile
#2 transferring dockerfile: 2B done
#2 DONE 0.0s

Seems the issues is it's not going to pull the docker image from our test registry but build from source, then not able to get the Dockerfile, if that's the issue, you should able reproduce in your local test.

If this works in your local test, I will do more test to identify the issue.

@chensuyue

Tests works locally on Xeon.

image

Ok, then we will do some local test on CI machine.

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
@jaswanth8888 jaswanth8888 merged commit 1b48e54 into opea-project:main Jul 24, 2024
19 of 20 checks passed
yogeshmpandey pushed a commit to hteeyeoh/GenAIExamples that referenced this pull request Aug 12, 2024
* CodeGen: Manifest to deploy CodeGen with ReactUI into Kubernetes

Include manifest to deploy CodeGen pipeline with ReactUI into
Kubernetes. Include as well README file for the steps.

Signed-off-by: Yeoh, Hoong Tee <hoong.tee.yeoh@intel.com>

* code-gen-react-ui

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* made changes as per PR suggestions

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

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

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

* updated readme

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

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

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

* update base image

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* updated codegn dockerfile react

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* updated wait time

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* updated as per PR comments

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* aded react ui in gaudi test file

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* fixed PR comments

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* added docker pull for text-generation-model

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

* removed unused files

Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>

---------

Signed-off-by: Yeoh, Hoong Tee <hoong.tee.yeoh@intel.com>
Signed-off-by: jaswanth8888 <karani.jaswanth@gmail.com>
Co-authored-by: Yeoh, Hoong Tee <hoong.tee.yeoh@intel.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: chen, suyue <suyue.chen@intel.com>
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.

6 participants