-
Notifications
You must be signed in to change notification settings - Fork 217
Add openEuler support for llm-docsum
#1939
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
base: main
Are you sure you want to change the base?
Conversation
afd31b2 to
2f85367
Compare
4ad2475 to
8de534c
Compare
Signed-off-by: zhihang <[email protected]>
529a381 to
5c1875e
Compare
Signed-off-by: zhihang <[email protected]>
|
(llms_doc-summarization_vllm Timeout failure. @zhihangdeng Could you take a look at it? I tested the main branch and it works fine. https://github.com/opea-project/GenAIComps/actions/runs/19491488676/job/55784405366 |
Signed-off-by: zhihang <[email protected]>
I tried to figure out the reason, and the cause of the timeout does not appear to be related to this PR. |
This pull request adds support for building and testing the document summarization microservice using the openEuler OS in addition to the existing setup. It introduces a new
Dockerfile.openEulerfor openEuler-based builds and updates the test scripts to build and validate both the default and openEuler Docker images. The test scripts are also refactored for improved maintainability.The most important changes are:
openEuler Docker support:
Dockerfile.openEulerincomps/llms/src/doc-summarizationto build the document summarization microservice using the openEuler Python 3.11 image, with specific dependencies and environment setup for air-gapped and model caching support.Test script enhancements:
test_llms_doc-summarization_tgi.shandtest_llms_doc-summarization_vllm.shto:DATA_PATHis set. [1] [2]Test script refactoring:
stop_dockerfunction tostop_servicein both test scripts for clearer intent, and updated all usages accordingly. [1] [2]