Add structure to `requirements/` directory by hmellor · Pull Request #39024 · vllm-project/vllm

hmellor · 2026-04-05T09:39:18Z

Currently build and test requirements live alongside the user requirements in requirements/.

This PR:

moves all build-only requirements files to requirements/build
moves all test-only requirements files to requirements/test
adds a pip compile hook for XPU so that all requirements/test/<device>.in files are treated the same

This should reduce noise in requirements/ and improve the consistency of depencency management across devices.

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>

mergify · 2026-04-05T09:39:58Z

Documentation preview: https://vllm--39024.org.readthedocs.build/en/39024/

gemini-code-assist

Code Review

This pull request reorganizes the project's requirement files by moving test-related dependencies into a new structured requirements/test/ directory. The changes include renaming and relocating CUDA, ROCm, XPU, and nightly-torch requirement files, while updating all corresponding references across CI configurations, Dockerfiles, pre-commit hooks, and documentation. I have no feedback to provide.

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>

mergify · 2026-04-06T15:49:34Z

This pull request has merge conflicts that must be resolved before it can be
merged. Please rebase the PR, @hmellor.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>

…-requirements

mergify · 2026-04-06T18:06:58Z

Hi @hmellor, the pre-commit checks have failed. Please run:

uv pip install pre-commit>=4.5.1
pre-commit install
pre-commit run --all-files

Then, commit the changes and push to your branch.

For future commits, pre-commit will run automatically on changed files before each commit.

Tip

Is mypy failing?

mypy is run differently in CI. If the failure is related to this check, please use the following command to run it locally:

# For mypy (substitute "3.10" with the failing version if needed)
pre-commit run --hook-stage manual mypy-3.10

Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com>

…ts (fixes #39024) (#39793) Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com>

…quirements paths and nixl_connector imports after upstream restructuring (#1342) ## Summary Fixes two regressions introduced by recent upstream vLLM directory restructurings that break Docker builds, pre-commit hooks, and HPU unit tests. ## Changes 1. **Update requirements paths in Dockerfiles, README, docs and pre-commit config** — upstream moved `requirements/build.txt` → `requirements/build/cuda.txt` and `requirements/test.in` → `requirements/test/cuda.in` 2. **Update nixl_connector imports in HPU model runner and NIXL connectors** — upstream split monolithic `nixl_connector.py` into a `nixl/` package with sub-modules (`connector.py`, `metadata.py`, `scheduler.py`, `stats.py`, `utils.py`, `worker.py`) ## Upstream PRs that introduced these regressions - vllm-project/vllm#39024 — restructured `requirements/` directory layout (fix 1) - vllm-project/vllm#39354 — reorganized NIXL connector into its own directory (fix 2) --------- Signed-off-by: Paweł Olejniczak <pawelx.olejniczak@intel.com>

…ts (fixes vllm-project#39024) (vllm-project#39793) Signed-off-by: Netanel Haber <58652339+netanel-haber@users.noreply.github.com> Signed-off-by: zengxian <xiangdong.zeng@intel.com>

…quirements paths and nixl_connector imports after upstream restructuring (vllm-project#1342) ## Summary Fixes two regressions introduced by recent upstream vLLM directory restructurings that break Docker builds, pre-commit hooks, and HPU unit tests. ## Changes 1. **Update requirements paths in Dockerfiles, README, docs and pre-commit config** — upstream moved `requirements/build.txt` → `requirements/build/cuda.txt` and `requirements/test.in` → `requirements/test/cuda.in` 2. **Update nixl_connector imports in HPU model runner and NIXL connectors** — upstream split monolithic `nixl_connector.py` into a `nixl/` package with sub-modules (`connector.py`, `metadata.py`, `scheduler.py`, `stats.py`, `utils.py`, `worker.py`) ## Upstream PRs that introduced these regressions - vllm-project/vllm#39024 — restructured `requirements/` directory layout (fix 1) - vllm-project/vllm#39354 — reorganized NIXL connector into its own directory (fix 2) --------- Signed-off-by: Paweł Olejniczak <pawelx.olejniczak@intel.com> Signed-off-by: Yeonsil Yoon <yeon.sil.yoon@intel.com>

frgossen · 2026-04-21T20:49:16Z

I just ran into this. The documentation at https://docs.vllm.ai/en/stable/contributing/#license is now incorrect. Can you update it?

…quirements paths and nixl_connector imports after upstream restructuring (vllm-project#1342) ## Summary Fixes two regressions introduced by recent upstream vLLM directory restructurings that break Docker builds, pre-commit hooks, and HPU unit tests. ## Changes 1. **Update requirements paths in Dockerfiles, README, docs and pre-commit config** — upstream moved `requirements/build.txt` → `requirements/build/cuda.txt` and `requirements/test.in` → `requirements/test/cuda.in` 2. **Update nixl_connector imports in HPU model runner and NIXL connectors** — upstream split monolithic `nixl_connector.py` into a `nixl/` package with sub-modules (`connector.py`, `metadata.py`, `scheduler.py`, `stats.py`, `utils.py`, `worker.py`) ## Upstream PRs that introduced these regressions - vllm-project/vllm#39024 — restructured `requirements/` directory layout (fix 1) - vllm-project/vllm#39354 — reorganized NIXL connector into its own directory (fix 2) --------- Signed-off-by: Paweł Olejniczak <pawelx.olejniczak@intel.com> Signed-off-by: bmyrcha <bartosz.myrcha@intel.com>

hmellor · 2026-04-22T16:45:58Z

You are looking at stable docs, this PR is not in a release yet.