watcher.rs 28.1 KB
Newer Older
Ryan Olson's avatar
Ryan Olson committed
1
2
3
// SPDX-FileCopyrightText: Copyright (c) 2024-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
// SPDX-License-Identifier: Apache-2.0

Ryan Olson's avatar
Ryan Olson committed
4
use std::sync::Arc;
5
use tokio::sync::Notify;
6
use tokio::sync::mpsc::Sender;
7

8
use anyhow::Context as _;
9
use futures::StreamExt;
Ryan Olson's avatar
Ryan Olson committed
10

Neelay Shah's avatar
Neelay Shah committed
11
use dynamo_runtime::{
12
    DistributedRuntime,
13
    discovery::{DiscoveryEvent, DiscoveryInstance, DiscoveryQuery, DiscoveryStream},
14
    pipeline::{
15
16
        ManyOut, Operator, RouterMode, SegmentSource, ServiceBackend, SingleIn, Source,
        network::egress::push_router::PushRouter,
17
    },
18
    protocols::{EndpointId, annotated::Annotated},
19
};
Ryan Olson's avatar
Ryan Olson committed
20

21
22
use crate::{
    backend::Backend,
23
    entrypoint::{self, EngineFactoryCallback, RouterConfig},
24
    http::service::metrics::Metrics,
25
    kv_router::PrefillRouter,
26
    model_card::ModelDeploymentCard,
27
28
    model_type::{ModelInput, ModelType},
    preprocessor::{OpenAIPreprocessor, PreprocessedEmbeddingRequest, prompt::PromptFormatter},
29
30
31
32
33
34
35
36
37
    protocols::{
        common::llm_backend::EmbeddingsEngineOutput,
        openai::{
            chat_completions::{
                NvCreateChatCompletionRequest, NvCreateChatCompletionStreamResponse,
            },
            completions::{NvCreateCompletionRequest, NvCreateCompletionResponse},
            embeddings::{NvCreateEmbeddingRequest, NvCreateEmbeddingResponse},
        },
38
        tensor::{NvCreateTensorRequest, NvCreateTensorResponse},
39
40
    },
};
41

42
use super::ModelManager;
43
use crate::namespace::is_global_namespace;
44

45
#[derive(Debug, Clone)]
46
pub enum ModelUpdate {
47
48
    Added(ModelDeploymentCard),
    Removed(ModelDeploymentCard),
49
50
}

51
pub struct ModelWatcher {
52
    manager: Arc<ModelManager>,
53
    drt: DistributedRuntime,
54
    router_config: RouterConfig,
55
    notify_on_model: Notify,
56
    model_update_tx: Option<Sender<ModelUpdate>>,
57
    engine_factory: Option<EngineFactoryCallback>,
58
    metrics: Arc<Metrics>,
Ryan Olson's avatar
Ryan Olson committed
59
60
}

61
62
63
64
const ALL_MODEL_TYPES: &[ModelType] = &[
    ModelType::Chat,
    ModelType::Completions,
    ModelType::Embedding,
65
    ModelType::TensorBased,
66
    ModelType::Prefill,
67
];
68

69
impl ModelWatcher {
70
    pub fn new(
71
        runtime: DistributedRuntime,
72
        model_manager: Arc<ModelManager>,
73
        router_config: RouterConfig,
74
        engine_factory: Option<EngineFactoryCallback>,
75
        metrics: Arc<Metrics>,
76
77
    ) -> ModelWatcher {
        Self {
78
            manager: model_manager,
79
            drt: runtime,
80
            router_config,
81
            notify_on_model: Notify::new(),
82
            model_update_tx: None,
83
            engine_factory,
84
            metrics,
85
        }
86
    }
Ryan Olson's avatar
Ryan Olson committed
87

88
89
90
91
    pub fn set_notify_on_model_update(&mut self, tx: Sender<ModelUpdate>) {
        self.model_update_tx = Some(tx);
    }

92
93
94
95
96
97
98
99
100
101
102
    /// Wait until we have at least one chat completions model and return it's name.
    pub async fn wait_for_chat_model(&self) -> String {
        // Loop in case it gets added and immediately deleted
        loop {
            if let Some(model_name) = self.manager.list_chat_completions_models().first() {
                return model_name.to_owned();
            }
            self.notify_on_model.notified().await
        }
    }

103
    /// Common watch logic with optional namespace filtering
104
105
106
107
108
    pub async fn watch(
        &self,
        mut discovery_stream: DiscoveryStream,
        target_namespace: Option<&str>,
    ) {
109
        let global_namespace = target_namespace.is_none_or(is_global_namespace);
110

111
112
113
114
115
116
117
118
119
        while let Some(result) = discovery_stream.next().await {
            let event = match result {
                Ok(event) => event,
                Err(err) => {
                    tracing::error!(%err, "Error in discovery stream");
                    continue;
                }
            };

120
            match event {
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
                DiscoveryEvent::Added(instance) => {
                    // Extract EndpointId, instance_id, and card from the discovery instance
                    let (endpoint_id, instance_id, mut card) = match &instance {
                        DiscoveryInstance::Model {
                            namespace,
                            component,
                            endpoint,
                            instance_id,
                            ..
                        } => {
                            let eid = EndpointId {
                                namespace: namespace.clone(),
                                component: component.clone(),
                                name: endpoint.clone(),
                            };

                            match instance.deserialize_model::<ModelDeploymentCard>() {
                                Ok(card) => (eid, *instance_id, card),
                                Err(err) => {
                                    tracing::error!(%err, instance_id, "Failed to deserialize model card");
                                    continue;
                                }
                            }
                        }
                        _ => {
                            tracing::error!(
                                "Unexpected discovery instance type (expected ModelCard)"
                            );
149
150
151
                            continue;
                        }
                    };
152
153
154
155

                    // Filter by namespace if target_namespace is specified
                    if !global_namespace
                        && let Some(target_ns) = target_namespace
156
                        && endpoint_id.namespace != target_ns
157
158
                    {
                        tracing::debug!(
159
                            model_namespace = endpoint_id.namespace,
160
161
162
163
164
165
                            target_namespace = target_ns,
                            "Skipping model from different namespace"
                        );
                        continue;
                    }

166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
                    // If we already have a worker for this model, and the ModelDeploymentCard
                    // cards don't match, alert, and don't add the new instance
                    let can_add =
                        self.manager
                            .is_valid_checksum(card.model_type, card.name(), card.mdcsum());
                    if can_add.is_some_and(|is_valid| !is_valid) {
                        tracing::error!(
                            model_name = card.name(),
                            "Checksum for new model does not match existing model."
                        );

                        // TODO: mark that instance down in clients
                        // Not obvious how to do that given the current design
                        // Instances come from an `InstanceSource` in a `Client` in a `PushRouter`.
                        // Calling `report_instance_down` on the Client should do it (although
                        // needs more testing).
                        // The `PushRouter` is in `ModelMananger` (`self.manager` here), but inside
                        // interface `AsyncEngine` which only has a `generate` method.

                        continue;
                    }
187

188
189
190
191
                    // Use instance_id as the HashMap key (simpler and sufficient since keys are opaque)
                    let key = format!("{:x}", instance_id);

                    match self.handle_put(&key, &endpoint_id, &mut card).await {
192
                        Ok(()) => {
193
                            tracing::info!(
194
195
                                model_name = card.name(),
                                namespace = endpoint_id.namespace,
196
197
                                "added model"
                            );
198
                            self.notify_on_model.notify_waiters();
199
                        }
200
201
                        Err(err) => {
                            tracing::error!(
202
203
                                model_name = card.name(),
                                namespace = endpoint_id.namespace,
204
                                error = format!("{err:#}"),
205
                                "Error adding model from discovery",
206
                            );
207
208
209
                        }
                    }
                }
210
211
212
213
                DiscoveryEvent::Removed(instance_id) => {
                    // Use instance_id hex as the HashMap key (matches what we saved with)
                    let key = format!("{:x}", instance_id);

214
                    match self
215
                        .handle_delete(&key, target_namespace, global_namespace)
216
217
218
219
220
221
222
223
224
225
226
                        .await
                    {
                        Ok(Some(model_name)) => {
                            tracing::info!(model_name, "removed model");
                        }
                        Ok(None) => {
                            // There are other instances running this model, nothing to do
                        }
                        Err(e) => {
                            tracing::error!(error = %e, "error removing model");
                        }
227
                    }
228
                }
229
            }
Ryan Olson's avatar
Ryan Olson committed
230
231
232
        }
    }

233
234
    /// If the last instance running this model has gone delete it.
    /// Returns the name of the model we just deleted, if any.
235
236
    async fn handle_delete(
        &self,
237
        key: &str,
238
239
240
        target_namespace: Option<&str>,
        is_global_namespace: bool,
    ) -> anyhow::Result<Option<String>> {
241
242
        let card = match self.manager.remove_model_card(key) {
            Some(card) => card,
243
            None => {
244
                anyhow::bail!("Missing ModelDeploymentCard for {key}");
245
246
            }
        };
247
        let model_name = card.name().to_string();
248
        let active_instances = self
249
            .cards_for_model(&model_name, target_namespace, is_global_namespace)
250
251
252
            .await
            .with_context(|| model_name.clone())?;
        if !active_instances.is_empty() {
253
254
255
256
257
258
            tracing::debug!(
                model_name,
                target_namespace = ?target_namespace,
                active_instance_count = active_instances.len(),
                "Model has other active instances, not removing"
            );
259
260
            return Ok(None);
        }
261

262
        // Ignore the errors because model could be either type
263
264
265
        let chat_model_remove_err = self.manager.remove_chat_completions_model(&model_name);
        let completions_model_remove_err = self.manager.remove_completions_model(&model_name);
        let embeddings_model_remove_err = self.manager.remove_embeddings_model(&model_name);
266
        let tensor_model_remove_err = self.manager.remove_tensor_model(&model_name);
267
        let prefill_model_remove_err = self.manager.remove_prefill_model(&model_name);
268
269
270
271

        let mut chat_model_removed = false;
        let mut completions_model_removed = false;
        let mut embeddings_model_removed = false;
272
        let mut tensor_model_removed = false;
273
        let mut prefill_model_removed = false;
274
275
276
277
278
279
280
281
282
283
284

        if chat_model_remove_err.is_ok() && self.manager.list_chat_completions_models().is_empty() {
            chat_model_removed = true;
        }
        if completions_model_remove_err.is_ok() && self.manager.list_completions_models().is_empty()
        {
            completions_model_removed = true;
        }
        if embeddings_model_remove_err.is_ok() && self.manager.list_embeddings_models().is_empty() {
            embeddings_model_removed = true;
        }
285
286
287
        if tensor_model_remove_err.is_ok() && self.manager.list_tensor_models().is_empty() {
            tensor_model_removed = true;
        }
288
289
290
        if prefill_model_remove_err.is_ok() && self.manager.list_prefill_models().is_empty() {
            prefill_model_removed = true;
        }
291

292
293
294
295
        if !chat_model_removed
            && !completions_model_removed
            && !embeddings_model_removed
            && !tensor_model_removed
296
            && !prefill_model_removed
297
        {
298
            tracing::debug!(
299
                "No updates to send for model {}: chat_model_removed: {}, completions_model_removed: {}, embeddings_model_removed: {}, tensor_model_removed: {}, prefill_model_removed: {}",
300
301
302
                model_name,
                chat_model_removed,
                completions_model_removed,
303
                embeddings_model_removed,
304
305
                tensor_model_removed,
                prefill_model_removed
306
307
308
            );
        } else {
            for model_type in ALL_MODEL_TYPES {
309
                if ((chat_model_removed && *model_type == ModelType::Chat)
310
                    || (completions_model_removed && *model_type == ModelType::Completions)
311
                    || (embeddings_model_removed && *model_type == ModelType::Embedding)
312
313
                    || (tensor_model_removed && *model_type == ModelType::TensorBased)
                    || (prefill_model_removed && *model_type == ModelType::Prefill))
314
                    && let Some(tx) = &self.model_update_tx
315
                {
316
                    tx.send(ModelUpdate::Removed(card.clone())).await.ok();
317
318
319
                }
            }
        }
320

321
        Ok(Some(model_name))
322
    }
Ryan Olson's avatar
Ryan Olson committed
323

324
    // Handles a PUT event from store, this usually means adding a new model to the list of served
325
    // models.
326
327
328
329
330
331
    async fn handle_put(
        &self,
        key: &str,
        endpoint_id: &EndpointId,
        card: &mut ModelDeploymentCard,
    ) -> anyhow::Result<()> {
332
333
        card.download_config().await?;

334
        let component = self
335
336
            .drt
            .namespace(&endpoint_id.namespace)?
337
            .component(&endpoint_id.component)?;
338
339
        let endpoint = component.endpoint(&endpoint_id.name);
        let client = endpoint.client().await?;
340
341
        tracing::debug!(model_name = card.name(), "adding model");
        self.manager.save_model_card(key, card.clone())?;
342

343
344
345
346
347
348
349
350
351
352
        // Skip duplicate registrations based on model type.
        // Prefill and decode models are tracked separately, so registering one
        // doesn't block the other (they can arrive in any order).
        let already_registered = if card.model_type.supports_prefill() {
            self.manager.has_prefill_model(card.name())
        } else {
            self.manager.has_decode_model(card.name())
        };

        if already_registered {
353
354
355
            tracing::debug!(
                model_name = card.name(),
                namespace = endpoint_id.namespace,
356
                model_type = %card.model_type,
357
                "Model already registered, skipping"
358
359
360
361
362
363
364
            );
            return Ok(());
        }

        if let Some(tx) = &self.model_update_tx {
            tx.send(ModelUpdate::Added(card.clone())).await.ok();
        }
365

366
        let checksum = card.mdcsum();
367
368
369

        if card.model_input == ModelInput::Tokens
            && (card.model_type.supports_chat() || card.model_type.supports_completions())
370
371
372
373
374
        {
            // Case 1: Tokens + (Chat OR Completions OR Both)
            // A model that expects pre-processed requests meaning it's up to us whether we
            // handle Chat or Completions requests, so handle whatever the model supports.

375
            let endpoint = component.endpoint(&endpoint_id.name);
376
            let kv_chooser = if self.router_config.router_mode == RouterMode::KV {
377
378
                Some(
                    self.manager
379
380
381
382
383
                        .kv_chooser_for(
                            &endpoint,
                            card.kv_cache_block_size,
                            Some(self.router_config.kv_router_config),
                        )
384
385
386
387
388
                        .await?,
                )
            } else {
                None
            };
389

390
            // This is expensive, we are loading ~10MiB JSON, so only do it once
391
            let tokenizer_hf = card.tokenizer_hf().context("tokenizer_hf")?;
392

393
394
395
396
397
398
399
            // Create prefill chooser once if we're building pipelines
            // Both chat and completions will share the same prefill chooser instance
            let prefill_chooser = self
                .manager
                .register_prefill_router(card.name().to_string())
                .map(|rx| {
                    // Create prefill-specific config with track_active_blocks disabled
400
                    let mut prefill_config = self.router_config.kv_router_config;
401
402
403
404
405
                    prefill_config.router_track_active_blocks = false;

                    PrefillRouter::new(
                        rx,
                        self.manager.clone(),
406
                        self.router_config.router_mode,
407
408
                        card.kv_cache_block_size,
                        Some(prefill_config),
409
                        self.router_config.enforce_disagg,
410
411
412
                    )
                });

413
414
            // Get or create the worker monitor for this model
            // This allows dynamic threshold updates via the ModelManager
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
            // Create monitor if either threshold is configured
            let worker_monitor = if self.router_config.active_decode_blocks_threshold.is_some()
                || self.router_config.active_prefill_tokens_threshold.is_some()
            {
                // Default thresholds: active_decode_blocks=1.0 (disabled), active_prefill_tokens=1000000 (effectively disabled)
                let active_decode_blocks = self
                    .router_config
                    .active_decode_blocks_threshold
                    .unwrap_or(1.0);
                let active_prefill_tokens = self
                    .router_config
                    .active_prefill_tokens_threshold
                    .unwrap_or(1000000);
                Some(self.manager.get_or_create_worker_monitor(
                    card.name(),
                    client.clone(),
                    active_decode_blocks,
                    active_prefill_tokens,
                ))
            } else {
                None
            };
437

438
            // Add chat engine only if the model supports chat
439
            if card.model_type.supports_chat() {
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
                // Work in progress. This will allow creating  a chat_engine from Python.
                let chat_engine = if let Some(ref factory) = self.engine_factory {
                    factory(card.clone())
                        .await
                        .context("python engine_factory")?
                } else {
                    entrypoint::build_routed_pipeline::<
                        NvCreateChatCompletionRequest,
                        NvCreateChatCompletionStreamResponse,
                    >(
                        card,
                        &client,
                        self.router_config.router_mode,
                        worker_monitor.clone(),
                        kv_chooser.clone(),
                        tokenizer_hf.clone(),
                        prefill_chooser.clone(),
                        self.router_config.enforce_disagg,
458
                        self.metrics.clone(),
459
460
461
462
                    )
                    .await
                    .context("build_routed_pipeline")?
                };
463
                self.manager
464
                    .add_chat_completions_model(card.name(), checksum, chat_engine)
465
                    .context("add_chat_completions_model")?;
466
                tracing::info!("Chat completions is ready");
467
            }
468

469
            // Add completions engine only if the model supports completions
470
            if card.model_type.supports_completions() {
471
472
                let formatter = PromptFormatter::no_op();
                let PromptFormatter::OAI(formatter) = formatter;
473
474
475
476
                let preprocessor = OpenAIPreprocessor::new_with_parts(
                    card.clone(),
                    formatter,
                    tokenizer_hf.clone(),
477
478
                )
                .context("OpenAIPreprocessor::new_with_parts")?;
479
                let completions_engine = entrypoint::build_routed_pipeline_with_preprocessor::<
480
481
482
                    NvCreateCompletionRequest,
                    NvCreateCompletionResponse,
                >(
483
                    card,
484
                    &client,
485
                    self.router_config.router_mode,
486
                    worker_monitor,
487
                    kv_chooser,
488
                    preprocessor,
489
                    tokenizer_hf,
490
                    prefill_chooser,
491
                    self.router_config.enforce_disagg,
492
                    self.metrics.clone(),
493
                )
494
495
                .await
                .context("build_routed_pipeline_with_preprocessor")?;
496
                self.manager
497
                    .add_completions_model(card.name(), checksum, completions_engine)
498
                    .context("add_completions_model")?;
499
                tracing::info!("Completions is ready");
500
            }
501
502
503
504
505
506
        } else if card.model_input == ModelInput::Text && card.model_type.supports_embedding() {
            // Case: Text + Embeddings
            let push_router = PushRouter::<
                NvCreateEmbeddingRequest,
                Annotated<NvCreateEmbeddingResponse>,
            >::from_client_with_threshold(
507
                client, self.router_config.router_mode, None, None
508
509
510
511
            )
            .await?;
            let engine = Arc::new(push_router);
            self.manager
512
                .add_embeddings_model(card.name(), checksum, engine)?;
513
        } else if card.model_input == ModelInput::Text && card.model_type.supports_chat() {
514
            // Case 3: Text + Chat
515
516
517
518
519
520
521
            let push_router = PushRouter::<
                NvCreateChatCompletionRequest,
                Annotated<NvCreateChatCompletionStreamResponse>,
            >::from_client_with_threshold(
                client, self.router_config.router_mode, None, None
            )
            .await?;
522
523
            let engine = Arc::new(push_router);
            self.manager
524
                .add_chat_completions_model(card.name(), checksum, engine)?;
525
        } else if card.model_input == ModelInput::Text && card.model_type.supports_completions() {
526
527
528
529
530
            // Case 2: Text + Completions
            let push_router = PushRouter::<
                NvCreateCompletionRequest,
                Annotated<NvCreateCompletionResponse>,
            >::from_client_with_threshold(
531
                client, self.router_config.router_mode, None, None
532
533
534
535
            )
            .await?;
            let engine = Arc::new(push_router);
            self.manager
536
                .add_completions_model(card.name(), checksum, engine)?;
537
        } else if card.model_input == ModelInput::Tokens && card.model_type.supports_embedding() {
538
539
540
541
542
543
544
545
            // Case 4: Tokens + Embeddings

            // Create preprocessing pipeline similar to Backend
            let frontend = SegmentSource::<
                SingleIn<NvCreateEmbeddingRequest>,
                ManyOut<Annotated<NvCreateEmbeddingResponse>>,
            >::new();

546
            let preprocessor = OpenAIPreprocessor::new(card.clone())?.into_operator();
547
            let backend = Backend::from_mdc(card).into_operator();
548
549
550
551
552

            let router = PushRouter::<
                PreprocessedEmbeddingRequest,
                Annotated<EmbeddingsEngineOutput>,
            >::from_client_with_threshold(
553
                client, self.router_config.router_mode, None, None
554
555
556
            )
            .await?;

557
            // Note: Embeddings don't need KV routing complexity or load monitoring
558
559
560
561
562
563
564
565
566
567
568
569
            let service_backend = ServiceBackend::from_engine(Arc::new(router));

            // Link the pipeline: frontend -> preprocessor -> backend -> service_backend -> backend -> preprocessor -> frontend
            let embedding_engine = frontend
                .link(preprocessor.forward_edge())?
                .link(backend.forward_edge())?
                .link(service_backend)?
                .link(backend.backward_edge())?
                .link(preprocessor.backward_edge())?
                .link(frontend)?;

            self.manager
570
                .add_embeddings_model(card.name(), checksum, embedding_engine)?;
571
        } else if card.model_input == ModelInput::Tensor && card.model_type.supports_tensor() {
572
            // Case 5: Tensor + Tensor (non-LLM)
573
            // No KV cache concepts - not an LLM model
574
575
576
577
            let push_router = PushRouter::<
                NvCreateTensorRequest,
                Annotated<NvCreateTensorResponse>,
            >::from_client_with_threshold(
578
                client, self.router_config.router_mode, None, None
579
580
581
            )
            .await?;
            let engine = Arc::new(push_router);
582
583
            self.manager
                .add_tensor_model(card.name(), checksum, engine)?;
584
585
586
587
588
589
590
591
592
593
594
595
        } else if card.model_type.supports_prefill() {
            // Case 6: Prefill
            // Guardrail: Verify model_input is Tokens
            if card.model_input != ModelInput::Tokens {
                anyhow::bail!(
                    "Prefill models must use ModelInput::Tokens, got {}",
                    card.model_input.as_str()
                );
            }

            tracing::info!(
                model_name = card.name(),
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
                "Prefill model detected, registering and activating prefill router"
            );

            // Register prefill model for tracking (no engine needed, just lifecycle)
            self.manager
                .add_prefill_model(card.name(), checksum)
                .context("add_prefill_model")?;

            // Activate the prefill router with the endpoint for this prefill model
            let Ok(()) = self.manager.activate_prefill_router(card.name(), endpoint) else {
                tracing::warn!(
                    model_name = card.name(),
                    "Failed to activate prefill router - prefill model may already be activated"
                );
                return Ok(());
            };

            tracing::info!(
                model_name = card.name(),
                "Prefill model registered and router activated successfully"
616
            );
617
618
619
620
        } else {
            // Reject unsupported combinations
            anyhow::bail!(
                "Unsupported model configuration: {} with {} input. Supported combinations: \
621
                Tokens+(Chat|Completions|Prefill), Text+Chat, Text+Completions, Tokens+Embeddings, Tensor+TensorBased",
622
623
                card.model_type,
                card.model_input.as_str()
624
            );
625
        }
Ryan Olson's avatar
Ryan Olson committed
626

627
628
        Ok(())
    }
629

630
    /// All the registered ModelDeploymentCard with the EndpointId they are attached to, one per instance
631
    async fn all_cards(&self) -> anyhow::Result<Vec<(EndpointId, ModelDeploymentCard)>> {
632
633
        let discovery = self.drt.discovery();
        let instances = discovery.list(DiscoveryQuery::AllModels).await?;
634

635
636
637
        let mut results = Vec::with_capacity(instances.len());
        for instance in instances {
            match instance.deserialize_model::<ModelDeploymentCard>() {
638
                Ok(card) => {
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
                    // Extract EndpointId from the instance
                    let endpoint_id = match &instance {
                        dynamo_runtime::discovery::DiscoveryInstance::Model {
                            namespace,
                            component,
                            endpoint,
                            ..
                        } => EndpointId {
                            namespace: namespace.clone(),
                            component: component.clone(),
                            name: endpoint.clone(),
                        },
                        _ => {
                            tracing::error!(
                                "Unexpected discovery instance type (expected ModelCard)"
                            );
655
656
657
                            continue;
                        }
                    };
658
                    results.push((endpoint_id, card));
659
                }
660
                Err(err) => {
661
                    tracing::error!(%err, "Failed to deserialize model card");
662
663
                    continue;
                }
664
            }
665
        }
666
        Ok(results)
667
668
    }

669
    pub async fn cards_for_model(
670
671
672
673
        &self,
        model_name: &str,
        target_namespace: Option<&str>,
        is_global_namespace: bool,
674
675
676
677
    ) -> anyhow::Result<Vec<ModelDeploymentCard>> {
        let mut all = self.all_cards().await?;
        all.retain(|(endpoint_id, card)| {
            let matches_name = card.name() == model_name;
678
679
680
            let matches_namespace = match (is_global_namespace, target_namespace) {
                (true, _) => true,
                (false, None) => true,
681
                (false, Some(target_ns)) => endpoint_id.namespace == target_ns,
682
683
684
            };
            matches_name && matches_namespace
        });
685
686
687
        Ok(all.into_iter().map(|(_eid, card)| card).collect())
    }
}