Skip to content

Conversation

ZePan110
Copy link
Collaborator

Description

Use built image opea/vllm-gaudi:1.4 for AgentQnA, CodeGen, CodeTrans, DocSum and VisualQnA.

Issues

List the issue or RFC link this PR is working on. If there is no such link, please mark it as 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.

Copy link

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

Purpose: Replace locally built Gaudi vLLM images with the pre-built pinned image opea/vllm-gaudi:1.4 across AgentQnA, CodeGen, CodeTrans, DocSum, and VisualQnA; remove build logic for the Gaudi forked vLLM image.

  • Removed cloning/build steps and vllm-gaudi service definitions from build scripts and build.yaml files.
  • Hard‑coded the runtime Gaudi vLLM image reference (opea/vllm-gaudi:1.4) in Gaudi docker-compose files.
  • Adjusted test/build scripts accordingly (some inconsistency introduced in image validation between ROCm and Gaudi paths).

Reviewed Changes

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

Show a summary per file
File Description
VisualQnA/tests/test_compose_on_gaudi.sh Removed Gaudi vLLM fork clone and vllm-gaudi from service build list.
VisualQnA/docker_image_build/build.yaml Removed vllm-gaudi build service.
VisualQnA/docker_compose/intel/hpu/gaudi/compose.yaml Hard‑coded vLLM Gaudi image tag 1.4.
DocSum/tests/test_compose_on_gaudi.sh Removed fork clone and vllm-gaudi from build list.
DocSum/docker_image_build/build.yaml Removed vllm-gaudi build service.
DocSum/docker_compose/intel/hpu/gaudi/compose.yaml Hard‑coded vLLM Gaudi image tag 1.4.
CodeTrans/tests/test_compose_on_gaudi.sh Removed fork clone and vllm-gaudi from build list.
CodeTrans/docker_image_build/build.yaml Removed vllm-gaudi build service.
CodeTrans/docker_compose/intel/hpu/gaudi/compose.yaml Hard‑coded vLLM Gaudi image tag 1.4.
CodeGen/tests/test_compose_on_gaudi.sh Removed fork clone and vllm-gaudi from build list.
CodeGen/docker_image_build/build.yaml Removed vllm-gaudi build service.
CodeGen/docker_compose/intel/hpu/gaudi/compose.yaml Hard‑coded vLLM Gaudi image tag 1.4.
AgentQnA/tests/step1_build_images.sh Removed Gaudi vLLM build; added selective image listing only for ROCm; Gaudi function now misnamed.
AgentQnA/docker_image_build/build.yaml Removed vllm-gaudi build service.
AgentQnA/docker_compose/intel/hpu/gaudi/compose.yaml Hard‑coded vLLM Gaudi image tag 1.4.

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

@ZePan110
Copy link
Collaborator Author

DocSum test failure is known issue #2251

@ZePan110 ZePan110 merged commit b611e10 into main Sep 19, 2025
40 of 43 checks passed
@ZePan110 ZePan110 deleted the ze-switch/3 branch September 19, 2025 08:06
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.

3 participants