distributed.rs 22.8 KB
Newer Older
1
2
// 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
3

4
use crate::component::{Component, Instance};
5
use crate::pipeline::PipelineError;
6
use crate::storage::key_value_store::{
7
8
    EtcdStore, KeyValueStore, KeyValueStoreEnum, KeyValueStoreManager, KeyValueStoreSelect,
    MemoryStore,
9
};
10
use crate::transports::nats::DRTNatsClientPrometheusMetrics;
Ryan Olson's avatar
Ryan Olson committed
11
use crate::{
12
    component::{self, ComponentBuilder, Endpoint, Namespace},
13
    discovery::Discovery,
14
15
    metrics::PrometheusUpdateCallback,
    metrics::{MetricsHierarchy, MetricsRegistry},
Ryan Olson's avatar
Ryan Olson committed
16
17
    transports::{etcd, nats, tcp},
};
18
use crate::{discovery, system_status_server, transports};
Ryan Olson's avatar
Ryan Olson committed
19

20
use super::utils::GracefulShutdownTracker;
21
22
use crate::SystemHealth;
use crate::runtime::Runtime;
Ryan Olson's avatar
Ryan Olson committed
23

24
use async_once_cell::OnceCell;
25
use std::fmt;
26
use std::sync::{Arc, OnceLock, Weak};
27
use tokio::sync::watch::Receiver;
28
29

use anyhow::Result;
Ryan Olson's avatar
Ryan Olson committed
30
31
use derive_getters::Dissolve;
use figment::error;
32
33
use std::collections::HashMap;
use tokio::sync::Mutex;
34
use tokio_util::sync::CancellationToken;
Ryan Olson's avatar
Ryan Olson committed
35

36
37
type InstanceMap = HashMap<Endpoint, Weak<Receiver<Vec<Instance>>>>;

38
39
40
41
42
43
44
45
46
47
/// Distributed [Runtime] which provides access to shared resources across the cluster, this includes
/// communication protocols and transports.
#[derive(Clone)]
pub struct DistributedRuntime {
    // local runtime
    runtime: Runtime,

    nats_client: Option<transports::nats::Client>,
    store: KeyValueStoreManager,
    tcp_server: Arc<OnceCell<Arc<transports::tcp::server::TcpStreamServer>>>,
48
    network_manager: Arc<OnceCell<Arc<crate::pipeline::network::manager::NetworkManager>>>,
49
    system_status_server: Arc<OnceLock<Arc<system_status_server::SystemStatusServerInfo>>>,
50
    request_plane: RequestPlaneMode,
51
52
53
54

    // Service discovery client
    discovery_client: Arc<dyn discovery::Discovery>,

55
56
57
58
    // Discovery metadata (only used for Kubernetes backend)
    // Shared with system status server to expose via /metadata endpoint
    discovery_metadata: Option<Arc<tokio::sync::RwLock<discovery::DiscoveryMetadata>>>,

59
60
61
62
63
64
65
    // local registry for components
    // the registry allows us to use share runtime resources across instances of the same component object.
    // take for example two instances of a client to the same remote component. The registry allows us to use
    // a single endpoint watcher for both clients, this keeps the number background tasking watching specific
    // paths in etcd to a minimum.
    component_registry: component::Registry,

66
    instance_sources: Arc<tokio::sync::Mutex<InstanceMap>>,
67
68
69
70
71
72
73
74

    // Health Status
    system_health: Arc<parking_lot::Mutex<SystemHealth>>,

    // This hierarchy's own metrics registry
    metrics_registry: MetricsRegistry,
}

75
impl MetricsHierarchy for DistributedRuntime {
76
77
78
79
    fn basename(&self) -> String {
        "".to_string() // drt has no basename. Basename only begins with the Namespace.
    }

80
81
82
83
84
85
    fn parent_hierarchies(&self) -> Vec<&dyn MetricsHierarchy> {
        vec![] // drt is the root, so no parent hierarchies
    }

    fn get_metrics_registry(&self) -> &MetricsRegistry {
        &self.metrics_registry
86
87
88
    }
}

Ryan Olson's avatar
Ryan Olson committed
89
90
91
92
93
94
impl std::fmt::Debug for DistributedRuntime {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        write!(f, "DistributedRuntime")
    }
}

Ryan Olson's avatar
Ryan Olson committed
95
96
impl DistributedRuntime {
    pub async fn new(runtime: Runtime, config: DistributedConfig) -> Result<Self> {
97
        let (selected_kv_store, nats_config, request_plane) = config.dissolve();
Ryan Olson's avatar
Ryan Olson committed
98
99
100

        let runtime_clone = runtime.clone();

101
        let store = match selected_kv_store {
102
            KeyValueStoreSelect::Etcd(etcd_config) => {
103
104
105
106
                let etcd_client = etcd::Client::new(*etcd_config, runtime_clone).await.inspect_err(|err|
                    // The returned error doesn't show because of a dropped runtime error, so
                    // log it first.
                    tracing::error!(%err, "Could not connect to etcd. Pass `--store-kv ..` to use a different backend or start etcd."))?;
107
                KeyValueStoreManager::etcd(etcd_client)
108
            }
109
110
            KeyValueStoreSelect::File(root) => KeyValueStoreManager::file(root),
            KeyValueStoreSelect::Memory => KeyValueStoreManager::memory(),
111
        };
Ryan Olson's avatar
Ryan Olson committed
112

113
114
115
116
        let nats_client = match nats_config {
            Some(nc) => Some(nc.connect().await?),
            None => None,
        };
Ryan Olson's avatar
Ryan Olson committed
117

118
        // Start system status server for health and metrics if enabled in configuration
119
120
121
122
123
124
125
126
        let config = crate::config::RuntimeConfig::from_settings().unwrap_or_default();
        // IMPORTANT: We must extract cancel_token from runtime BEFORE moving runtime into the struct below.
        // This is because after moving, runtime is no longer accessible in this scope (ownership rules).
        let cancel_token = if config.system_server_enabled() {
            Some(runtime.clone().child_token())
        } else {
            None
        };
127
128
        let starting_health_status = config.starting_health_status.clone();
        let use_endpoint_health_status = config.use_endpoint_health_status.clone();
129
130
        let health_endpoint_path = config.system_health_path.clone();
        let live_endpoint_path = config.system_live_path.clone();
131
        let system_health = Arc::new(parking_lot::Mutex::new(SystemHealth::new(
132
133
            starting_health_status,
            use_endpoint_health_status,
134
135
            health_endpoint_path,
            live_endpoint_path,
136
        )));
137

138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
        // Initialize discovery client based on backend configuration
        let discovery_backend =
            std::env::var("DYN_DISCOVERY_BACKEND").unwrap_or_else(|_| "kv_store".to_string());

        let (discovery_client, discovery_metadata) = match discovery_backend.as_str() {
            "kubernetes" => {
                tracing::info!("Initializing Kubernetes discovery backend");
                let metadata = Arc::new(tokio::sync::RwLock::new(
                    crate::discovery::DiscoveryMetadata::new(),
                ));
                let client = crate::discovery::KubeDiscoveryClient::new(
                    metadata.clone(),
                    runtime.primary_token(),
                )
                .await
                .inspect_err(
                    |err| tracing::error!(%err, "Failed to initialize Kubernetes discovery client"),
                )?;
                (Arc::new(client) as Arc<dyn Discovery>, Some(metadata))
            }
            _ => {
                tracing::info!("Initializing KV store discovery backend");
                use crate::discovery::KVStoreDiscovery;
                (
                    Arc::new(KVStoreDiscovery::new(
                        store.clone(),
                        runtime.primary_token(),
                    )) as Arc<dyn Discovery>,
                    None,
                )
            }
169
170
        };

171
172
        let nats_client_for_metrics = nats_client.clone();

173
        let distributed_runtime = Self {
Ryan Olson's avatar
Ryan Olson committed
174
            runtime,
175
            store,
Ryan Olson's avatar
Ryan Olson committed
176
177
            nats_client,
            tcp_server: Arc::new(OnceCell::new()),
178
            network_manager: Arc::new(OnceCell::new()),
179
            system_status_server: Arc::new(OnceLock::new()),
180
            discovery_client,
181
            discovery_metadata,
Ryan Olson's avatar
Ryan Olson committed
182
            component_registry: component::Registry::new(),
183
            instance_sources: Arc::new(Mutex::new(HashMap::new())),
184
            metrics_registry: crate::MetricsRegistry::new(),
185
            system_health,
186
            request_plane,
187
188
        };

189
190
191
192
193
        if let Some(nats_client_for_metrics) = nats_client_for_metrics {
            let nats_client_metrics = DRTNatsClientPrometheusMetrics::new(
                &distributed_runtime,
                nats_client_for_metrics.client().clone(),
            )?;
194
            // Register a callback to update NATS client metrics on the DRT's metrics registry
195
196
197
198
199
200
201
202
            let nats_client_callback = Arc::new({
                let nats_client_clone = nats_client_metrics.clone();
                move || {
                    nats_client_clone.set_from_client_stats();
                    Ok(())
                }
            });
            distributed_runtime
203
204
                .metrics_registry
                .add_update_callback(nats_client_callback);
205
        }
206

207
208
209
210
211
212
        // Initialize the uptime gauge in SystemHealth
        distributed_runtime
            .system_health
            .lock()
            .initialize_uptime_gauge(&distributed_runtime)?;

213
        // Handle system status server initialization
214
        if let Some(cancel_token) = cancel_token {
215
            // System server is enabled - start both the state and HTTP server
216
            let host = config.system_host.clone();
217
            let port = config.system_port as u16;
218

219
            // Start system status server (it creates SystemStatusState internally)
220
            match crate::system_status_server::spawn_system_status_server(
221
222
223
224
                &host,
                port,
                cancel_token,
                Arc::new(distributed_runtime.clone()),
225
                distributed_runtime.discovery_metadata.clone(),
226
227
228
            )
            .await
            {
229
                Ok((addr, handle)) => {
230
                    tracing::info!("System status server started successfully on {}", addr);
231

232
233
234
235
236
237
                    // Store system status server information
                    let system_status_server_info =
                        crate::system_status_server::SystemStatusServerInfo::new(
                            addr,
                            Some(handle),
                        );
238

239
                    // Initialize the system_status_server field
240
                    distributed_runtime
241
242
243
                        .system_status_server
                        .set(Arc::new(system_status_server_info))
                        .expect("System status server info should only be set once");
244
245
                }
                Err(e) => {
246
                    tracing::error!("System status server startup failed: {}", e);
247
                }
248
            }
249
        } else {
250
            // System server HTTP is disabled, but uptime metrics are still being tracked via SystemHealth
251
252
253
            tracing::debug!(
                "System status server HTTP endpoints disabled, but uptime metrics are being tracked"
            );
254
255
        }

256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
        // Start health check manager if enabled
        if config.health_check_enabled {
            let health_check_config = crate::health_check::HealthCheckConfig {
                canary_wait_time: std::time::Duration::from_secs(config.canary_wait_time_secs),
                request_timeout: std::time::Duration::from_secs(
                    config.health_check_request_timeout_secs,
                ),
            };

            // Start the health check manager (spawns per-endpoint monitoring tasks)
            match crate::health_check::start_health_check_manager(
                distributed_runtime.clone(),
                Some(health_check_config),
            )
            .await
            {
                Ok(()) => tracing::info!(
                    "Health check manager started (canary_wait_time: {}s, request_timeout: {}s)",
                    config.canary_wait_time_secs,
                    config.health_check_request_timeout_secs
                ),
                Err(e) => tracing::error!("Health check manager failed to start: {}", e),
            }
        }

281
        Ok(distributed_runtime)
Ryan Olson's avatar
Ryan Olson committed
282
283
284
    }

    pub async fn from_settings(runtime: Runtime) -> Result<Self> {
285
        let config = DistributedConfig::from_settings();
Ryan Olson's avatar
Ryan Olson committed
286
287
288
289
290
291
292
        Self::new(runtime, config).await
    }

    pub fn runtime(&self) -> &Runtime {
        &self.runtime
    }

293
294
295
296
    pub fn primary_token(&self) -> CancellationToken {
        self.runtime.primary_token()
    }

297
298
299
300
301
302
303
304
305
306
307
    // TODO: Don't hand out pointers, instead have methods to use the registry in friendly ways
    // (without being aware of async locks and so on)
    pub fn component_registry(&self) -> &component::Registry {
        &self.component_registry
    }

    // TODO: Don't hand out pointers, instead provide system health related services.
    pub fn system_health(&self) -> Arc<parking_lot::Mutex<SystemHealth>> {
        self.system_health.clone()
    }

308
    pub fn connection_id(&self) -> u64 {
309
        self.discovery_client.instance_id()
Ryan Olson's avatar
Ryan Olson committed
310
311
312
313
    }

    pub fn shutdown(&self) {
        self.runtime.shutdown();
314
        self.store.shutdown();
Ryan Olson's avatar
Ryan Olson committed
315
316
317
318
    }

    /// Create a [`Namespace`]
    pub fn namespace(&self, name: impl Into<String>) -> Result<Namespace> {
319
        Namespace::new(self.clone(), name.into())
Ryan Olson's avatar
Ryan Olson committed
320
321
    }

322
323
324
    /// Returns the discovery interface for service registration and discovery
    pub fn discovery(&self) -> Arc<dyn Discovery> {
        self.discovery_client.clone()
325
326
    }

327
    pub async fn tcp_server(&self) -> Result<Arc<tcp::server::TcpStreamServer>> {
Ryan Olson's avatar
Ryan Olson committed
328
329
330
331
332
        Ok(self
            .tcp_server
            .get_or_try_init(async move {
                let options = tcp::server::ServerOptions::default();
                let server = tcp::server::TcpStreamServer::new(options).await?;
333
                Ok::<_, PipelineError>(server)
Ryan Olson's avatar
Ryan Olson committed
334
335
336
337
338
            })
            .await?
            .clone())
    }

339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
    /// Get the network manager (lazy initialization)
    ///
    /// The network manager consolidates all network configuration and provides
    /// unified access to request plane servers and clients.
    pub async fn network_manager(
        &self,
    ) -> Result<Arc<crate::pipeline::network::manager::NetworkManager>> {
        use crate::pipeline::network::manager::NetworkManager;

        let manager = self
            .network_manager
            .get_or_try_init(async {
                // Get NATS client if available
                let nats_client = self.nats_client().map(|c| c.client().clone());

                // NetworkManager handles all config reading and mode selection
                anyhow::Ok(NetworkManager::new(
                    self.child_token(),
                    nats_client,
                    self.component_registry.clone(),
359
                    self.request_plane,
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
                ))
            })
            .await?;

        Ok(manager.clone())
    }

    /// Get the request plane server (convenience method)
    ///
    /// This is a shortcut for `network_manager().await?.server().await`.
    pub async fn request_plane_server(
        &self,
    ) -> Result<Arc<dyn crate::pipeline::network::ingress::unified_server::RequestPlaneServer>>
    {
        let manager = self.network_manager().await?;
        manager.server().await
    }

378
    pub(crate) fn nats_client(&self) -> Option<&nats::Client> {
379
        self.nats_client.as_ref()
Ryan Olson's avatar
Ryan Olson committed
380
381
    }

382
383
384
385
386
    /// Get system status server information if available
    pub fn system_status_server_info(
        &self,
    ) -> Option<Arc<crate::system_status_server::SystemStatusServerInfo>> {
        self.system_status_server.get().cloned()
387
388
    }

389
    /// An interface to store things outside of the process. Usually backed by something like etcd.
390
    /// Currently does key-value, but will grow to include whatever we need to store.
391
392
    pub fn store(&self) -> &KeyValueStoreManager {
        &self.store
393
394
    }

395
396
397
398
399
    /// How the frontend should talk to the backend.
    pub fn request_plane(&self) -> RequestPlaneMode {
        self.request_plane
    }

400
401
402
    pub fn child_token(&self) -> CancellationToken {
        self.runtime.child_token()
    }
403

404
405
406
407
    pub(crate) fn graceful_shutdown_tracker(&self) -> Arc<GracefulShutdownTracker> {
        self.runtime.graceful_shutdown_tracker()
    }

408
    pub fn instance_sources(&self) -> Arc<Mutex<InstanceMap>> {
409
410
        self.instance_sources.clone()
    }
Ryan Olson's avatar
Ryan Olson committed
411
412
413
414
}

#[derive(Dissolve)]
pub struct DistributedConfig {
415
    pub store_backend: KeyValueStoreSelect,
416
    pub nats_config: Option<nats::ClientOptions>,
417
    pub request_plane: RequestPlaneMode,
Ryan Olson's avatar
Ryan Olson committed
418
419
420
}

impl DistributedConfig {
421
    pub fn from_settings() -> DistributedConfig {
422
        let request_plane = RequestPlaneMode::from_env();
Ryan Olson's avatar
Ryan Olson committed
423
        DistributedConfig {
424
            store_backend: KeyValueStoreSelect::Etcd(Box::default()),
425
426
427
428
429
430
            nats_config: if request_plane.is_nats() {
                Some(nats::ClientOptions::default())
            } else {
                None
            },
            request_plane,
Ryan Olson's avatar
Ryan Olson committed
431
432
        }
    }
Ryan Olson's avatar
Ryan Olson committed
433
434

    pub fn for_cli() -> DistributedConfig {
435
436
437
438
        let etcd_config = etcd::ClientOptions {
            attach_lease: false,
            ..Default::default()
        };
439
        let request_plane = RequestPlaneMode::from_env();
440
441
        DistributedConfig {
            store_backend: KeyValueStoreSelect::Etcd(Box::new(etcd_config)),
442
443
444
445
446
447
            nats_config: if request_plane.is_nats() {
                Some(nats::ClientOptions::default())
            } else {
                None
            },
            request_plane,
448
449
        }
    }
450
451
452
453
454
455
456
457
458
459
460
461

    /// A DistributedConfig that isn't distributed, for when the frontend and backend are in the
    /// same process.
    pub fn process_local() -> DistributedConfig {
        DistributedConfig {
            store_backend: KeyValueStoreSelect::Memory,
            nats_config: None,
            // This won't be used in process local, so we likely need a "none" option to
            // communicate that and avoid opening the ports.
            request_plane: RequestPlaneMode::Tcp,
        }
    }
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
}

/// Request plane transport mode configuration
///
/// This determines how requests are distributed from routers to workers:
/// - `Nats`: Use NATS for request distribution (default, legacy)
/// - `Http`: Use HTTP/2 for request distribution
/// - `Tcp`: Use raw TCP for request distribution with msgpack support
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum RequestPlaneMode {
    /// Use NATS for request plane (default for backward compatibility)
    Nats,
    /// Use HTTP/2 for request plane
    Http,
    /// Use raw TCP for request plane with msgpack support
    Tcp,
}

impl Default for RequestPlaneMode {
    fn default() -> Self {
        Self::Nats
    }
}

impl fmt::Display for RequestPlaneMode {
    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
        match self {
            Self::Nats => write!(f, "nats"),
            Self::Http => write!(f, "http"),
            Self::Tcp => write!(f, "tcp"),
        }
    }
}

impl std::str::FromStr for RequestPlaneMode {
    type Err = anyhow::Error;

    fn from_str(s: &str) -> std::result::Result<Self, Self::Err> {
        match s.to_lowercase().as_str() {
            "nats" => Ok(Self::Nats),
            "http" => Ok(Self::Http),
            "tcp" => Ok(Self::Tcp),
            _ => Err(anyhow::anyhow!(
                "Invalid request plane mode: '{}'. Valid options are: 'nats', 'http', 'tcp'",
                s
            )),
508
        }
Ryan Olson's avatar
Ryan Olson committed
509
    }
Ryan Olson's avatar
Ryan Olson committed
510
}
511

512
513
514
515
516
517
518
519
520
impl RequestPlaneMode {
    /// Get the request plane mode from environment variable (uncached)
    /// Reads from `DYN_REQUEST_PLANE` environment variable.
    fn from_env() -> Self {
        std::env::var("DYN_REQUEST_PLANE")
            .ok()
            .and_then(|s| s.parse().ok())
            .unwrap_or_default()
    }
521
522
523
524

    pub fn is_nats(&self) -> bool {
        matches!(self, RequestPlaneMode::Nats)
    }
525
526
}

527
pub mod distributed_test_utils {
528
529
    //! Common test helper functions for DistributedRuntime tests

530
    /// Helper function to create a DRT instance for integration-only tests.
531
    /// Uses from_current to leverage existing tokio runtime
532
    /// Note: Settings are read from environment variables inside DistributedRuntime::from_settings
533
    #[cfg(feature = "integration")]
534
535
536
    pub async fn create_test_drt_async() -> super::DistributedRuntime {
        use crate::{storage::key_value_store::KeyValueStoreSelect, transports::nats};

537
        let rt = crate::Runtime::from_current().unwrap();
538
539
        let config = super::DistributedConfig {
            store_backend: KeyValueStoreSelect::Memory,
540
            nats_config: Some(nats::ClientOptions::default()),
541
            request_plane: crate::distributed::RequestPlaneMode::default(),
542
543
        };
        super::DistributedRuntime::new(rt, config).await.unwrap()
544
545
    }
}
546

547
#[cfg(all(test, feature = "integration"))]
548
mod tests {
549
    use super::RequestPlaneMode;
550
551
552
553
    use super::distributed_test_utils::create_test_drt_async;

    #[tokio::test]
    async fn test_drt_uptime_after_delay_system_disabled() {
554
        use crate::config::environment_names::runtime::system as env_system;
555
        // Test uptime with system status server disabled
556
        temp_env::async_with_vars([(env_system::DYN_SYSTEM_PORT, None::<&str>)], async {
557
558
559
560
561
562
563
            // Start a DRT
            let drt = create_test_drt_async().await;

            // Wait 50ms
            tokio::time::sleep(tokio::time::Duration::from_millis(50)).await;

            // Check that uptime is 50+ ms
564
            let uptime = drt.system_health.lock().uptime();
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
            assert!(
                uptime >= std::time::Duration::from_millis(50),
                "Expected uptime to be at least 50ms, but got {:?}",
                uptime
            );

            println!(
                "✓ DRT uptime test passed (system disabled): uptime = {:?}",
                uptime
            );
        })
        .await;
    }

    #[tokio::test]
    async fn test_drt_uptime_after_delay_system_enabled() {
581
        use crate::config::environment_names::runtime::system as env_system;
582
        // Test uptime with system status server enabled
583
        temp_env::async_with_vars([(env_system::DYN_SYSTEM_PORT, Some("8081"))], async {
584
585
586
587
588
589
590
            // Start a DRT
            let drt = create_test_drt_async().await;

            // Wait 50ms
            tokio::time::sleep(tokio::time::Duration::from_millis(50)).await;

            // Check that uptime is 50+ ms
591
            let uptime = drt.system_health.lock().uptime();
592
593
594
595
596
597
598
599
600
601
602
603
604
            assert!(
                uptime >= std::time::Duration::from_millis(50),
                "Expected uptime to be at least 50ms, but got {:?}",
                uptime
            );

            println!(
                "✓ DRT uptime test passed (system enabled): uptime = {:?}",
                uptime
            );
        })
        .await;
    }
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640

    #[test]
    fn test_request_plane_mode_from_str() {
        assert_eq!(
            "nats".parse::<RequestPlaneMode>().unwrap(),
            RequestPlaneMode::Nats
        );
        assert_eq!(
            "http".parse::<RequestPlaneMode>().unwrap(),
            RequestPlaneMode::Http
        );
        assert_eq!(
            "tcp".parse::<RequestPlaneMode>().unwrap(),
            RequestPlaneMode::Tcp
        );
        assert_eq!(
            "NATS".parse::<RequestPlaneMode>().unwrap(),
            RequestPlaneMode::Nats
        );
        assert_eq!(
            "HTTP".parse::<RequestPlaneMode>().unwrap(),
            RequestPlaneMode::Http
        );
        assert_eq!(
            "TCP".parse::<RequestPlaneMode>().unwrap(),
            RequestPlaneMode::Tcp
        );
        assert!("invalid".parse::<RequestPlaneMode>().is_err());
    }

    #[test]
    fn test_request_plane_mode_display() {
        assert_eq!(RequestPlaneMode::Nats.to_string(), "nats");
        assert_eq!(RequestPlaneMode::Http.to_string(), "http");
        assert_eq!(RequestPlaneMode::Tcp.to_string(), "tcp");
    }
641
}