search

opea-project / GenAIExamples

Generative AI Examples is a collection of GenAI examples such as ChatQnA, Copilot, which illustrate the pipeline capabilities of the Open Platform for Enterprise AI (OPEA) project.
https://opea.dev
Apache License 2.0
230 stars 148 forks source link

[Bug] Submit Helm charts to Artifact repo #813

Open arun-gupta opened 1 week ago

arun-gupta commented 1 week ago

Priority

Undecided

OS type

Ubuntu

Hardware type

Xeon-GNR

Installation method

Deploy method

Running nodes

Single Node

What's the version?

latest

Description

Currently, the Helm charts are downloaded using curl or the workspace needs to be cloned. The Helm community provides a standard Artifact Repo to store such artifacts. The Helm charts should be stored in this repo providing a more natural experience.

Reproduce steps

The steps are listed at https://github.com/opea-project/GenAIExamples/blob/main/ChatQnA/kubernetes/intel/README.md and need to be simplified.

Raw log

No response

yongfengdu commented 1 week ago

Now we're putting all the helm charts to this repo: (Included v0.8, v0.9 releases, the 1.0 release is ongoing) https://opea-project.github.io/GenAIInfra/

arun-gupta commented 1 week ago

That sounds like a good experience, looking forward to test it. Will all the docs be updated to include a link to use this Helm repo?

yongfengdu commented 1 week ago

The helm repo doc was included in an overall README for helm charts themselves, but not linked from the portal yet. https://github.com/opea-project/GenAIInfra/tree/main/helm-charts#using-helm-charts-repository

poussa commented 2 days ago

Also, the manifest files which are generated from the GenAIInfra helm charts and are part of the GenAIExamples repo should be deleted. This is very confusing user experience.

For example, the GenAIInfra/ChatQnA helm chart should be enough. Now we have the generated manifest files (via helm template also checked in GenAIExamples.

This pollutes the repo with extra 50+ yaml files plus READMEs which add no value and confuses users.

yongfengdu commented 1 day ago

Also, the manifest files which are generated from the GenAIInfra helm charts and are part of the GenAIExamples repo should be deleted. This is very confusing user experience.

For example, the GenAIInfra/ChatQnA helm chart should be enough. Now we have the generated manifest files (via helm template also checked in GenAIExamples.

This pollutes the repo with extra 50+ yaml files plus READMEs which add no value and confuses users.

Agree, I also think the manifests are duplicate stuff for ChatQnA. More confusing is that, some of the manifests (Which have Helm chart support) are automatically generated from "helm template", but some of them are not supported by Helm chart, but also have manually maintained manifests(https://github.com/opea-project/GenAIExamples/tree/main/FaqGen/kubernetes/intel/cpu/xeon/manifest).

There should be an overall guideline of how to deploy on k8s, including code structure and everyone should follow it. After this, we can re-organize/cleanup everything, otherwise, just delete the manifests will cause more confusion.