Skip to content

Commit 56ab66c

Browse files
fix(node): warn when no foreign chain is configured
A node is expected to configure the chains the code supports, so an empty section is a misconfiguration rather than a quiet path. The startup probe is the only place that observes it.
1 parent 5c5cd1d commit 56ab66c

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

crates/node/src/foreign_chain_probe.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ use crate::metrics;
1717
/// serving, because a boot time blip should not take a chain out of signing.
1818
pub async fn run_startup_probe(foreign_chains: ForeignChainsConfig) {
1919
if foreign_chains.is_empty() {
20-
debug!("no foreign chain is configured, skipping the RPC provider probe");
20+
warn!("no foreign chain is configured: this node cannot verify foreign-chain transactions");
2121
return;
2222
}
2323

@@ -80,7 +80,6 @@ fn log_report(report: &ProbeReport) {
8080
info!("foreign-chain RPC provider probe complete: {healthy}/{probed} providers healthy");
8181
}
8282

83-
/// Publish healthy/Configured counts per chain
8483
fn publish_metrics(report: &ProbeReport) {
8584
let probed_chains: BTreeSet<dtos::ForeignChain> = report
8685
.rows()

crates/node/src/run.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,7 @@ pub async fn run_mpc_node(config: StartConfig) -> anyhow::Result<()> {
209209

210210
let _web_server_join_handle = root_runtime.spawn(web_server);
211211

212-
// Detached and diagnostic only: reports which network each configured provider serves.
212+
// Check number of healthy foreign chain RPC providers, detached and diagnostic only
213213
root_runtime.spawn(crate::foreign_chain_probe::run_startup_probe(
214214
node_config.foreign_chains.clone(),
215215
));

0 commit comments

Comments
 (0)