Create UBI based vLLM docker build instructions#713
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds documentation and build instructions for a UBI (Red Hat Universal Base Image) based vLLM Docker container. The changes enable users to build vLLM with Intel® Gaudi® support on RHEL-based systems in addition to the existing Ubuntu-based option.
Key Changes:
- Updated installation documentation to reference both Ubuntu and UBI-based Dockerfiles
- Added comprehensive UBI Dockerfile build instructions with configurable build arguments
- Corrected COPY paths in the UBI Dockerfile to build from repository root
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/getting_started/installation.md | Added new section documenting UBI-based Dockerfile build process with build arguments and usage examples |
| .cd/Dockerfile.rhel.ubi.vllm | Updated COPY paths to use .cd/ prefix for building from repository root directory |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
My suggestions are in this PR: ghandoura#1 |
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
Minor change in the docker file to build from root directory. Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com>
Signed-off-by: mhelf-intel <monika.helfer@intel.com>
|
@PatrykWo document updates done |
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
|
@PatrykWo - please verify if this is still needed and merge if it is |
Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com>
Minor change in the docker file to build from root directory. --------- Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com> Signed-off-by: Rohit kumar Singh <rksingh@habana.ai>
Minor change in the docker file to build from root directory. --------- Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com>
Minor change in the docker file to build from root directory. --------- Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com>
Minor change in the docker file to build from root directory. --------- Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com> (cherry picked from commit 40a425f)
Minor change in the docker file to build from root directory. --------- Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com> (cherry picked from commit 40a425f) Signed-off-by: PatrykWo <patryk.wolsza@intel.com>
…0.15.1 (#1049) ## Summary This PR cherry-picks all RHEL/UBI Dockerfile changes merged to `main` after `releases/v0.15.1` into the v0.15.1 release branch. ## Cherry-picked PRs | PR | Commit | Description | |----|--------|-------------| | [#923](#923) | `6d15fdc` | [GAUDISW-244821] Modify UBI docker to support both internal and external builds | | [#811](#811) | `a0a0d36` | Fix reported version of vllm | | [#713](#713) | `40a425f` | Create UBI based vLLM docker build instructions | | [#974](#974) | `6db03ad` | [GADC-941] Add libfdt-devel (new habanalabs-thunk dependency) to UBI Dockerfile | | [#971](#971) | `a3855ac` | [GAUDISW-246357] UBI images improvements | | [#1008](#1008) | `b3b2fb3` | Fix Dockerfile for RHEL 9.6 build by updating package installation order | ## Key changes in `.cd/Dockerfile.rhel.ubi.vllm` - Added new build args: `OS_VERSION`, `OS_STRING`, `PT_MODULES_REPO_NAME`, `PT_PACKAGE_NAME_NON_DEFAULT_PYTHON_SUBSTRING`, `PYPI_INDEX_URL`, `HABANA_RPM_REPO_PATH` - Support `SYNAPSE_REVISION=latest` (auto-detects newest available revision) - Detected Synapse revision stored in `/etc/habanalabs/synapse_revision` for use across stages - `dnf install` uses `--allowerasing` throughout; `openssl-fips-provider-so` removal has `|| true` to support RHEL 9.4 - Added packages: `libomp`, `libjpeg-turbo-devel` (replaces `libjpeg-devel`), `libfdt-devel` - `pip` calls use `--no-cache-dir` - vLLM install: replaced `use_existing_torch.py` with `pip install -r <(sed '/^torch/d' requirements/build.txt)` - `pip check` added after installation - Final stage renamed to `AS vllm-openai` - `OS_STRING` is now parametric (supports both RHEL 9.4 and 9.6) --------- Signed-off-by: Michal Muszynski <mmuszynski@habana.ai> Signed-off-by: PatrykWo <patryk.wolsza@intel.com> Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Signed-off-by: Michal Muszynski <michal.muszynski@intel.com> Co-authored-by: Michal Muszynski <141021743+mmuszynskihabana@users.noreply.github.com> Co-authored-by: Adam Ghandoura <adam.ghandoura@intel.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Iryna Boiko <iboiko@habana.ai> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: Michał Kuligowski <michal.kuligowski@intel.com> Co-authored-by: Agata Dobrzyniewicz <160237065+adobrzyn@users.noreply.github.com>
…0.15.1 (#1049) This PR cherry-picks all RHEL/UBI Dockerfile changes merged to `main` after `releases/v0.15.1` into the v0.15.1 release branch. | PR | Commit | Description | |----|--------|-------------| | [#923](#923) | `6d15fdc` | [GAUDISW-244821] Modify UBI docker to support both internal and external builds | | [#811](#811) | `a0a0d36` | Fix reported version of vllm | | [#713](#713) | `40a425f` | Create UBI based vLLM docker build instructions | | [#974](#974) | `6db03ad` | [GADC-941] Add libfdt-devel (new habanalabs-thunk dependency) to UBI Dockerfile | | [#971](#971) | `a3855ac` | [GAUDISW-246357] UBI images improvements | | [#1008](#1008) | `b3b2fb3` | Fix Dockerfile for RHEL 9.6 build by updating package installation order | - Added new build args: `OS_VERSION`, `OS_STRING`, `PT_MODULES_REPO_NAME`, `PT_PACKAGE_NAME_NON_DEFAULT_PYTHON_SUBSTRING`, `PYPI_INDEX_URL`, `HABANA_RPM_REPO_PATH` - Support `SYNAPSE_REVISION=latest` (auto-detects newest available revision) - Detected Synapse revision stored in `/etc/habanalabs/synapse_revision` for use across stages - `dnf install` uses `--allowerasing` throughout; `openssl-fips-provider-so` removal has `|| true` to support RHEL 9.4 - Added packages: `libomp`, `libjpeg-turbo-devel` (replaces `libjpeg-devel`), `libfdt-devel` - `pip` calls use `--no-cache-dir` - vLLM install: replaced `use_existing_torch.py` with `pip install -r <(sed '/^torch/d' requirements/build.txt)` - `pip check` added after installation - Final stage renamed to `AS vllm-openai` - `OS_STRING` is now parametric (supports both RHEL 9.4 and 9.6) --------- Signed-off-by: Michal Muszynski <mmuszynski@habana.ai> Signed-off-by: PatrykWo <patryk.wolsza@intel.com> Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Signed-off-by: Michal Muszynski <michal.muszynski@intel.com> Co-authored-by: Michal Muszynski <141021743+mmuszynskihabana@users.noreply.github.com> Co-authored-by: Adam Ghandoura <adam.ghandoura@intel.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Iryna Boiko <iboiko@habana.ai> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: Michał Kuligowski <michal.kuligowski@intel.com> Co-authored-by: Agata Dobrzyniewicz <160237065+adobrzyn@users.noreply.github.com>
Minor change in the docker file to build from root directory. --------- Signed-off-by: Adam Ghandoura <adam.ghandoura@intel.com> Signed-off-by: mhelf-intel <monika.helfer@intel.com> Co-authored-by: aghandoura <adam.ghandoura@gmail.com> Co-authored-by: mhelf-intel <monika.helfer@intel.com>
Minor change in the docker file to build from root directory.