Skip to content

add a cpu specific helm chart values file for AgentQnA #1601

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

haoruan
Copy link

@haoruan haoruan commented Mar 3, 2025

Description

Add a CPU specific values file as an example

Issues

Fixes #1523

Type of change

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

  • Others (enhancement, documentation, validation, etc.)

Dependencies

N/A

Tests

Install the helm chart from source code https://github.com/opea-project/GenAIInfra/tree/main/helm-charts/agentqna

Copy link

github-actions bot commented Mar 3, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

@lianhao
Copy link
Collaborator

lianhao commented Mar 6, 2025

@haoruan Thanks for submitting this. However, the helm chart related PRs should be submit to https://github.com/opea-project/GenAIInfra. The values files in this repo is copied from there.

@haoruan
Copy link
Author

haoruan commented Mar 7, 2025

Sure, I'll close this

@haoruan haoruan closed this Mar 7, 2025
@haoruan haoruan reopened this Mar 17, 2025
@haoruan haoruan force-pushed the feature/agentqna-create-cpu-helm-chart branch from 38f2507 to dde6cba Compare March 17, 2025 08:35
@letonghan letonghan added the OPEAHack Issue created for OPEA Hackathon label Mar 17, 2025
@haoruan
Copy link
Author

haoruan commented Mar 25, 2025

@letonghan Mind taking a look at this when you get a chance?

@letonghan
Copy link
Collaborator

letonghan commented Mar 25, 2025

Hi @haoruan , which version of retriever do you use? this ModuleNotFoundError: No module named 'aiofiles' issue in retriever should be already fixed in #1393

@haoruan
Copy link
Author

haoruan commented Mar 25, 2025

The retriever should be the default one, I installed the helm chart locally and there was no problem. By the way the values file is just copied from https://github.com/opea-project/GenAIInfra, not sure why it's failed.

@lianhao lianhao force-pushed the feature/agentqna-create-cpu-helm-chart branch from dde6cba to 27cc813 Compare March 31, 2025 09:26
@lianhao
Copy link
Collaborator

lianhao commented Mar 31, 2025

The CI is under migration. Please hold this until the CI migration is finished. I don't see any issue of this PR. In GenAIInfra side, corresponding PR is already merged

@xiguiw xiguiw force-pushed the feature/agentqna-create-cpu-helm-chart branch from 27cc813 to 33c7ea2 Compare April 8, 2025 08:24
Signed-off-by: Ruan Hao <hao.ruan@intel.com>
@lianhao lianhao force-pushed the feature/agentqna-create-cpu-helm-chart branch from 33c7ea2 to 5d9d810 Compare April 8, 2025 12:18
@xiguiw
Copy link
Collaborator

xiguiw commented Apr 17, 2025

@haoruan

[Remind] There is conflicts.
Would you please resolve the conflicts?

@haoruan
Copy link
Author

haoruan commented Apr 17, 2025

The values file already synced to this repo, do we still need this PR?

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

Successfully merging this pull request may close these issues.

AgentQnA - Create a CPU specific helm chart for Kubernetes environments
4 participants