test_vllm.py 12.8 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
            "--enforce-eager",
            "--gpu-memory-utilization",
            "0.45",
            "--max-model-len",
            "8192",
            "--migration-limit",
            "3",
        ]

43
44
45
46
        health_check_urls = [
            (f"http://localhost:{FRONTEND_PORT}/v1/models", check_models_api),
            (f"http://localhost:{FRONTEND_PORT}/health", check_health_generate),
        ]
47
48
49
50

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

51
52
53
54
55
        # Add prefill worker flag if needed
        if is_prefill:
            command.append("--is-prefill-worker")
            health_check_urls = [(f"http://localhost:{port}/health", self.is_ready)]

56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
        # Set debug logging environment
        env = os.environ.copy()
        env["DYN_LOG"] = "debug"
        env["DYN_SYSTEM_ENABLED"] = "true"
        env["DYN_SYSTEM_USE_ENDPOINT_HEALTH_STATUS"] = '["generate"]'
        env["DYN_SYSTEM_PORT"] = port

        # 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,
78
            health_check_urls=health_check_urls,
79
80
81
            timeout=300,
            display_output=True,
            terminate_existing=False,
82
83
84
85
86
87
88
            # Ensure any orphaned vLLM engine cores or child helpers are cleaned up
            stragglers=[
                "VLLM::EngineCore",
            ],
            straggler_commands=[
                "-m dynamo.vllm",
            ],
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
            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
117
@pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME)
118
119
120
def test_request_cancellation_vllm_aggregated(
    request, runtime_services, predownload_models
):
121
    """
122
    End-to-end test for request cancellation functionality in aggregated mode.
123
124
125

    This test verifies that when a request is cancelled by the client,
    the system properly handles the cancellation and cleans up resources
126
    on the worker side in aggregated (single worker) mode. Tests three scenarios:
127
128
129
130
131
132
133
134
135
136
    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
137
        with DynamoWorkerProcess(request) as worker:
138
139
            logger.info(f"Worker PID: {worker.get_pid()}")

140
            # Step 3: Test request cancellation with polling approach
141
142
143
144
145
146
147
148
149
150
151
            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",
                ),
            ]

152
            for request_type, description in test_scenarios:
153
154
                logger.info(f"Testing {description.lower()}...")

155
156
157
158
159
160
161
162
163
                # Send the request (non-blocking)
                cancellable_req = send_cancellable_request(request_type)

                # Poll for "New Request ID" pattern
                request_id, worker_log_offset = poll_for_pattern(
                    process=worker,
                    pattern="New Request ID: ",
                    log_offset=worker_log_offset,
                    match_type="contains",
164
                )
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185

                # 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,
186
187
188
189
190
191
192
193
                )

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


@pytest.mark.vllm
@pytest.mark.gpu_1
@pytest.mark.e2e
Alec's avatar
Alec committed
194
@pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME)
195
def test_request_cancellation_vllm_decode_cancel(
Alec's avatar
Alec committed
196
    request, runtime_services, predownload_models, set_ucx_tls_no_mm
Alec's avatar
Alec committed
197
):
198
    """
199
    End-to-end test for request cancellation during decode phase.
200

201
    This test verifies that when a request is cancelled by the client during the decode phase,
202
203
204
205
206
207
208
209
210
    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
211
        with DynamoWorkerProcess(request, is_prefill=True) as prefill_worker:
212
213
214
            logger.info(f"Prefill Worker PID: {prefill_worker.get_pid()}")

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

218
                # Step 4: Test request cancellation for streaming scenario
219
                logger.info(
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
                    "Testing chat completion stream request cancellation in decode worker (decode phase)..."
                )

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

                # Poll for "New Request ID" pattern in decode worker
                request_id, decode_log_offset = poll_for_pattern(
                    process=decode_worker,
                    pattern="New Request ID: ",
                    match_type="contains",
                )

                # Verify same request ID reached prefill worker (as "New Prefill Request ID")
                _, prefill_log_offset = poll_for_pattern(
                    process=prefill_worker,
                    pattern=f"New Prefill Request ID: {request_id}",
                )

                # 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",
257
258
259
                )

                logger.info(
260
                    "Chat completion stream cancellation in decode phase detected successfully"
261
262
263
264
265
266
                )


@pytest.mark.vllm
@pytest.mark.gpu_1
@pytest.mark.e2e
267
@pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME)
268
def test_request_cancellation_vllm_remote_prefill_cancel(
Alec's avatar
Alec committed
269
    request, runtime_services, predownload_models, set_ucx_tls_no_mm
270
):
271
    """
272
    End-to-end test for request cancellation during remote prefill phase.
273

274
275
276
    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.
277
    """
278
279
280
281
282
283

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

        # Step 2: Start the prefill worker
284
        with DynamoWorkerProcess(request, is_prefill=True) as prefill_worker:
285
286
287
            logger.info(f"Prefill Worker PID: {prefill_worker.get_pid()}")

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

291
                # Step 4: Test request cancellation during remote prefill phase
292
                logger.info(
293
                    "Testing completion request cancellation during remote prefill phase..."
294
295
                )

296
297
298
                # Send request with long prompt (non-blocking)
                cancellable_req = send_cancellable_request(
                    "completion", use_long_prompt=True
299
                )
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339

                # Poll for "New Request ID" pattern in decode worker
                request_id, decode_log_offset = poll_for_pattern(
                    process=decode_worker,
                    pattern="New Request ID: ",
                    match_type="contains",
                )

                # Poll for same request ID in prefill worker (as "New Prefill Request ID")
                _, prefill_log_offset = poll_for_pattern(
                    process=prefill_worker,
                    pattern=f"New Prefill Request ID: {request_id}",
                )

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

                # Poll for "Aborted Prefill Request ID" in prefill worker first (where cancellation happens)
                _, prefill_log_offset = poll_for_pattern(
                    process=prefill_worker,
                    pattern=f"Aborted Prefill Request ID: {request_id}",
                    log_offset=prefill_log_offset,
                )

                # Then poll for "Aborted Remote Prefill Request ID" in decode worker
                _, decode_log_offset = poll_for_pattern(
                    process=decode_worker,
                    pattern=f"Aborted Remote Prefill 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",
                )

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