test_trtllm.py 15.4 KB
Newer Older
1
# SPDX-FileCopyrightText: Copyright (c) 2025-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2
3
# SPDX-License-Identifier: Apache-2.0

4
"""
5
6
7
8
9
Test Execution Times (Last Run: 2026-01-12):
- test_request_migration_trtllm_aggregated: ~95s
- test_request_migration_trtllm_prefill: N/A
- test_request_migration_trtllm_kv_transfer: N/A
- test_request_migration_trtllm_decode: N/A
10
11
"""

12
13
14
15
16
17
18
import logging
import os
import shutil

import pytest

from tests.utils.constants import FAULT_TOLERANCE_MODEL_NAME
19
from tests.utils.managed_process import ManagedProcess
20
from tests.utils.payloads import check_models_api
21
from tests.utils.port_utils import allocate_port, deallocate_port
22

23
24
# Customized utils for migration tests
from .utils import DynamoFrontendProcess, run_migration_test
25
26
27
28

logger = logging.getLogger(__name__)

pytestmark = [
29
    pytest.mark.fault_tolerance,
30
31
32
33
    pytest.mark.trtllm,
    pytest.mark.gpu_1,
    pytest.mark.e2e,
    pytest.mark.model(FAULT_TOLERANCE_MODEL_NAME),
34
35
36
37
    pytest.mark.parametrize(
        "migration_limit", [3, 0], ids=["migration_enabled", "migration_disabled"]
    ),
    pytest.mark.parametrize(
38
        "immediate_kill", [True, False], ids=["worker_failure", "graceful_shutdown"]
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
    ),
    pytest.mark.parametrize(
        "request_api",
        [
            pytest.param("chat"),
            pytest.param(
                "completion",
                marks=pytest.mark.skip(reason="Behavior unverified yet"),
            ),
        ],
    ),
    pytest.mark.parametrize(
        "stream",
        [
            pytest.param(True, id="stream"),
            pytest.param(
                False,
                id="unary",
                marks=pytest.mark.skip(reason="Behavior unverified yet"),
            ),
        ],
    ),
61
    pytest.mark.parametrize("request_plane", ["nats", "tcp"], indirect=True),
62
63
64
65
]


class DynamoWorkerProcess(ManagedProcess):
66
67
68
69
70
71
72
73
74
75
76
    """Process manager for Dynamo worker with TRT-LLM backend

    Supports both aggregated mode (single worker) and disaggregated mode
    (separate prefill and decode workers).

    Args:
        request: pytest request fixture
        worker_id: Unique identifier for the worker (e.g., "worker1", "prefill1")
        frontend_port: Port where the frontend is running
        mode: "prefill_and_decode" for aggregated, "prefill" or "decode" for disaggregated
    """
77

78
79
80
81
82
    def __init__(
        self,
        request,
        worker_id: str,
        frontend_port: int,
83
        mode: str = "prefill_and_decode",
84
    ):
85
        self.worker_id = worker_id
86
87
        self.system_port = allocate_port(9100)
        self.mode = mode
88

89
90
91
92
93
94
95
        command = [
            "python3",
            "-m",
            "dynamo.trtllm",
            "--model",
            FAULT_TOLERANCE_MODEL_NAME,
            "--disaggregation-mode",
96
            mode,
97
98
            "--max-seq-len",
            "8192",
99
100
101
102
            "--max-num-tokens",
            "8192",
            "--free-gpu-memory-fraction",
            "0.15",  # avoid validation error on TRT-LLM available memory checks
103
        ]
104
105
106
107
108
109
110
111
112
113
114
        if mode != "prefill_and_decode":
            config_file = (
                f"test_request_migration_trtllm_config_{self.system_port}.yaml"
            )
            with open(config_file, "w") as f:
                f.write(
                    "cache_transceiver_config:\n  backend: DEFAULT\n  max_tokens_in_buffer: 8192\n"
                )
                f.write("disable_overlap_scheduler: true\n")
                f.write("kv_cache_config:\n  max_tokens: 8192\n")
            command += ["--extra-engine-args", config_file]
115

116
        # Set environment variables
117
        env = os.environ.copy()
118
        env["DYN_REQUEST_PLANE"] = request.getfixturevalue("request_plane")
119

120
        env["DYN_LOG"] = "debug"
121
122
123
124
125
        # Disable canary health check - these tests expect full control over requests
        # sent to the workers where canary health check intermittently sends dummy
        # requests to workers interfering with the test process which may cause
        # intermittent failures
        env["DYN_HEALTH_CHECK_ENABLED"] = "false"
126
        env["DYN_SYSTEM_USE_ENDPOINT_HEALTH_STATUS"] = '["generate"]'
127
128
129
        env["DYN_SYSTEM_PORT"] = str(self.system_port)
        env["DYN_HTTP_PORT"] = str(frontend_port)

130
131
132
        # Disable backend shutdown grace period for all migration tests
        env["DYN_GRACEFUL_SHUTDOWN_GRACE_PERIOD_SECS"] = "0"

133
134
135
136
137
138
139
140
        # Configure health check based on worker type
        health_check_urls = [
            (f"http://localhost:{self.system_port}/health", self.is_ready)
        ]
        if mode in ["decode", "prefill_and_decode"]:
            health_check_urls.append(
                (f"http://localhost:{frontend_port}/v1/models", check_models_api)
            )
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156

        # TODO: Have the managed process take a command name explicitly to distinguish
        #       between processes started with the same command.
        log_dir = f"{request.node.name}_{worker_id}"

        # 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,
157
            health_check_urls=health_check_urls,
158
159
            timeout=300,
            display_output=True,
160
            terminate_all_matching_process_names=False,
161
            log_dir=log_dir,
162
            display_name=worker_id,
163
164
        )

165
166
167
168
169
170
171
172
173
    def __exit__(self, exc_type, exc_val, exc_tb):
        """Release allocated port when worker exits."""
        try:
            # system_port is always allocated in __init__
            deallocate_port(self.system_port)
        except Exception as e:
            logging.warning(f"Failed to release TRT-LLM worker port: {e}")

        return super().__exit__(exc_type, exc_val, exc_tb)
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189

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


Dmitry Tokarev's avatar
Dmitry Tokarev committed
190
191
@pytest.mark.timeout(290)  # 3x average
@pytest.mark.post_merge
192
193
194
195
196
197
198
199
200
def test_request_migration_trtllm_aggregated(
    request,
    runtime_services_dynamic_ports,
    set_ucx_tls_no_mm,
    predownload_models,
    migration_limit,
    immediate_kill,
    request_api,
    stream,
201
202
):
    """
203
    End-to-end test for aggregated worker request migration.
204

205
206
207
208
209
    Parameters:
        immediate_kill: True for abrupt kill (SIGKILL), False for graceful shutdown (SIGTERM)
        migration_limit: > 0 to verify migration succeeds, 0 to verify request fails
        request_api: "chat" for chat completion API, "completion" for completion API
        stream: True for streaming, False for non-streaming
210
211
    """

212
    # Step 1: Start the frontend
213
    with DynamoFrontendProcess(request, migration_limit=migration_limit) as frontend:
214
215
        logger.info("Frontend started successfully")

216
        # Step 2: Start 2 workers
217
        with DynamoWorkerProcess(request, "worker1", frontend.frontend_port) as worker1:
218
219
            logger.info(f"Worker 1 PID: {worker1.get_pid()}")

220
            with DynamoWorkerProcess(
221
222
223
                request,
                "worker2",
                frontend.frontend_port,
224
            ) as worker2:
225
226
                logger.info(f"Worker 2 PID: {worker2.get_pid()}")

227
228
229
230
231
                # Step 3: Run migration test
                run_migration_test(
                    frontend,
                    worker1,
                    worker2,
232
                    receiving_pattern="AggregatedHandler Request ID: ",
233
234
235
236
                    migration_limit=migration_limit,
                    immediate_kill=immediate_kill,
                    use_chat_completion=(request_api == "chat"),
                    stream=stream,
237
238
                )

239

240
241
242
@pytest.mark.skip(
    reason="Prefill migration not yet supported, XFail eats up CI time due to timeout"
)
243
@pytest.mark.timeout(350)  # 3x average
244
@pytest.mark.nightly
245
246
247
248
249
250
251
252
253
def test_request_migration_trtllm_prefill(
    request,
    runtime_services_dynamic_ports,
    set_ucx_tls_no_mm,
    predownload_models,
    migration_limit,
    immediate_kill,
    request_api,
    stream,
254
255
):
    """
256
257
258
259
260
261
262
263
264
    End-to-end test for prefill worker request migration in disaggregated mode.

    Setup: 1 decode worker + 2 prefill workers

    Parameters:
        immediate_kill: True for abrupt kill (SIGKILL), False for graceful shutdown (SIGTERM)
        migration_limit: > 0 to verify migration succeeds, 0 to verify request fails
        request_api: "chat" for chat completion API, "completion" for completion API
        stream: True for streaming, False for non-streaming
265
266
    """

267
    # Step 1: Start the frontend
268
    with DynamoFrontendProcess(request, migration_limit=migration_limit) as frontend:
269
270
        logger.info("Frontend started successfully")

271
272
273
274
275
276
277
278
        # Step 2: Start decode worker first (required for prefill workers to connect)
        with DynamoWorkerProcess(
            request,
            "worker0",
            frontend.frontend_port,
            mode="decode",
        ) as decode_worker:
            logger.info(f"Decode Worker PID: {decode_worker.get_pid()}")
279

280
            # Step 3: Start 2 prefill workers
281
            with DynamoWorkerProcess(
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
                request,
                "worker1",
                frontend.frontend_port,
                mode="prefill",
            ) as prefill1:
                logger.info(f"Prefill Worker 1 PID: {prefill1.get_pid()}")

                with DynamoWorkerProcess(
                    request,
                    "worker2",
                    frontend.frontend_port,
                    mode="prefill",
                ) as prefill2:
                    logger.info(f"Prefill Worker 2 PID: {prefill2.get_pid()}")

                    # Step 4: Run migration test
                    run_migration_test(
                        frontend,
                        prefill1,
                        prefill2,
                        receiving_pattern="Prefill Request ID: ",
                        migration_limit=migration_limit,
                        immediate_kill=immediate_kill,
                        use_chat_completion=(request_api == "chat"),
                        stream=stream,
                        use_long_prompt=True,
                    )
309

310

311
312
@pytest.mark.skip(reason="Decode worker can get stuck downloading kv cache")
@pytest.mark.timeout(350)  # 3x average
313
@pytest.mark.nightly
314
315
316
317
318
319
320
321
322
def test_request_migration_trtllm_kv_transfer(
    request,
    runtime_services_dynamic_ports,
    set_ucx_tls_no_mm,
    predownload_models,
    migration_limit,
    immediate_kill,
    request_api,
    stream,
323
324
):
    """
325
    End-to-end test for request migration during KV transfer in disaggregated mode.
326

327
    Setup: 1 prefill worker + 2 decode workers
328

329
330
331
332
333
    Parameters:
        immediate_kill: True for abrupt kill (SIGKILL), False for graceful shutdown (SIGTERM)
        migration_limit: > 0 to verify migration succeeds, 0 to verify request fails
        request_api: "chat" for chat completion API, "completion" for completion API
        stream: True for streaming, False for non-streaming
334
335
    """

336
    # Step 1: Start the frontend
337
    with DynamoFrontendProcess(request, migration_limit=migration_limit) as frontend:
338
339
        logger.info("Frontend started successfully")

340
        # Step 2: Start prefill worker first
341
342
        with DynamoWorkerProcess(
            request,
343
            "worker0",
344
            frontend.frontend_port,
345
346
347
            mode="prefill",
        ) as prefill_worker:
            logger.info(f"Prefill Worker PID: {prefill_worker.get_pid()}")
348

349
            # Step 3: Start 2 decode workers
350
351
            with DynamoWorkerProcess(
                request,
352
                "worker1",
353
                frontend.frontend_port,
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
                mode="decode",
            ) as decode1:
                logger.info(f"Decode Worker 1 PID: {decode1.get_pid()}")

                with DynamoWorkerProcess(
                    request,
                    "worker2",
                    frontend.frontend_port,
                    mode="decode",
                ) as decode2:
                    logger.info(f"Decode Worker 2 PID: {decode2.get_pid()}")

                    # Step 4: Run migration test
                    run_migration_test(
                        frontend,
                        decode1,
                        decode2,
                        receiving_pattern="Decode Request ID: ",
                        migration_limit=migration_limit,
                        immediate_kill=immediate_kill,
                        use_chat_completion=(request_api == "chat"),
                        stream=stream,
                        use_long_prompt=True,
377
378
379
                    )


380
@pytest.mark.timeout(350)  # 3x average
381
@pytest.mark.post_merge
382
383
384
385
386
387
388
389
390
def test_request_migration_trtllm_decode(
    request,
    runtime_services_dynamic_ports,
    set_ucx_tls_no_mm,
    predownload_models,
    migration_limit,
    immediate_kill,
    request_api,
    stream,
391
392
):
    """
393
    End-to-end test for decode worker request migration in disaggregated mode.
394

395
    Setup: 1 prefill worker + 2 decode workers
396

397
398
399
400
401
    Parameters:
        immediate_kill: True for abrupt kill (SIGKILL), False for graceful shutdown (SIGTERM)
        migration_limit: > 0 to verify migration succeeds, 0 to verify request fails
        request_api: "chat" for chat completion API, "completion" for completion API
        stream: True for streaming, False for non-streaming
402
    """
403
404
405
406
    if not stream:
        pytest.skip(
            "Decode test requires streaming to wait for response before stopping worker"
        )
407

408
    # Step 1: Start the frontend
409
    with DynamoFrontendProcess(request, migration_limit=migration_limit) as frontend:
410
411
        logger.info("Frontend started successfully")

412
        # Step 2: Start prefill worker first
413
414
        with DynamoWorkerProcess(
            request,
415
            "worker0",
416
            frontend.frontend_port,
417
418
419
            mode="prefill",
        ) as prefill_worker:
            logger.info(f"Prefill Worker PID: {prefill_worker.get_pid()}")
420

421
            # Step 3: Start 2 decode workers
422
423
            with DynamoWorkerProcess(
                request,
424
                "worker1",
425
                frontend.frontend_port,
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
                mode="decode",
            ) as decode1:
                logger.info(f"Decode Worker 1 PID: {decode1.get_pid()}")

                with DynamoWorkerProcess(
                    request,
                    "worker2",
                    frontend.frontend_port,
                    mode="decode",
                ) as decode2:
                    logger.info(f"Decode Worker 2 PID: {decode2.get_pid()}")

                    # Step 4: Run migration test
                    run_migration_test(
                        frontend,
                        decode1,
                        decode2,
                        receiving_pattern="Decode Request ID: ",
                        migration_limit=migration_limit,
                        immediate_kill=immediate_kill,
                        use_chat_completion=(request_api == "chat"),
                        stream=stream,
                        wait_for_new_response_before_stop=True,
449
                    )