Skip to content

Commit 883c4a6

Browse files
committed
test: update metrics count expectation after refactoring
Update the minimum expected unique dynamo_component_* metrics from 23 to 11 to reflect the current metric count. This fixes the failing vllm e2e tests: - test_serve_deployment[aggregated] - test_serve_deployment[aggregated_lmcache] Signed-off-by: Keiven Chang <keivenchang@users.noreply.github.com>
1 parent 5b8e3a0 commit 883c4a6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

tests/utils/payloads.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -274,9 +274,9 @@ def metric_pattern(name):
274274
name=f"{prefix}_*",
275275
pattern=lambda name: rf"^{prefix}_\w+",
276276
validator=lambda value: len(set(value))
277-
>= 23, # 80% of typical ~29 metrics (excluding _bucket) as of 2025-10-22 (but will grow)
278-
error_msg=lambda name, value: f"Expected at least 23 unique {prefix}_* metrics, but found only {len(set(value))}",
279-
success_msg=lambda name, value: f"SUCCESS: Found {len(set(value))} unique {prefix}_* metrics (minimum required: 23)",
277+
>= 11, # 80% of typical ~17 metrics (excluding _bucket) as of 2025-12-02
278+
error_msg=lambda name, value: f"Expected at least 11 unique {prefix}_* metrics, but found only {len(set(value))}",
279+
success_msg=lambda name, value: f"SUCCESS: Found {len(set(value))} unique {prefix}_* metrics (minimum required: 11)",
280280
multiline=True,
281281
),
282282
MetricCheck(

0 commit comments

Comments
 (0)