Skip to content

Commit 2669a0d

Browse files
authored
Fix ValueError: Missing value for tag key(s): model_name,engine. (vllm-project#19113)
Signed-off-by: Seiji Eicher <[email protected]>
1 parent 8e972d9 commit 2669a0d

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

tests/v1/metrics/test_ray_metrics.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,12 +47,15 @@ async def run(self):
4747
engine_args, stat_loggers=[RayPrometheusStatLogger])
4848

4949
for i, prompt in enumerate(example_prompts):
50-
engine.generate(
50+
results = engine.generate(
5151
request_id=f"request-id-{i}",
5252
prompt=prompt,
5353
sampling_params=SamplingParams(max_tokens=max_tokens),
5454
)
5555

56+
async for _ in results:
57+
pass
58+
5659
# Create the actor and call the async method
5760
actor = EngineTestActor.remote() # type: ignore[attr-defined]
5861
ray.get(actor.run.remote())

vllm/v1/metrics/ray_wrappers.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,16 @@ def labels(self, *labels, **labelskwargs):
3131

3232
self.metric.set_default_tags(labelskwargs)
3333

34+
if labels:
35+
if len(labels) != len(self.metric._tag_keys):
36+
raise ValueError(
37+
"Number of labels must match the number of tag keys. "
38+
f"Expected {len(self.metric._tag_keys)}, got {len(labels)}"
39+
)
40+
41+
self.metric.set_default_tags(
42+
dict(zip(self.metric._tag_keys, labels)))
43+
3444
return self
3545

3646

0 commit comments

Comments
 (0)