Add openEuler Helm values for AudioQnA and CodeGen #2274
Conversation
Dependency Review✅ No vulnerabilities or license issues found.Scanned FilesNone |
There was a problem hiding this comment.
Pull Request Overview
This PR adds openEuler platform support for the AudioQnA Helm deployment by introducing a specialized values file. The configuration enables specific services compatible with openEuler while disabling others, and standardizes on openEuler-specific image tags.
- Adds
cpu-openeuler-value.yamlwith openEuler-specific service configurations - Enables
vllmandspeecht5services while disablingtgiandgpt-sovits - Configures all enabled services to use
latest-openeulerimage tags
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| image: | ||
| tag: latest-openeuler | ||
|
|
There was a problem hiding this comment.
The global latest-openeuler tag may not apply to all services. Consider removing this global setting since individual services already specify their own tags explicitly (lines 17, 22, 29, 35).
| image: | |
| tag: latest-openeuler |
Copilot uses AI. Check for mistakes.
| enabled: true | ||
| image: | ||
| repository: openeuler/vllm-cpu | ||
| tag: 0.10.1-oe2403lts |
There was a problem hiding this comment.
The vllm service uses a specific versioned tag 0.10.1-oe2403lts while other services use latest-openeuler. Consider using consistent tagging strategy across all services for better maintainability and predictable deployments.
| tag: 0.10.1-oe2403lts | |
| tag: latest-openeuler |
Copilot uses AI. Check for mistakes.
zhihangdeng
force-pushed
the
audioqna
branch
from
668c595 to
69d923c
Compare
Signed-off-by: zhihang <[email protected]>
zhihangdeng
force-pushed
the
audioqna
branch
2 times, most recently
from
5b5cbc5 to
1ff1c73
Compare
zhihangdeng
requested review from
lvliang-intel and
yao531441
as code owners
zhihangdeng
changed the title
zhihangdeng
force-pushed
the
audioqna
branch
5 times, most recently
from
78dd724 to
1e27848
Compare
Signed-off-by: zhihang <[email protected]>
zhihangdeng
force-pushed
the
audioqna
branch
from
47841dd to
27ce1d8
Compare
for more information, see https://pre-commit.ci
Description
This pull request introduces new Helm values files for deploying services on openEuler-based CPU images in both the
AudioQnAandCodeGenprojects. The main goal is to override image tags and repositories to ensure compatibility with openEuler, and to selectively enable or disable services based on the deployment needs.Helm values for openEuler compatibility:
cpu-openeuler-values.yamltoAudioQnA/kubernetes/helm, setting image tags tolatest-openeulerand updating repositories for services likevllm, while enabling/disabling services as appropriate.cpu-openeuler-values.yamltoCodeGen/kubernetes/helm, overriding image tags and repositories for all enabled services to use openEuler-compatible images, and configuring backend settings for text generation and embeddings.cpu-tgi-openeuler-values.yamltoCodeGen/kubernetes/helm, specifically enabling the TGI service with openEuler images and disabling VLLM, while updating other service images and backend settings for openEuler compatibility.Issues
#2264
Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.