From cd96a3c9ee3ed6f552df75d57e4cfa0556f64af2 Mon Sep 17 00:00:00 2001
From: Remi Dettai <rdettai@gmail.com>
Date: Thu, 7 Mar 2024 18:54:07 +0100
Subject: [PATCH] Remove seed resolution validation (#4688)

* Remove seed resolution validation

* Add extra warning when failing to resolve all seeds
---
 quickwit/quickwit-config/src/node_config/mod.rs       |  5 +----
 quickwit/quickwit-config/src/node_config/serialize.rs | 10 ----------
 2 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/quickwit/quickwit-config/src/node_config/mod.rs b/quickwit/quickwit-config/src/node_config/mod.rs
index 5ef9165504f..839d760ade1 100644
--- a/quickwit/quickwit-config/src/node_config/mod.rs
+++ b/quickwit/quickwit-config/src/node_config/mod.rs
@@ -459,10 +459,7 @@ impl NodeConfig {
             peer_seed_addrs.push(peer_seed_addr.to_string())
         }
         if !self.peer_seeds.is_empty() && peer_seed_addrs.is_empty() {
-            bail!(
-                "failed to resolve any of the peer seed addresses: `{}`",
-                self.peer_seeds.join(", ")
-            )
+            warn!("failed to resolve all the peer seed addresses")
         }
         Ok(peer_seed_addrs)
     }
diff --git a/quickwit/quickwit-config/src/node_config/serialize.rs b/quickwit/quickwit-config/src/node_config/serialize.rs
index 8cee5930502..175309d0e76 100644
--- a/quickwit/quickwit-config/src/node_config/serialize.rs
+++ b/quickwit/quickwit-config/src/node_config/serialize.rs
@@ -863,16 +863,6 @@ mod tests {
             .unwrap();
             assert!(node_config.peer_seed_addrs().await.unwrap().is_empty());
         }
-        {
-            let node_config = NodeConfigBuilder {
-                peer_seeds: ConfigValue::for_test(List(vec!["unresolvable-host".to_string()])),
-                ..Default::default()
-            }
-            .build_and_validate(&HashMap::new())
-            .await
-            .unwrap();
-            assert!(node_config.peer_seed_addrs().await.is_err());
-        }
         {
             let node_config = NodeConfigBuilder {
                 rest_config_builder: RestConfigBuilder {