Skip to content

Conversation

@zhihangdeng
Copy link
Collaborator

This pull request adds comprehensive support for building, deploying, and testing the DocSum application and its related services on the openEuler platform, specifically targeting Intel Xeon CPUs. The changes introduce new Dockerfiles, docker-compose configurations, and build/test scripts to enable seamless multi-service orchestration and validation in openEuler environments.

openEuler Platform Support

  • Added a new Dockerfile.openEuler for the DocSum service, including necessary dependencies and configuration for openEuler compatibility.
  • Introduced multiple openEuler-specific docker-compose files (compose_openeuler.yaml, compose_tgi_openeuler.yaml) for orchestrating the DocSum service stack (LLM, Whisper, backend, and UI) on Intel Xeon CPUs. [1] [2]

Build System Enhancements

  • Updated build.yaml to add openEuler variants for DocSum, Gradio UI, Whisper, LLM-DocSum, and React UI services, each using their respective openEuler Dockerfiles and image tags. [1] [2] [3]

Testing and Validation

  • Added a new test script test_compose_openeuler_on_xeon.sh that builds images, starts services, and performs extensive validation of microservices and megaservice endpoints for text, audio, and video summarization scenarios.

Signed-off-by: zhihang <zhihangdeng@link.cuhk.edu.cn>
Copilot AI review requested due to automatic review settings October 11, 2025 05:56
@github-actions
Copy link

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive openEuler platform support for DocSum, including new Dockerfiles, docker compose configurations, and test scripts for deploying the DocSum application stack on Intel Xeon CPUs using the openEuler operating system.

  • Creates openEuler-specific Dockerfiles for all DocSum services (main service, UI variants, Whisper, and LLM components)
  • Adds docker compose files for orchestrating services with both vLLM and TGI backends on openEuler
  • Introduces test scripts to validate microservices and megaservice functionality across text, audio, and video summarization scenarios

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
DocSum/ui/docker/Dockerfile.react.openEuler React UI Dockerfile for openEuler with nginx deployment
DocSum/ui/docker/Dockerfile.openEuler Svelte UI Dockerfile for openEuler with Node.js
DocSum/ui/docker/Dockerfile.gradio.openEuler Gradio UI Dockerfile for openEuler with Python environment
DocSum/tests/test_compose_tgi_openeuler_on_xeon.sh Test script for TGI-based openEuler deployment validation
DocSum/tests/test_compose_openeuler_on_xeon.sh Test script for vLLM-based openEuler deployment validation
DocSum/docker_image_build/build.yaml Build configuration updated with openEuler service variants
DocSum/docker_compose/intel/cpu/xeon/compose_tgi_openeuler.yaml Docker compose for TGI backend on openEuler
DocSum/docker_compose/intel/cpu/xeon/compose_openeuler.yaml Docker compose for vLLM backend on openEuler
DocSum/Dockerfile.openEuler Main DocSum service Dockerfile for openEuler

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +7 to +10
# Update package manager and install Git
RUN yum update -y && \
yum install -y \
git && \
Copy link

Copilot AI Oct 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Git installation is unnecessary since the code is already copied into the container. This adds unnecessary bloat to the Docker image.

Suggested change
# Update package manager and install Git
RUN yum update -y && \
yum install -y \
git && \
# Update package manager
RUN yum update -y && \

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant