Skip to content

Commit 1a76529

Browse files
committed
[Build/CI] Add tracing deps to vllm container image
A report requested that these dependencies be included in the image for tracing. That seems reasonable - they are needed by `vllm.tracing`. Stop manually installing them in the relevant test pipeline and just include them in the image. Signed-off-by: Russell Bryant <[email protected]>
1 parent 098900d commit 1a76529

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

.buildkite/test-pipeline.yaml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -163,11 +163,6 @@ steps:
163163
- tests/tracing
164164
commands:
165165
- pytest -v -s metrics
166-
- "pip install \
167-
'opentelemetry-sdk>=1.26.0,<1.27.0' \
168-
'opentelemetry-api>=1.26.0,<1.27.0' \
169-
'opentelemetry-exporter-otlp>=1.26.0,<1.27.0' \
170-
'opentelemetry-semantic-conventions-ai>=0.4.1,<0.5.0'"
171166
- pytest -v -s tracing
172167

173168
##### fast check tests #####

requirements/common.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,7 @@ watchfiles # required for http server to monitor the updates of TLS files
4343
python-json-logger # Used by logging as per examples/other/logging_configuration.md
4444
scipy # Required for phi-4-multimodal-instruct
4545
ninja # Required for xgrammar, rocm, tpu, xpu
46+
opentelemetry-sdk>=1.26.0,<1.27.0 # vllm.tracing
47+
opentelemetry-api>=1.26.0,<1.27.0 # vllm.tracing
48+
opentelemetry-exporter-otlp>=1.26.0,<1.27.0 # vllm.tracing
49+
opentelemetry-semantic-conventions-ai>=0.4.1,<0.5.0 # vllm.tracing

0 commit comments

Comments
 (0)