test_vllm.py 13 KB
Newer Older
1
2
3
4
5
6
7
8
9
# SPDX-FileCopyrightText: Copyright (c) 2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# SPDX-License-Identifier: Apache-2.0

import logging
import os
import shutil

import pytest

10
11
from tests.fault_tolerance.cancellation.utils import (
    DynamoFrontendProcess,
12
13
14
    poll_for_pattern,
    read_streaming_responses,
    send_cancellable_request,
15
)
Alec's avatar
Alec committed
16
from tests.utils.constants import FAULT_TOLERANCE_MODEL_NAME
17
from tests.utils.engine_process import FRONTEND_PORT
18
from tests.utils.managed_process import ManagedProcess
19
from tests.utils.payloads import check_health_generate, check_models_api
20
21
22
23
24
25
26
27
28
29
30
31
32

logger = logging.getLogger(__name__)


class DynamoWorkerProcess(ManagedProcess):
    """Process manager for Dynamo worker with vLLM backend"""

    def __init__(self, request, is_prefill: bool = False):
        command = [
            "python3",
            "-m",
            "dynamo.vllm",
            "--model",
Alec's avatar
Alec committed
33
            FAULT_TOLERANCE_MODEL_NAME,
34
35
36
37
38
39
40
41
42
43
44
45
            "--enforce-eager",
            "--gpu-memory-utilization",
            "0.45",
            "--max-model-len",
            "8192",
            "--migration-limit",
            "3",
        ]

        # Set port based on worker type
        port = "8082" if is_prefill else "8081"

46
        # Configure health check based on worker type
47
        if is_prefill:
48
            # Prefill workers check their own status endpoint
49
50
            command.append("--is-prefill-worker")
            health_check_urls = [(f"http://localhost:{port}/health", self.is_ready)]
51
52
53
54
55
56
57
58
        else:
            # Decode workers should also check their own status endpoint first,
            # then verify the frontend sees the model
            health_check_urls = [
                (f"http://localhost:{port}/health", self.is_ready),
                (f"http://localhost:{FRONTEND_PORT}/v1/models", check_models_api),
                (f"http://localhost:{FRONTEND_PORT}/health", check_health_generate),
            ]
59

60
61
62
63
64
65
        # Set debug logging environment
        env = os.environ.copy()
        env["DYN_LOG"] = "debug"
        env["DYN_SYSTEM_USE_ENDPOINT_HEALTH_STATUS"] = '["generate"]'
        env["DYN_SYSTEM_PORT"] = port

66
67
68
69
70
71
        # Set KV event port and NIXL side channel port only for prefill worker
        # to avoid conflicts with decode worker
        if is_prefill:
            env["DYN_VLLM_KV_EVENT_PORT"] = "20082"
            env["VLLM_NIXL_SIDE_CHANNEL_PORT"] = "5601"

72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
        # Set log directory based on worker type
        worker_type = "prefill_worker" if is_prefill else "worker"
        log_dir = f"{request.node.name}_{worker_type}"

        # Clean up any existing log directory from previous runs
        try:
            shutil.rmtree(log_dir)
            logger.info(f"Cleaned up existing log directory: {log_dir}")
        except FileNotFoundError:
            # Directory doesn't exist, which is fine
            pass

        super().__init__(
            command=command,
            env=env,
87
            health_check_urls=health_check_urls,
88
89
90
            timeout=300,
            display_output=True,
            terminate_existing=False,
91
92
93
94
95
96
97
            # Ensure any orphaned vLLM engine cores or child helpers are cleaned up
            stragglers=[
                "VLLM::EngineCore",
            ],
            straggler_commands=[
                "-m dynamo.vllm",
            ],
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
            log_dir=log_dir,
        )

        self.is_prefill = is_prefill

    def get_pid(self):
        """Get the PID of the worker process"""
        return self.proc.pid if self.proc else None

    def is_ready(self, response) -> bool:
        """Check the health of the worker process"""
        try:
            data = response.json()
            if data.get("status") == "ready":
                worker_type = "Prefill worker" if self.is_prefill else "Worker"
                logger.info(f"{worker_type} status is ready")
                return True
            worker_type = "Prefill worker" if self.is_prefill else "Worker"
            logger.warning(f"{worker_type} status is not ready: {data.get('status')}")
        except ValueError:
            worker_type = "Prefill worker" if self.is_prefill else "Worker"
            logger.warning(f"{worker_type} health response is not valid JSON")
        return False


@pytest.mark.vllm
@pytest.mark.gpu_1
@pytest.mark.e2e
Alec's avatar
Alec committed
126
@pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME)
127
128
129
def test_request_cancellation_vllm_aggregated(
    request, runtime_services, predownload_models
):
130
    """
131
    End-to-end test for request cancellation functionality in aggregated mode.
132
133
134

    This test verifies that when a request is cancelled by the client,
    the system properly handles the cancellation and cleans up resources
135
    on the worker side in aggregated (single worker) mode. Tests three scenarios:
136
137
138
139
140
141
142
143
144
145
    1. Completion request
    2. Chat completion request (non-streaming)
    3. Chat completion request (streaming)
    """

    # Step 1: Start the frontend
    with DynamoFrontendProcess(request) as frontend:
        logger.info("Frontend started successfully")

        # Step 2: Start a single worker
146
        with DynamoWorkerProcess(request) as worker:
147
148
            logger.info(f"Worker PID: {worker.get_pid()}")

149
            # Step 3: Test request cancellation with polling approach
150
151
152
153
154
155
156
157
158
159
160
            frontend_log_offset, worker_log_offset = 0, 0

            test_scenarios = [
                ("completion", "Completion request cancellation"),
                ("chat_completion", "Chat completion request cancellation"),
                (
                    "chat_completion_stream",
                    "Chat completion stream request cancellation",
                ),
            ]

161
            for request_type, description in test_scenarios:
162
163
                logger.info(f"Testing {description.lower()}...")

164
165
166
                # Send the request (non-blocking)
                cancellable_req = send_cancellable_request(request_type)

167
                # Poll for "Decode Request ID" pattern (vLLM v2 pattern)
168
169
                request_id, worker_log_offset = poll_for_pattern(
                    process=worker,
170
                    pattern="Decode Request ID: ",
171
172
                    log_offset=worker_log_offset,
                    match_type="contains",
173
                )
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194

                # For streaming, read 5 responses before cancelling
                if request_type == "chat_completion_stream":
                    read_streaming_responses(cancellable_req, expected_count=5)

                # Now cancel the request
                cancellable_req.cancel()
                logger.info(f"Cancelled request ID: {request_id}")

                # Poll for "Aborted Request ID" with matching ID
                _, worker_log_offset = poll_for_pattern(
                    process=worker,
                    pattern=f"Aborted Request ID: {request_id}",
                    log_offset=worker_log_offset,
                )

                # Verify frontend log has kill message
                _, frontend_log_offset = poll_for_pattern(
                    process=frontend,
                    pattern="issued control message Kill to sender",
                    log_offset=frontend_log_offset,
195
196
197
198
199
200
201
202
                )

                logger.info(f"{description} detected successfully")


@pytest.mark.vllm
@pytest.mark.gpu_1
@pytest.mark.e2e
Alec's avatar
Alec committed
203
@pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME)
204
def test_request_cancellation_vllm_decode_cancel(
Alec's avatar
Alec committed
205
    request, runtime_services, predownload_models, set_ucx_tls_no_mm
Alec's avatar
Alec committed
206
):
207
    """
208
    End-to-end test for request cancellation during decode phase.
209

210
    This test verifies that when a request is cancelled by the client during the decode phase,
211
212
213
214
215
216
217
218
219
    the system properly handles the cancellation and cleans up resources
    on the decode worker side in a disaggregated setup.
    """

    # Step 1: Start the frontend
    with DynamoFrontendProcess(request) as frontend:
        logger.info("Frontend started successfully")

        # Step 2: Start the prefill worker
220
        with DynamoWorkerProcess(request, is_prefill=True) as prefill_worker:
221
222
223
            logger.info(f"Prefill Worker PID: {prefill_worker.get_pid()}")

            # Step 3: Start the decode worker
224
            with DynamoWorkerProcess(request, is_prefill=False) as decode_worker:
225
226
                logger.info(f"Decode Worker PID: {decode_worker.get_pid()}")

227
                # Step 4: Test request cancellation for streaming scenario
228
                logger.info(
229
230
231
232
233
234
                    "Testing chat completion stream request cancellation in decode worker (decode phase)..."
                )

                # Send streaming request (non-blocking)
                cancellable_req = send_cancellable_request("chat_completion_stream")

235
                # Poll for "Decode Request ID" pattern in decode worker (vLLM v2 pattern)
236
237
                request_id, decode_log_offset = poll_for_pattern(
                    process=decode_worker,
238
                    pattern="Decode Request ID: ",
239
240
241
                    match_type="contains",
                )

242
                # Verify same request ID reached prefill worker (as "Prefill Request ID")
243
244
                _, prefill_log_offset = poll_for_pattern(
                    process=prefill_worker,
245
                    pattern=f"Prefill Request ID: {request_id}",
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
                )

                # Read 5 streaming responses (decode phase)
                read_streaming_responses(cancellable_req, expected_count=5)

                # Now cancel the request
                cancellable_req.cancel()
                logger.info(f"Cancelled request ID: {request_id}")

                # Poll for "Aborted Request ID" in decode worker
                _, decode_log_offset = poll_for_pattern(
                    process=decode_worker,
                    pattern=f"Aborted Request ID: {request_id}",
                    log_offset=decode_log_offset,
                )

                # Verify frontend log has kill message
                _, frontend_log_offset = poll_for_pattern(
                    process=frontend,
                    pattern="issued control message Kill to sender",
266
267
268
                )

                logger.info(
269
                    "Chat completion stream cancellation in decode phase detected successfully"
270
271
272
273
274
275
                )


@pytest.mark.vllm
@pytest.mark.gpu_1
@pytest.mark.e2e
276
@pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME)
277
def test_request_cancellation_vllm_remote_prefill_cancel(
Alec's avatar
Alec committed
278
    request, runtime_services, predownload_models, set_ucx_tls_no_mm
279
):
280
    """
281
    End-to-end test for request cancellation during remote prefill phase.
282

283
284
285
    This test verifies that when a request is cancelled by the client during the remote prefill phase,
    the system properly handles the cancellation and cleans up resources
    on both the decode and prefill workers in a disaggregated setup.
286
    """
287
288
289
290
291
292

    # Step 1: Start the frontend
    with DynamoFrontendProcess(request) as frontend:
        logger.info("Frontend started successfully")

        # Step 2: Start the prefill worker
293
        with DynamoWorkerProcess(request, is_prefill=True) as prefill_worker:
294
295
296
            logger.info(f"Prefill Worker PID: {prefill_worker.get_pid()}")

            # Step 3: Start the decode worker
297
            with DynamoWorkerProcess(request, is_prefill=False) as decode_worker:
298
299
                logger.info(f"Decode Worker PID: {decode_worker.get_pid()}")

300
301
302
                # Step 4: Test request cancellation during prefill phase
                # Note: With the new architecture, prefill routing happens in the frontend,
                # so the request goes directly to the prefill worker first
303
                logger.info(
304
                    "Testing completion request cancellation during prefill phase..."
305
306
                )

307
308
309
                # Send request with long prompt (non-blocking)
                cancellable_req = send_cancellable_request(
                    "completion", use_long_prompt=True
310
                )
311

312
313
314
                # Poll for "Prefill Request ID" pattern in prefill worker (vLLM v2 pattern)
                # With new architecture, prefill is routed by frontend's internal router
                request_id, prefill_log_offset = poll_for_pattern(
315
                    process=prefill_worker,
316
317
                    pattern="Prefill Request ID: ",
                    match_type="contains",
318
319
320
321
                )

                # Cancel during prefill phase
                cancellable_req.cancel()
322
                logger.info(f"Cancelled request ID: {request_id} during prefill")
323

324
                # Poll for "Aborted Prefill Request ID" in prefill worker (where cancellation happens)
325
326
327
328
329
330
331
332
333
334
335
336
337
338
                _, prefill_log_offset = poll_for_pattern(
                    process=prefill_worker,
                    pattern=f"Aborted Prefill Request ID: {request_id}",
                    log_offset=prefill_log_offset,
                )

                # Verify frontend log has kill message
                _, frontend_log_offset = poll_for_pattern(
                    process=frontend,
                    pattern="issued control message Kill to sender",
                )

                logger.info(
                    "Completion request cancellation during remote prefill phase detected successfully"
339
                )