Skip to content

Conversation

@bhagyashrigai
Copy link

@bhagyashrigai bhagyashrigai commented Nov 18, 2025

Purpose

This PR updates the ppc64le Dockerfile to use gcc-toolset-14 instead of gcc-toolset-13 and includes additional environment fixes required for successful builds. The update resolves multiple test case failures observed during vLLM and dependency builds on the ppc64le platform.

Key Changes

  • Updated all Dockerfile stages from gcc-toolset-13 → gcc-toolset-14.
  • Updated compiler activation paths (source /opt/rh/gcc-toolset-14/enable).
  • Exported LLVM_CONFIG and updated PATH to ensure correct LLVM detection.
  • Added missing dependency (libicu) for ppc64le build stability.
  • Updated platform selectors in requirements/common.txt to include:platform_machine == "ppc64le" for llguidance and xgrammar

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

@mergify mergify bot added the ci/build label Nov 18, 2025
@bhagyashrigai bhagyashrigai changed the title Update Dockerfile to use gcc-toolset-14 and fix test case failures on… Update Dockerfile to use gcc-toolset-14 and fix test case failures on power Nov 18, 2025
@bhagyashrigai bhagyashrigai changed the title Update Dockerfile to use gcc-toolset-14 and fix test case failures on power Update Dockerfile to use gcc-toolset-14 and fix test case failures on power (ppc64le) Nov 18, 2025
@mergify mergify bot added the v1 label Nov 18, 2025
@bhagyashrigai bhagyashrigai marked this pull request as ready for review November 19, 2025 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant