Compare commits

...
Author SHA1 Message Date
khluu 74690b24e7 Add tasks/version/logging_utils to models_basic GPU deps
Following the deeper audit pattern from #42219:

- `vllm/tasks.py`: imported by `vllm/model_executor/models/registry.py`
  and other model-side code; a change to `POOLING_TASKS` /
  `SupportedTask` would otherwise silently skip these jobs.
- `vllm/version.py`: imported by `vllm/v1/engine/core.py` and
  `vllm/engine/arg_utils.py`. Release-tooling bumps would
  otherwise skip the model init jobs.
- `vllm/logging_utils/`: imported by `vllm/model_executor/models/registry.py`
  (`logtime`) and `vllm/v1/engine/core.py`.

Applied to GPU jobs (Initialization, Other). The CPU job
(test_utils, test_vision) does not transitively reach these
modules — its tests only import `vllm.distributed`,
`vllm.model_executor.models.{utils,vision}`, `vllm.platforms`,
`vllm.utils`, none of which import the new edge modules — so
the CPU job is unchanged.

Signed-off-by: Kevin H. Luu <kevin@anyscale.com>

Signed-off-by: khluu <khluu000@gmail.com>
2026-05-10 03:48:11 -07:00
khluu 8ac682b513 Add sequence.py and config/ to models_basic deps
Address review on #42220:

- Add `vllm/sequence.py` to GPU jobs (Initialization, Other) — it
  defines `IntermediateTensors` and other core types used in the
  model forward path; an incompatible change can break model
  initialization even without a direct test import.
- Add `vllm/config/` to the CPU job —
  `vllm/model_executor/models/vision.py` (imported by
  `tests/models/test_vision.py`) imports
  `MultiModalConfig` from `vllm.config`.

Skipped gemini's other suggestions: `vllm/core/` does not exist at
the top level (the relevant code lives under `vllm/v1/core/` which
is already covered by `vllm/v1/`), and `vllm/multimodal/` is not
directly imported by `test_utils.py` or `test_vision.py`
(`MultiModalConfig` is the config class, listed under `vllm/config/`).

Signed-off-by: Kevin H. Luu <kevin@anyscale.com>

Signed-off-by: khluu <khluu000@gmail.com>
2026-05-10 03:35:03 -07:00
khluu 1966261291 [CI] Narrow models_basic.yaml source dependencies
Three jobs in `.buildkite/test_areas/models_basic.yaml` listed
`vllm/` as a source dependency. Replace each with the modules the
underlying tests actually import:

- Basic Models Tests (Initialization): runs
  `tests/models/test_initialization.py`, which instantiates models
  via `vllm_runner`. Narrowed to the inference-stack modules used
  during model load.
- Basic Models Tests (Other): same narrowing as Initialization.
- Basic Models Test (Other CPU): unit tests
  (`test_utils.py`, `test_vision.py`) that only import
  `vllm.distributed`, `vllm.model_executor.models`, `vllm.platforms`,
  `vllm.utils` — narrowed to those four.

`Basic Models Tests (Extra Initialization)` already had a narrow
`vllm/model_executor/models/` dep and is unchanged. The optional
nightly transformers jobs have no `source_file_dependencies` and
are unchanged.

Signed-off-by: Kevin H. Luu <kevin@anyscale.com>

Signed-off-by: khluu <khluu000@gmail.com>
2026-05-10 03:27:42 -07:00
+35 -3
View File
@@ -7,7 +7,21 @@ steps:
timeout_in_minutes: 45
torch_nightly: true
source_file_dependencies:
- vllm/
- vllm/config/
- vllm/distributed/
- vllm/engine/
- vllm/inputs/
- vllm/logging_utils/
- vllm/model_executor/
- vllm/multimodal/
- vllm/platforms/
- vllm/sampling_params.py
- vllm/sequence.py
- vllm/tasks.py
- vllm/transformers_utils/
- vllm/utils/
- vllm/v1/
- vllm/version.py
- tests/models/test_initialization.py
- tests/models/registry.py
commands:
@@ -36,7 +50,21 @@ steps:
key: basic-models-tests-other
timeout_in_minutes: 45
source_file_dependencies:
- vllm/
- vllm/config/
- vllm/distributed/
- vllm/engine/
- vllm/inputs/
- vllm/logging_utils/
- vllm/model_executor/
- vllm/multimodal/
- vllm/platforms/
- vllm/sampling_params.py
- vllm/sequence.py
- vllm/tasks.py
- vllm/transformers_utils/
- vllm/utils/
- vllm/v1/
- vllm/version.py
- tests/models/test_terratorch.py
- tests/models/test_transformers.py
- tests/models/test_registry.py
@@ -49,7 +77,11 @@ steps:
- image-build-cpu
timeout_in_minutes: 10
source_file_dependencies:
- vllm/
- vllm/config/
- vllm/distributed/
- vllm/model_executor/
- vllm/platforms/
- vllm/utils/
- tests/models/test_utils.py
- tests/models/test_vision.py
device: cpu-small