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

[Feature] Helm charts should mention HW specific values that need to be provided #784

Open
mkbhanda opened this issue Feb 6, 2025 · 3 comments
Assignees
Labels
feature New feature or request

Comments

@mkbhanda
Copy link
Collaborator

mkbhanda commented Feb 6, 2025

Priority

Undecided

OS type

Ubuntu

Hardware type

Xeon-SPR, Gaudi etc

Running nodes

Single Node

Description

As a cloud user I want to use a helm chart but need guidance on values.yaml that needs to be provided based on available infrastructure.

For example the AgentQnA chart is generic across Gaudi and CPU
Thinking how the user can trip up, how to get them to refer to a README for help first.
https://github.com/orgs/opea-project/packages/container/package/charts%2Fagentqna

@mkbhanda mkbhanda added the feature New feature or request label Feb 6, 2025
@mkbhanda mkbhanda changed the title [Feature] Helm charts should mention infrastructure assumptions in their naming [Feature] Helm charts should mention HW specific values that need to be provided Feb 6, 2025
@eero-t
Copy link
Contributor

eero-t commented Feb 17, 2025

Btw. I just noticed that GH workflow is not updating the charts when only their documentation is updated. Updates happen only when other files are (also) updated => filed PR to fix it: #817.

@eero-t
Copy link
Contributor

eero-t commented Feb 17, 2025

Thinking how the user can trip up, how to get them to refer to a README for help first.

As they are Helm charts, user can use helm show readme to view README.md included with the chart.

@yongfengdu
Copy link
Collaborator

The github package https://github.com/orgs/opea-project/packages doesn't provide a way to show each helm charts README.
We shall point the user to https://github.com/opea-project/GenAIInfra/tree/main/helm-charts#readme and improve it continuously.

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

No branches or pull requests

3 participants