Skip to content

Conversation

zhihangdeng
Copy link
Collaborator

@zhihangdeng zhihangdeng commented Aug 13, 2025

Description

openEuler is an open source OS run by the OpenAtom Foundation that builds a global community to enhance digital infrastructure and support diverse architectures.
This PR adds support for openEuler-based container images to AudioQnA by introducing Dockerfiles that use openEuler as the base image instead of the current Ubuntu/Alpine base images.

Benefits of this change:

  • Provides alternative base image option for users preferring openEuler ecosystem
  • Enhances compatibility with openEuler-based Kubernetes clusters
  • Supports diverse enterprise environments that standardize on openEuler
  • Maintains feature parity with existing images while offering different base OS option

The changes include:

  • New Dockerfiles using openEuler base images for core AudioQnA components
  • Updated package installation commands compatible with openEuler's package manager
  • Preserved all existing functionality and dependencies
  • Added openEuler-specific optimizations where applicable

Issues

n/a

Type of change

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

@Copilot Copilot AI review requested due to automatic review settings August 13, 2025 02:38
Copy link

github-actions bot commented Aug 13, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

Copy link
Contributor

@Copilot 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 AudioQnA support for the openEuler operating system by introducing openEuler-specific Docker configurations and deployment scripts. The changes enable AudioQnA to run on openEuler with specialized container images and compose configurations.

  • Adds openEuler-specific Dockerfiles for the AudioQnA service and UI
  • Introduces a comprehensive test script for openEuler deployment on Xeon processors
  • Updates build configuration to include openEuler variants of all AudioQnA components
  • Provides a complete Docker Compose setup for openEuler-based AudioQnA deployment

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
AudioQnA/ui/docker/Dockerfile.openEuler New Dockerfile for building AudioQnA UI on openEuler base image
AudioQnA/tests/test_compose_openeuler_on_xeon.sh Test script for validating AudioQnA deployment on openEuler/Xeon
AudioQnA/docker_image_build/build.yaml Updates to build openEuler variants of AudioQnA components
AudioQnA/docker_compose/intel/cpu/xeon/compose_openeuler.yaml Docker Compose configuration for openEuler deployment
AudioQnA/Dockerfile.openEuler Main AudioQnA service Dockerfile for openEuler

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

@zhihangdeng zhihangdeng changed the title add AudioQnA for openEuler Add AudioQnA for openEuler Aug 13, 2025
@zhihangdeng zhihangdeng changed the title Add AudioQnA for openEuler Add openEuler support for AudioQnA Aug 26, 2025
@zhihangdeng
Copy link
Collaborator Author

Hi, @Spycsh and @WenjiaoYue, could you please review this? Let me know if you have any questions. Thanks!

Copy link
Collaborator

@Spycsh Spycsh left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants