http-service.rs 40.2 KB
Newer Older
1
// SPDX-FileCopyrightText: Copyright (c) 2024-2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
2
3
4
5
// SPDX-License-Identifier: Apache-2.0

use anyhow::Error;
use async_stream::stream;
6
use dynamo_async_openai::config::OpenAIConfig;
Neelay Shah's avatar
Neelay Shah committed
7
use dynamo_llm::protocols::{
8
    Annotated,
Ryan Olson's avatar
Ryan Olson committed
9
10
    codec::SseLineCodec,
    convert_sse_stream,
11
    openai::{
12
        chat_completions::{NvCreateChatCompletionRequest, NvCreateChatCompletionStreamResponse},
13
        completions::{NvCreateCompletionRequest, NvCreateCompletionResponse},
14
15
    },
};
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
use dynamo_llm::{
    http::{
        client::{
            GenericBYOTClient, HttpClientConfig, HttpRequestContext, NvCustomClient,
            PureOpenAIClient,
        },
        service::{
            Metrics,
            error::HttpError,
            metrics::{Endpoint, RequestType, Status},
            service_v2::HttpService,
        },
    },
    model_card::ModelDeploymentCard,
};
31
use dynamo_runtime::metrics::prometheus_names::{frontend_service, name_prefix};
Neelay Shah's avatar
Neelay Shah committed
32
use dynamo_runtime::{
33
    CancellationToken,
34
    engine::AsyncEngineContext,
35
    pipeline::{
36
        AsyncEngine, AsyncEngineContextProvider, ManyOut, ResponseStream, SingleIn, async_trait,
37
38
    },
};
39
use futures::StreamExt;
40
use prometheus::{Registry, proto::MetricType};
41
use reqwest::StatusCode;
Ryan Olson's avatar
Ryan Olson committed
42
43
44
use std::{io::Cursor, sync::Arc};
use tokio::time::timeout;
use tokio_util::codec::FramedRead;
45

46
47
48
49
#[path = "common/ports.rs"]
mod ports;
use ports::get_random_port;

50
51
struct CounterEngine {}

Ryan Olson's avatar
Ryan Olson committed
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
// Add a new long-running test engine
struct LongRunningEngine {
    delay_ms: u64,
    cancelled: Arc<std::sync::atomic::AtomicBool>,
}

impl LongRunningEngine {
    fn new(delay_ms: u64) -> Self {
        Self {
            delay_ms,
            cancelled: Arc::new(std::sync::atomic::AtomicBool::new(false)),
        }
    }

    fn was_cancelled(&self) -> bool {
        self.cancelled.load(std::sync::atomic::Ordering::Acquire)
    }
}

71
72
73
#[async_trait]
impl
    AsyncEngine<
74
        SingleIn<NvCreateChatCompletionRequest>,
75
        ManyOut<Annotated<NvCreateChatCompletionStreamResponse>>,
76
77
78
79
80
        Error,
    > for CounterEngine
{
    async fn generate(
        &self,
81
        request: SingleIn<NvCreateChatCompletionRequest>,
82
    ) -> Result<ManyOut<Annotated<NvCreateChatCompletionStreamResponse>>, Error> {
83
84
85
        let (request, context) = request.transfer(());
        let ctx = context.context();

Paul Hendricks's avatar
Paul Hendricks committed
86
        // ALLOW: max_tokens is deprecated in favor of completion_usage_tokens
Ryan Olson's avatar
Ryan Olson committed
87
        #[allow(deprecated)]
Paul Hendricks's avatar
Paul Hendricks committed
88
        let max_tokens = request.inner.max_tokens.unwrap_or(0) as u64;
89

90
        // let generator = NvCreateChatCompletionStreamResponse::generator(request.model.clone());
91
        let mut generator = request.response_generator(ctx.id().to_string());
92
93
94
95

        let stream = stream! {
            tokio::time::sleep(std::time::Duration::from_millis(max_tokens)).await;
            for i in 0..10 {
96
                let output = generator.create_choice(i, Some(format!("choice {i}")), None, None, None);
Paul Hendricks's avatar
Paul Hendricks committed
97
98

                yield Annotated::from_data(output);
99
100
101
102
103
104
105
            }
        };

        Ok(ResponseStream::new(Box::pin(stream), ctx))
    }
}

Ryan Olson's avatar
Ryan Olson committed
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
#[async_trait]
impl
    AsyncEngine<
        SingleIn<NvCreateChatCompletionRequest>,
        ManyOut<Annotated<NvCreateChatCompletionStreamResponse>>,
        Error,
    > for LongRunningEngine
{
    async fn generate(
        &self,
        request: SingleIn<NvCreateChatCompletionRequest>,
    ) -> Result<ManyOut<Annotated<NvCreateChatCompletionStreamResponse>>, Error> {
        let (_request, context) = request.transfer(());
        let ctx = context.context();

        tracing::info!(
            "LongRunningEngine: Starting generation with {}ms delay",
            self.delay_ms
        );

        let cancelled_flag = self.cancelled.clone();
        let delay_ms = self.delay_ms;

        let ctx_clone = ctx.clone();
        let stream = async_stream::stream! {

            // the stream can be dropped or it can be cancelled
            // either way we consider this a cancellation
            cancelled_flag.store(true, std::sync::atomic::Ordering::SeqCst);

            tokio::select! {
                _ = tokio::time::sleep(std::time::Duration::from_millis(delay_ms)) => {
                    // the stream went to completion
                    cancelled_flag.store(false, std::sync::atomic::Ordering::SeqCst);

                }
                _ = ctx_clone.stopped() => {
                    cancelled_flag.store(true, std::sync::atomic::Ordering::SeqCst);
                }
            }

            yield Annotated::<NvCreateChatCompletionStreamResponse>::from_annotation("event.dynamo.test.sentinel", &"DONE".to_string()).expect("Failed to create annotated response");
        };

        Ok(ResponseStream::new(Box::pin(stream), ctx))
    }
}

154
155
156
157
158
struct AlwaysFailEngine {}

#[async_trait]
impl
    AsyncEngine<
159
        SingleIn<NvCreateChatCompletionRequest>,
160
        ManyOut<Annotated<NvCreateChatCompletionStreamResponse>>,
161
162
163
164
165
        Error,
    > for AlwaysFailEngine
{
    async fn generate(
        &self,
166
        _request: SingleIn<NvCreateChatCompletionRequest>,
167
    ) -> Result<ManyOut<Annotated<NvCreateChatCompletionStreamResponse>>, Error> {
168
169
170
171
172
173
174
175
        Err(HttpError {
            code: 403,
            message: "Always fail".to_string(),
        })?
    }
}

#[async_trait]
176
177
178
179
180
181
impl
    AsyncEngine<
        SingleIn<NvCreateCompletionRequest>,
        ManyOut<Annotated<NvCreateCompletionResponse>>,
        Error,
    > for AlwaysFailEngine
182
183
184
{
    async fn generate(
        &self,
185
        _request: SingleIn<NvCreateCompletionRequest>,
186
    ) -> Result<ManyOut<Annotated<NvCreateCompletionResponse>>, Error> {
187
188
189
190
191
192
193
194
        Err(HttpError {
            code: 401,
            message: "Always fail".to_string(),
        })?
    }
}

fn compare_counter(
195
    metrics: &Metrics,
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
    model: &str,
    endpoint: &Endpoint,
    request_type: &RequestType,
    status: &Status,
    expected: u64,
) {
    assert_eq!(
        metrics.get_request_counter(model, endpoint, request_type, status),
        expected,
        "model: {}, endpoint: {:?}, request_type: {:?}, status: {:?}",
        model,
        endpoint.as_str(),
        request_type.as_str(),
        status.as_str()
    );
}

fn compute_index(endpoint: &Endpoint, request_type: &RequestType, status: &Status) -> usize {
    let endpoint = match endpoint {
        Endpoint::Completions => 0,
        Endpoint::ChatCompletions => 1,
217
        Endpoint::Embeddings => todo!(),
218
        Endpoint::Responses => todo!(),
219
        Endpoint::Tensor => todo!(),
220
        Endpoint::Images => todo!(),
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
    };

    let request_type = match request_type {
        RequestType::Unary => 0,
        RequestType::Stream => 1,
    };

    let status = match status {
        Status::Success => 0,
        Status::Error => 1,
    };

    endpoint * 4 + request_type * 2 + status
}

236
fn compare_counters(metrics: &Metrics, model: &str, expected: &[u64; 8]) {
237
238
239
240
241
    for endpoint in &[Endpoint::Completions, Endpoint::ChatCompletions] {
        for request_type in &[RequestType::Unary, RequestType::Stream] {
            for status in &[Status::Success, Status::Error] {
                let index = compute_index(endpoint, request_type, status);
                compare_counter(
242
                    metrics,
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
                    model,
                    endpoint,
                    request_type,
                    status,
                    expected[index],
                );
            }
        }
    }
}

fn inc_counter(
    endpoint: Endpoint,
    request_type: RequestType,
    status: Status,
    expected: &mut [u64; 8],
) {
    let index = compute_index(&endpoint, &request_type, &status);
    expected[index] += 1;
}

Paul Hendricks's avatar
Paul Hendricks committed
264
#[allow(deprecated)]
265
266
#[tokio::test]
async fn test_http_service() {
267
    let port = get_random_port().await;
268
    let service = HttpService::builder()
269
        .port(port)
270
271
272
273
        .enable_chat_endpoints(true)
        .enable_cmpl_endpoints(true)
        .build()
        .unwrap();
274
275
    let state = service.state_clone();
    let manager = state.manager();
276
277
278
279
280

    let token = CancellationToken::new();
    let cancel_token = token.clone();
    let task = tokio::spawn(async move { service.run(token.clone()).await });

281
282
283
    // Wait for the service to be ready before proceeding
    wait_for_service_ready(port).await;

284
285
    let registry = Registry::new();

286
287
    // TODO: Shouldn't this test know the card before it registers a model?
    let card = ModelDeploymentCard::with_name_only("foo");
288
    let counter = Arc::new(CounterEngine {});
289
    let result = manager.add_chat_completions_model("foo", card.mdcsum(), counter);
290
291
292
    assert!(result.is_ok());

    let failure = Arc::new(AlwaysFailEngine {});
293
294
    let card = ModelDeploymentCard::with_name_only("bar");
    let result = manager.add_chat_completions_model("bar", card.mdcsum(), failure.clone());
295
296
    assert!(result.is_ok());

297
    let result = manager.add_completions_model("bar", card.mdcsum(), failure);
298
299
    assert!(result.is_ok());

300
    let metrics = state.metrics_clone();
301
302
303
304
305
    metrics.register(&registry).unwrap();

    let mut foo_counters = [0u64; 8];
    let mut bar_counters = [0u64; 8];

306
307
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
308
309
310

    let client = reqwest::Client::new();

311
312
313
    let message = dynamo_async_openai::types::ChatCompletionRequestMessage::User(
        dynamo_async_openai::types::ChatCompletionRequestUserMessage {
            content: dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
Paul Hendricks's avatar
Paul Hendricks committed
314
315
316
317
318
319
                "hi".to_string(),
            ),
            name: None,
        },
    );

320
    let mut request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
321
        .model("foo")
Paul Hendricks's avatar
Paul Hendricks committed
322
        .messages(vec![message])
323
        .build()
Paul Hendricks's avatar
Paul Hendricks committed
324
325
326
327
328
329
330
        .expect("Failed to build request");

    // let mut request = ChatCompletionRequest::builder()
    //     .model("foo")
    //     .add_user_message("hi")
    //     .build()
    //     .unwrap();
331
332
333

    // ==== ChatCompletions / Stream / Success ====
    request.stream = Some(true);
Paul Hendricks's avatar
Paul Hendricks committed
334
335

    // ALLOW: max_tokens is deprecated in favor of completion_usage_tokens
336
337
338
    request.max_tokens = Some(3000);

    let response = client
339
        .post(format!("http://localhost:{}/v1/chat/completions", port))
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
        .json(&request)
        .send()
        .await
        .unwrap();

    assert!(response.status().is_success(), "{:?}", response);

    tokio::time::sleep(tokio::time::Duration::from_millis(1000)).await;
    assert_eq!(metrics.get_inflight_count("foo"), 1);

    // process byte stream
    let _ = response.bytes().await.unwrap();

    inc_counter(
        Endpoint::ChatCompletions,
        RequestType::Stream,
        Status::Success,
        &mut foo_counters,
    );
359
360
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
361
362
363
364
365

    // check registry and look or the request duration histogram
    let families = registry.gather();
    let histogram_metric_family = families
        .into_iter()
366
367
368
369
370
371
372
373
        .find(|m| {
            m.get_name()
                == format!(
                    "{}_{}",
                    name_prefix::FRONTEND,
                    frontend_service::REQUEST_DURATION_SECONDS
                )
        })
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
        .expect("Histogram metric not found");

    assert_eq!(
        histogram_metric_family.get_field_type(),
        MetricType::HISTOGRAM
    );

    let histogram_metric = histogram_metric_family.get_metric();

    assert_eq!(histogram_metric.len(), 1); // We have one metric with label model

    let metric = &histogram_metric[0];
    let histogram = metric.get_histogram();

    let buckets = histogram.get_bucket();

    let mut found = false;
391
392
393
394
395
396
    let mut expected_count = 0;
    for bucket_idx in 1..buckets.len() {
        if buckets[bucket_idx].get_upper_bound() >= 2.5
            && buckets[bucket_idx - 1].get_upper_bound() < 2.5
        {
            found = true;
397
            assert_eq!(
398
399
400
                buckets[bucket_idx].get_cumulative_count(),
                1,
                "Observation should be counted in the bucket containing 2.5"
401
            );
402
            expected_count = 1;
403
404
        } else {
            assert_eq!(
405
406
                buckets[bucket_idx].get_cumulative_count(),
                expected_count,
407
408
409
410
411
412
413
414
415
416
                "No observations should be in this bucket"
            );
        }
    }

    assert!(found, "The expected bucket was not found");
    // ==== ChatCompletions / Stream / Success ====

    // ==== ChatCompletions / Unary / Success ====
    request.stream = Some(false);
Paul Hendricks's avatar
Paul Hendricks committed
417
418

    // ALLOW: max_tokens is deprecated in favor of completion_usage_tokens
419
420
421
    request.max_tokens = Some(0);

    let future = client
422
        .post(format!("http://localhost:{}/v1/chat/completions", port))
423
424
425
426
427
428
429
430
431
432
433
434
        .json(&request)
        .send();

    let response = future.await.unwrap();

    assert!(response.status().is_success(), "{:?}", response);
    inc_counter(
        Endpoint::ChatCompletions,
        RequestType::Unary,
        Status::Success,
        &mut foo_counters,
    );
435
436
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
437
438
439
440
    // ==== ChatCompletions / Unary / Success ====

    // ==== ChatCompletions / Stream / Error ====
    request.model = "bar".to_string();
Paul Hendricks's avatar
Paul Hendricks committed
441
442

    // ALLOW: max_tokens is deprecated in favor of completion_usage_tokens
443
444
445
446
    request.max_tokens = Some(0);
    request.stream = Some(true);

    let response = client
447
        .post(format!("http://localhost:{}/v1/chat/completions", port))
448
449
450
451
452
453
454
455
456
457
458
459
        .json(&request)
        .send()
        .await
        .unwrap();

    assert_eq!(response.status(), StatusCode::FORBIDDEN);
    inc_counter(
        Endpoint::ChatCompletions,
        RequestType::Stream,
        Status::Error,
        &mut bar_counters,
    );
460
461
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
462
463
464
465
466
467
    // ==== ChatCompletions / Stream / Error ====

    // ==== ChatCompletions / Unary / Error ====
    request.stream = Some(false);

    let response = client
468
        .post(format!("http://localhost:{}/v1/chat/completions", port))
469
470
471
472
473
474
475
476
477
478
479
480
        .json(&request)
        .send()
        .await
        .unwrap();

    assert_eq!(response.status(), StatusCode::FORBIDDEN);
    inc_counter(
        Endpoint::ChatCompletions,
        RequestType::Unary,
        Status::Error,
        &mut bar_counters,
    );
481
482
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
483
484
485
    // ==== ChatCompletions / Unary / Error ====

    // ==== Completions / Unary / Error ====
486
    let mut request = dynamo_async_openai::types::CreateCompletionRequestArgs::default()
487
488
489
490
491
492
        .model("bar")
        .prompt("hi")
        .build()
        .unwrap();

    let response = client
493
        .post(format!("http://localhost:{}/v1/completions", port))
494
495
496
497
498
499
500
501
502
503
504
505
        .json(&request)
        .send()
        .await
        .unwrap();

    assert_eq!(response.status(), StatusCode::UNAUTHORIZED);
    inc_counter(
        Endpoint::Completions,
        RequestType::Unary,
        Status::Error,
        &mut bar_counters,
    );
506
507
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
508
509
510
511
512
513
    // ==== Completions / Unary / Error ====

    // ==== Completions / Stream / Error ====
    request.stream = Some(true);

    let response = client
514
        .post(format!("http://localhost:{}/v1/completions", port))
515
516
517
518
519
520
521
522
523
524
525
526
        .json(&request)
        .send()
        .await
        .unwrap();

    assert_eq!(response.status(), StatusCode::UNAUTHORIZED);
    inc_counter(
        Endpoint::Completions,
        RequestType::Stream,
        Status::Error,
        &mut bar_counters,
    );
527
528
    compare_counters(&metrics, "foo", &foo_counters);
    compare_counters(&metrics, "bar", &bar_counters);
529
530
531
532
533
534
535
    // ==== Completions / Stream / Error ====

    // =========== Test Invalid Request ===========
    // send a completion request to a chat endpoint
    request.stream = Some(false);

    let response = client
536
        .post(format!("http://localhost:{}/v1/chat/completions", port))
537
538
539
540
541
        .json(&request)
        .send()
        .await
        .unwrap();

542
    assert_eq!(response.status(), StatusCode::BAD_REQUEST, "{:?}", response);
543
544
545

    // =========== Query /metrics endpoint ===========
    let response = client
546
        .get(format!("http://localhost:{}/metrics", port))
547
548
549
550
551
552
553
554
555
556
        .send()
        .await
        .unwrap();

    assert!(response.status().is_success(), "{:?}", response);
    println!("{}", response.text().await.unwrap());

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574

// === HTTP Client Tests ===

/// Wait for the HTTP service to be ready by checking its health endpoint
async fn wait_for_service_ready(port: u16) {
    let start = tokio::time::Instant::now();
    let timeout = tokio::time::Duration::from_secs(5);
    loop {
        match reqwest::get(&format!("http://localhost:{}/health", port)).await {
            Ok(_) => break,
            Err(_) if start.elapsed() < timeout => {
                tokio::time::sleep(tokio::time::Duration::from_millis(10)).await;
            }
            Err(e) => panic!("Service failed to start within timeout: {}", e),
        }
    }
}

575
576
async fn service_with_engines() -> (HttpService, Arc<CounterEngine>, Arc<AlwaysFailEngine>, u16) {
    let port = get_random_port().await;
577
578
579
580
581
582
    let service = HttpService::builder()
        .enable_chat_endpoints(true)
        .enable_cmpl_endpoints(true)
        .port(port)
        .build()
        .unwrap();
583
584
585
586
587
    let manager = service.model_manager();

    let counter = Arc::new(CounterEngine {});
    let failure = Arc::new(AlwaysFailEngine {});

588
    let card = ModelDeploymentCard::with_name_only("foo");
589
    manager
590
        .add_chat_completions_model("foo", card.mdcsum(), counter.clone())
591
        .unwrap();
592
    let card = ModelDeploymentCard::with_name_only("bar");
593
    manager
594
        .add_chat_completions_model("bar", card.mdcsum(), failure.clone())
595
596
        .unwrap();
    manager
597
        .add_completions_model("bar", card.mdcsum(), failure.clone())
598
599
        .unwrap();

600
    (service, counter, failure, port)
601
602
}

603
fn pure_openai_client(port: u16) -> PureOpenAIClient {
604
605
606
607
608
609
610
    let config = HttpClientConfig {
        openai_config: OpenAIConfig::new().with_api_base(format!("http://localhost:{}/v1", port)),
        verbose: false,
    };
    PureOpenAIClient::new(config)
}

611
fn nv_custom_client(port: u16) -> NvCustomClient {
612
613
614
615
616
617
618
    let config = HttpClientConfig {
        openai_config: OpenAIConfig::new().with_api_base(format!("http://localhost:{}/v1", port)),
        verbose: false,
    };
    NvCustomClient::new(config)
}

619
fn generic_byot_client(port: u16) -> GenericBYOTClient {
620
621
622
623
624
625
626
627
    let config = HttpClientConfig {
        openai_config: OpenAIConfig::new().with_api_base(format!("http://localhost:{}/v1", port)),
        verbose: false,
    };
    GenericBYOTClient::new(config)
}

#[tokio::test]
628
629
630
631
async fn test_pure_openai_client() {
    let (service, _counter, _failure, port) = service_with_engines().await;
    let pure_openai_client = pure_openai_client(port);

632
633
634
635
636
637
638
    let token = CancellationToken::new();
    let cancel_token = token.clone();

    // Start the service
    let task = tokio::spawn(async move { service.run(token).await });

    // Wait for service to be ready
639
    wait_for_service_ready(port).await;
640
641

    // Test successful streaming request
642
    let request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
643
644
        .model("foo")
        .messages(vec![
645
646
647
648
649
650
            dynamo_async_openai::types::ChatCompletionRequestMessage::User(
                dynamo_async_openai::types::ChatCompletionRequestUserMessage {
                    content:
                        dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
                            "Hi".to_string(),
                        ),
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
                    name: None,
                },
            ),
        ])
        .stream(true)
        .max_tokens(50u32)
        .build()
        .unwrap();

    let result = pure_openai_client.chat_stream(request).await;
    assert!(result.is_ok(), "PureOpenAI client should succeed");

    let (mut stream, _context) = result.unwrap().dissolve();
    let mut count = 0;
    while let Some(response) = stream.next().await {
        count += 1;
        assert!(response.is_ok(), "Response should be ok");
        if count >= 3 {
            break; // Don't consume entire stream
        }
    }
    assert!(count > 0, "Should receive at least one response");

    // Test error case with invalid model
675
    let request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
676
677
        .model("bar") // This model will fail
        .messages(vec![
678
679
680
681
682
683
            dynamo_async_openai::types::ChatCompletionRequestMessage::User(
                dynamo_async_openai::types::ChatCompletionRequestUserMessage {
                    content:
                        dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
                            "Hi".to_string(),
                        ),
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
                    name: None,
                },
            ),
        ])
        .stream(true)
        .max_tokens(50u32)
        .build()
        .unwrap();

    let result = pure_openai_client.chat_stream(request).await;
    assert!(
        result.is_ok(),
        "Client should return stream even for failing model"
    );

    let (mut stream, _context) = result.unwrap().dissolve();
    if let Some(response) = stream.next().await {
        assert!(
            response.is_err(),
            "Response should be error for failing model"
        );
    }

    // Test context management
    let ctx = HttpRequestContext::new();
709
    let request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
710
711
        .model("foo")
        .messages(vec![
712
713
714
715
716
717
            dynamo_async_openai::types::ChatCompletionRequestMessage::User(
                dynamo_async_openai::types::ChatCompletionRequestUserMessage {
                    content:
                        dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
                            "Hi".to_string(),
                        ),
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
                    name: None,
                },
            ),
        ])
        .stream(true)
        .max_tokens(50u32)
        .build()
        .unwrap();

    let result = pure_openai_client
        .chat_stream_with_context(request, ctx.clone())
        .await;
    assert!(result.is_ok(), "Context-based request should succeed");

    let (_stream, context) = result.unwrap().dissolve();
    assert_eq!(context.id(), ctx.id(), "Context ID should match");

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}

#[tokio::test]
740
741
742
743
async fn test_nv_custom_client() {
    let (service, _counter, _failure, port) = service_with_engines().await;
    let nv_custom_client = nv_custom_client(port);

744
745
746
747
748
749
750
    let token = CancellationToken::new();
    let cancel_token = token.clone();

    // Start the service
    let task = tokio::spawn(async move { service.run(token).await });

    // Wait for service to be ready
751
    wait_for_service_ready(port).await;
752
753

    // Test successful streaming request
754
    let inner_request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
755
756
        .model("foo")
        .messages(vec![
757
758
759
760
761
762
            dynamo_async_openai::types::ChatCompletionRequestMessage::User(
                dynamo_async_openai::types::ChatCompletionRequestUserMessage {
                    content:
                        dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
                            "Hi".to_string(),
                        ),
763
764
765
766
767
768
769
770
771
772
773
                    name: None,
                },
            ),
        ])
        .stream(true)
        .max_tokens(50u32)
        .build()
        .unwrap();

    let request = NvCreateChatCompletionRequest {
        inner: inner_request,
774
        common: Default::default(),
775
        nvext: None,
776
        chat_template_args: None,
777
        media_io_kwargs: None,
778
        unsupported_fields: Default::default(),
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
    };

    let result = nv_custom_client.chat_stream(request).await;
    assert!(result.is_ok(), "NvCustom client should succeed");

    let (mut stream, _context) = result.unwrap().dissolve();
    let mut count = 0;
    while let Some(response) = stream.next().await {
        count += 1;
        assert!(response.is_ok(), "Response should be ok");
        if count >= 3 {
            break; // Don't consume entire stream
        }
    }
    assert!(count > 0, "Should receive at least one response");

    // Test error case with invalid model
796
    let inner_request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
797
798
        .model("bar") // This model will fail
        .messages(vec![
799
800
801
802
803
804
            dynamo_async_openai::types::ChatCompletionRequestMessage::User(
                dynamo_async_openai::types::ChatCompletionRequestUserMessage {
                    content:
                        dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
                            "Hi".to_string(),
                        ),
805
806
807
808
809
810
811
812
813
814
815
                    name: None,
                },
            ),
        ])
        .stream(true)
        .max_tokens(50u32)
        .build()
        .unwrap();

    let request = NvCreateChatCompletionRequest {
        inner: inner_request,
816
        common: Default::default(),
817
        nvext: None,
818
        chat_template_args: None,
819
        media_io_kwargs: None,
820
        unsupported_fields: Default::default(),
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
    };

    let result = nv_custom_client.chat_stream(request).await;
    assert!(
        result.is_ok(),
        "Client should return stream even for failing model"
    );

    let (mut stream, _context) = result.unwrap().dissolve();
    if let Some(response) = stream.next().await {
        assert!(
            response.is_err(),
            "Response should be error for failing model"
        );
    }

    // Test context management
    let ctx = HttpRequestContext::new();
839
    let inner_request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
840
841
        .model("foo")
        .messages(vec![
842
843
844
845
846
847
            dynamo_async_openai::types::ChatCompletionRequestMessage::User(
                dynamo_async_openai::types::ChatCompletionRequestUserMessage {
                    content:
                        dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
                            "Hi".to_string(),
                        ),
848
849
850
851
852
853
854
855
856
857
858
                    name: None,
                },
            ),
        ])
        .stream(true)
        .max_tokens(50u32)
        .build()
        .unwrap();

    let request = NvCreateChatCompletionRequest {
        inner: inner_request,
859
        common: Default::default(),
860
        nvext: None,
861
        chat_template_args: None,
862
        media_io_kwargs: None,
863
        unsupported_fields: Default::default(),
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
    };

    let result = nv_custom_client
        .chat_stream_with_context(request, ctx.clone())
        .await;
    assert!(result.is_ok(), "Context-based request should succeed");

    let (_stream, context) = result.unwrap().dissolve();
    assert_eq!(context.id(), ctx.id(), "Context ID should match");

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}

#[tokio::test]
879
880
881
882
async fn test_generic_byot_client() {
    let (service, _counter, _failure, port) = service_with_engines().await;
    let generic_byot_client = generic_byot_client(port);

883
884
885
886
887
888
889
    let token = CancellationToken::new();
    let cancel_token = token.clone();

    // Start the service
    let task = tokio::spawn(async move { service.run(token).await });

    // Wait for service to be ready
890
    wait_for_service_ready(port).await;
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971

    // Test successful streaming request
    let request = serde_json::json!({
        "model": "foo",
        "messages": [
            {
                "role": "user",
                "content": "Hi"
            }
        ],
        "stream": true,
        "max_tokens": 50
    });

    let result = generic_byot_client.chat_stream(request).await;
    assert!(result.is_ok(), "GenericBYOT client should succeed");

    let (mut stream, _context) = result.unwrap().dissolve();
    let mut count = 0;
    while let Some(response) = stream.next().await {
        println!("Response: {:?}", response);
        count += 1;
        assert!(response.is_ok(), "Response should be ok");
        if count >= 3 {
            break; // Don't consume entire stream
        }
    }
    assert!(count > 0, "Should receive at least one response");

    // Test error case with invalid model
    let request = serde_json::json!({
        "model": "bar", // This model will fail
        "messages": [
            {
                "role": "user",
                "content": "Hi"
            }
        ],
        "stream": true,
        "max_tokens": 50
    });

    let result = generic_byot_client.chat_stream(request).await;
    assert!(
        result.is_ok(),
        "Client should return stream even for failing model"
    );

    let (mut stream, _context) = result.unwrap().dissolve();
    if let Some(response) = stream.next().await {
        assert!(
            response.is_err(),
            "Response should be error for failing model"
        );
    }

    // Test context management
    let ctx = HttpRequestContext::new();
    let request = serde_json::json!({
        "model": "foo",
        "messages": [
            {
                "role": "user",
                "content": "Hi"
            }
        ],
        "stream": true,
        "max_tokens": 50
    });

    let result = generic_byot_client
        .chat_stream_with_context(request, ctx.clone())
        .await;
    assert!(result.is_ok(), "Context-based request should succeed");

    let (_stream, context) = result.unwrap().dissolve();
    assert_eq!(context.id(), ctx.id(), "Context ID should match");

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}
Ryan Olson's avatar
Ryan Olson committed
972
973
974

#[tokio::test]
async fn test_client_disconnect_cancellation_unary() {
975
    let port = get_random_port().await;
976
977
978
    let service = HttpService::builder()
        .enable_chat_endpoints(true)
        .enable_cmpl_endpoints(true)
979
        .port(port)
980
981
        .build()
        .unwrap();
Ryan Olson's avatar
Ryan Olson committed
982
983
984
985
986
987
988
989
990
991
    let state = service.state_clone();
    let manager = state.manager();

    let token = CancellationToken::new();
    let cancel_token = token.clone();

    // Start the service
    let task = tokio::spawn(async move { service.run(token).await });

    // Wait for service to be ready
992
    wait_for_service_ready(port).await;
Ryan Olson's avatar
Ryan Olson committed
993
994

    // Create a long-running engine (10 seconds)
995
    let card = ModelDeploymentCard::with_name_only("slow-model");
Ryan Olson's avatar
Ryan Olson committed
996
997
    let long_running_engine = Arc::new(LongRunningEngine::new(10_000));
    manager
998
        .add_chat_completions_model("slow-model", card.mdcsum(), long_running_engine.clone())
Ryan Olson's avatar
Ryan Olson committed
999
1000
1001
1002
        .unwrap();

    let client = reqwest::Client::new();

1003
1004
1005
    let message = dynamo_async_openai::types::ChatCompletionRequestMessage::User(
        dynamo_async_openai::types::ChatCompletionRequestUserMessage {
            content: dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
Ryan Olson's avatar
Ryan Olson committed
1006
1007
1008
1009
1010
1011
                "This will take a long time".to_string(),
            ),
            name: None,
        },
    );

1012
    let request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
Ryan Olson's avatar
Ryan Olson committed
1013
1014
1015
1016
1017
1018
1019
1020
1021
1022
1023
        .model("slow-model")
        .messages(vec![message])
        .stream(false) // Test unary response
        .build()
        .expect("Failed to build request");

    // Start the request and cancel it after 1 second
    let start_time = std::time::Instant::now();

    let request_future = async {
        client
1024
            .post(format!("http://localhost:{}/v1/chat/completions", port))
Ryan Olson's avatar
Ryan Olson committed
1025
1026
1027
1028
1029
1030
1031
1032
1033
1034
1035
1036
1037
1038
1039
1040
1041
1042
1043
1044
1045
1046
1047
1048
1049
1050
1051
1052
1053
1054
1055
1056
1057
1058
1059
1060
1061
1062
1063
1064
1065
1066
            .json(&request)
            .send()
            .await
    };

    // Use timeout to simulate client disconnect after 1 second
    let result = timeout(std::time::Duration::from_millis(1000), request_future).await;

    let elapsed = start_time.elapsed();

    // The request should timeout (simulating client disconnect)
    assert!(result.is_err(), "Request should have timed out");

    // Give the service a moment to detect the disconnect and propagate cancellation
    tokio::time::sleep(std::time::Duration::from_millis(500)).await;

    // Verify the engine was cancelled
    assert!(
        long_running_engine.was_cancelled(),
        "Engine should have been cancelled due to client disconnect"
    );

    // Verify cancellation happened quickly (within 2 seconds, not the full 10 seconds)
    assert!(
        elapsed < std::time::Duration::from_secs(2),
        "Cancellation should have propagated quickly, took {:?}",
        elapsed
    );

    tracing::info!(
        "✅ Client disconnect test passed! Request cancelled in {:?}, engine detected cancellation",
        elapsed
    );

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}

#[tokio::test]
async fn test_client_disconnect_cancellation_streaming() {
    dynamo_runtime::logging::init();

1067
    let port = get_random_port().await;
1068
1069
1070
    let service = HttpService::builder()
        .enable_chat_endpoints(true)
        .enable_cmpl_endpoints(true)
1071
        .port(port)
1072
1073
        .build()
        .unwrap();
Ryan Olson's avatar
Ryan Olson committed
1074
1075
1076
1077
1078
1079
1080
1081
1082
1083
    let state = service.state_clone();
    let manager = state.manager();

    let token = CancellationToken::new();
    let cancel_token = token.clone();

    // Start the service
    let task = tokio::spawn(async move { service.run(token).await });

    // Wait for service to be ready
1084
    wait_for_service_ready(port).await;
Ryan Olson's avatar
Ryan Olson committed
1085
1086

    // Create a long-running engine (10 seconds)
1087
    let card = ModelDeploymentCard::with_name_only("slow-stream-model");
Ryan Olson's avatar
Ryan Olson committed
1088
1089
    let long_running_engine = Arc::new(LongRunningEngine::new(10_000));
    manager
1090
1091
1092
1093
1094
        .add_chat_completions_model(
            "slow-stream-model",
            card.mdcsum(),
            long_running_engine.clone(),
        )
Ryan Olson's avatar
Ryan Olson committed
1095
1096
1097
1098
        .unwrap();

    let client = reqwest::Client::new();

1099
1100
1101
    let message = dynamo_async_openai::types::ChatCompletionRequestMessage::User(
        dynamo_async_openai::types::ChatCompletionRequestUserMessage {
            content: dynamo_async_openai::types::ChatCompletionRequestUserMessageContent::Text(
Ryan Olson's avatar
Ryan Olson committed
1102
1103
1104
1105
1106
1107
                "This will stream for a long time".to_string(),
            ),
            name: None,
        },
    );

1108
    let request = dynamo_async_openai::types::CreateChatCompletionRequestArgs::default()
Ryan Olson's avatar
Ryan Olson committed
1109
1110
1111
1112
1113
1114
1115
1116
1117
1118
1119
        .model("slow-stream-model")
        .messages(vec![message])
        .stream(true) // Test streaming response
        .build()
        .expect("Failed to build request");

    // Start the request and cancel it after 1 second
    let start_time = std::time::Instant::now();

    let request_future = async {
        let response = client
1120
            .post(format!("http://localhost:{}/v1/chat/completions", port))
Ryan Olson's avatar
Ryan Olson committed
1121
1122
1123
1124
1125
1126
1127
1128
1129
1130
1131
1132
1133
1134
1135
1136
1137
1138
1139
1140
1141
1142
1143
1144
1145
1146
1147
1148
1149
1150
1151
1152
1153
1154
1155
1156
1157
1158
1159
1160
1161
1162
1163
1164
1165
1166
1167
1168
1169
            .json(&request)
            .send()
            .await
            .unwrap();

        // Start reading the stream, then drop it to simulate client disconnect
        let mut stream = response.bytes_stream();
        tokio::time::sleep(std::time::Duration::from_millis(500)).await;

        // Read one chunk then drop the stream (simulating client disconnect)
        let _ = StreamExt::next(&mut stream).await;
        // Stream gets dropped here when function exits
    };

    // Use timeout to simulate the streaming request timing out
    let _result = timeout(std::time::Duration::from_millis(1500), request_future).await;

    let elapsed = start_time.elapsed();

    // Give the service time to detect the disconnect
    tokio::time::sleep(std::time::Duration::from_millis(1000)).await;

    // Verify the engine was cancelled
    assert!(
        long_running_engine.was_cancelled(),
        "Engine should have been cancelled due to streaming client disconnect"
    );

    // Verify cancellation happened reasonably quickly
    assert!(
        elapsed < std::time::Duration::from_secs(3),
        "Stream cancellation should have propagated reasonably quickly, took {:?}",
        elapsed
    );

    tracing::info!(
        "✅ Streaming client disconnect test passed! Stream cancelled in {:?}, engine detected cancellation",
        elapsed
    );

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}

#[tokio::test]
async fn test_request_id_annotation() {
    // TODO(ryan): make better fixtures, this is too much to test sometime so simple
    dynamo_runtime::logging::init();

1170
    let port = get_random_port().await;
1171
1172
1173
    let service = HttpService::builder()
        .enable_chat_endpoints(true)
        .enable_cmpl_endpoints(true)
1174
        .port(port)
1175
1176
        .build()
        .unwrap();
Ryan Olson's avatar
Ryan Olson committed
1177
1178
1179
1180
1181
1182
1183
1184
1185
1186
    let state = service.state_clone();
    let manager = state.manager();

    let token = CancellationToken::new();
    let cancel_token = token.clone();

    // Start the service
    let task = tokio::spawn(async move { service.run(token).await });

    // Wait for service to be ready
1187
    wait_for_service_ready(port).await;
Ryan Olson's avatar
Ryan Olson committed
1188
1189

    // Add a counter engine for this test
1190
    let card = ModelDeploymentCard::with_name_only("test-model");
Ryan Olson's avatar
Ryan Olson committed
1191
1192
    let counter_engine = Arc::new(CounterEngine {});
    manager
1193
        .add_chat_completions_model("test-model", card.mdcsum(), counter_engine)
Ryan Olson's avatar
Ryan Olson committed
1194
1195
1196
1197
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
        .unwrap();

    // Create reqwest client directly
    let client = reqwest::Client::new();

    // Generate a UUID for the request ID
    let request_uuid = uuid::Uuid::new_v4();

    // Create the request JSON directly
    let request_json = serde_json::json!({
        "model": "test-model",
        "messages": [
            {
                "role": "user",
                "content": "Test request with annotation"
            }
        ],
        "stream": true,
        "max_tokens": 50,
        "nvext": {
            "annotations": ["request_id"]
        }
    });

    // Make the streaming request with custom header
    let response = client
1220
        .post(format!("http://localhost:{}/v1/chat/completions", port))
Ryan Olson's avatar
Ryan Olson committed
1221
1222
1223
1224
1225
1226
1227
1228
1229
1230
1231
1232
1233
1234
1235
1236
1237
1238
1239
1240
1241
1242
1243
1244
1245
1246
1247
1248
1249
1250
1251
        .header("x-dynamo-request-id", request_uuid.to_string())
        .json(&request_json)
        .send()
        .await
        .expect("Request should succeed");

    assert!(
        response.status().is_success(),
        "Response should be successful"
    );

    // Collect the entire response body as bytes first
    let body_bytes = response
        .bytes()
        .await
        .expect("Failed to read response body");
    let body_text = String::from_utf8_lossy(&body_bytes);

    // Create a cursor from the text and use SseLineCodec to parse it
    let cursor = Cursor::new(body_text.to_string());
    let framed = FramedRead::new(cursor, SseLineCodec::new());
    let annotated_stream = convert_sse_stream::<NvCreateChatCompletionStreamResponse>(framed);

    // Look for the annotation in the stream
    let mut found_request_id_annotation = false;
    let mut received_request_id = None;

    // Process the annotated stream and look for the request_id annotation
    let mut annotated_stream = std::pin::pin!(annotated_stream);
    while let Some(annotated_response) = annotated_stream.next().await {
        // Check if this is a request_id annotation
1252
1253
1254
1255
1256
1257
1258
1259
1260
1261
1262
1263
1264
1265
        if let Some(event) = &annotated_response.event
            && event == "request_id"
        {
            found_request_id_annotation = true;
            // Extract the request ID from the annotation
            if let Some(comments) = &annotated_response.comment
                && let Some(comment) = comments.first()
            {
                // The comment contains a JSON-encoded string, so we need to parse it
                if let Ok(parsed_value) = serde_json::from_str::<String>(comment) {
                    received_request_id = Some(parsed_value);
                } else {
                    // Fallback: remove quotes manually if JSON parsing fails
                    received_request_id = Some(comment.trim_matches('"').to_string());
Ryan Olson's avatar
Ryan Olson committed
1266
1267
                }
            }
1268
            break;
Ryan Olson's avatar
Ryan Olson committed
1269
1270
1271
1272
1273
1274
1275
1276
1277
1278
1279
1280
1281
1282
1283
1284
1285
1286
1287
1288
1289
1290
1291
1292
1293
1294
1295
1296
1297
1298
1299
1300
1301
        }
    }

    // Verify we found the annotation
    assert!(
        found_request_id_annotation,
        "Should have received request_id annotation in the stream"
    );

    // Verify the request ID matches what we sent
    assert!(
        received_request_id.is_some(),
        "Should have received the request ID in the annotation"
    );

    let received_uuid_str = received_request_id.unwrap();
    assert_eq!(
        received_uuid_str,
        request_uuid.to_string(),
        "Received request ID should match the one we sent: expected {}, got {}",
        request_uuid,
        received_uuid_str
    );

    tracing::info!(
        "✅ Request ID annotation test passed! Sent UUID: {}, Received: {}",
        request_uuid,
        received_uuid_str
    );

    cancel_token.cancel();
    task.await.unwrap().unwrap();
}