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

[Bug] Submit Helm charts to Artifact repo #813

Open
3 of 6 tasks
arun-gupta opened this issue Sep 15, 2024 · 5 comments
Open
3 of 6 tasks

[Bug] Submit Helm charts to Artifact repo #813

arun-gupta opened this issue Sep 15, 2024 · 5 comments
Assignees
Labels

Comments

@arun-gupta
Copy link
Contributor

Priority

Undecided

OS type

Ubuntu

Hardware type

Xeon-GNR

Installation method

  • Pull docker images from hub.docker.com
  • Build docker images from source

Deploy method

  • Docker compose
  • Docker
  • Kubernetes
  • Helm

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
Copy link
Collaborator

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
Copy link
Contributor Author

arun-gupta commented Sep 20, 2024

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
Copy link
Collaborator

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
Copy link

poussa commented Sep 26, 2024

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.

@devpramod devpramod added Dev and removed aitce labels Sep 26, 2024
@yongfengdu
Copy link
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants