skip failing api calls (#7059)

oxidecomputer · Nov 14, 2024 · 97d4c2c · 97d4c2c
1 parent 2e7724a
commit 97d4c2c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/dev-tools/ls-apis/tests/api_dependencies.out b/dev-tools/ls-apis/tests/api_dependencies.out
@@ -8,6 +8,9 @@ Clickhouse Cluster Admin for Keepers (client: clickhouse-admin-keeper-client)
 Clickhouse Cluster Admin for Servers (client: clickhouse-admin-server-client)
     consumed by: omicron-nexus (omicron/nexus) via 3 paths
 
+Clickhouse Single-Node Cluster Admin (client: clickhouse-admin-single-client)
+    consumed by: omicron-nexus (omicron/nexus) via 2 paths
+
 CockroachDB Cluster Admin (client: cockroach-admin-client)
     consumed by: omicron-nexus (omicron/nexus) via 2 paths
 

diff --git a/nexus/reconfigurator/execution/src/clickhouse.rs b/nexus/reconfigurator/execution/src/clickhouse.rs
@@ -181,11 +181,13 @@ pub(crate) async fn deploy_single_node(
         let admin_url = format!("http://{admin_addr}");
         let log = opctx.log.new(slog::o!("admin_url" => admin_url.clone()));
         let client = ClickhouseSingleClient::new(&admin_url, log.clone());
-        client.init_db().await.map(|_| ()).map_err(|e| {
-            anyhow!(
-                "failed to initialize single-node clickhouse database: {e}",
-            )
-        })
+        if let Err(e) = client.init_db().await {
+            warn!(
+                log,
+                "failed to initialize single-node clickhouse database: {e}"
+            );
+        }
+        Ok(())
     } else {
         Ok(())
     }