Add missing error logs for ReadFailed cases

.. some of the `ReadFailed` cases didn't log why they failed. Here we
fix that oversight.

Author: tnull

src/builder.rs

@@ -1138,6 +1138,7 @@ fn build_with_store_internal(
 			if e.kind() == std::io::ErrorKind::NotFound {
 				Arc::new(RwLock::new(NodeMetrics::default()))
 			} else {
+				log_error!(logger, "Failed to read node metrics from store: {}", e);
 				return Err(BuildError::ReadFailed);
 			}
 		},
@@ -1201,7 +1202,8 @@ fn build_with_store_internal(
 			Arc::clone(&kv_store),
 			Arc::clone(&logger),
 		)),
-		Err(_) => {
+		Err(e) => {
+			log_error!(logger, "Failed to read payment data from store: {}", e);
 			return Err(BuildError::ReadFailed);
 		},
 	};
@@ -1334,7 +1336,7 @@ fn build_with_store_internal(
 			if e.kind() == lightning::io::ErrorKind::NotFound {
 				Vec::new()
 			} else {
-				log_error!(logger, "Failed to read channel monitors: {}", e.to_string());
+				log_error!(logger, "Failed to read channel monitors from store: {}", e.to_string());
 				return Err(BuildError::ReadFailed);
 			}
 		},
@@ -1359,6 +1361,7 @@ fn build_with_store_internal(
 				if e.kind() == std::io::ErrorKind::NotFound {
 					Arc::new(Graph::new(config.network.into(), Arc::clone(&logger)))
 				} else {
+					log_error!(logger, "Failed to read network graph from store: {}", e);
 					return Err(BuildError::ReadFailed);
 				}
 			},
@@ -1379,6 +1382,7 @@ fn build_with_store_internal(
 					Arc::clone(&logger),
 				)))
 			} else {
+				log_error!(logger, "Failed to read scoring data from store: {}", e);
 				return Err(BuildError::ReadFailed);
 			}
 		},
@@ -1448,7 +1452,7 @@ fn build_with_store_internal(
 			);
 			let (_hash, channel_manager) =
 				<(BlockHash, ChannelManager)>::read(&mut reader, read_args).map_err(|e| {
-					log_error!(logger, "Failed to read channel manager from KVStore: {}", e);
+					log_error!(logger, "Failed to read channel manager from store: {}", e);
 					BuildError::ReadFailed
 				})?;
 			channel_manager
@@ -1677,6 +1681,7 @@ fn build_with_store_internal(
 					Arc::clone(&logger),
 				))
 			} else {
+				log_error!(logger, "Failed to read output sweeper data from store: {}", e);
 				return Err(BuildError::ReadFailed);
 			}
 		},
@@ -1689,6 +1694,7 @@ fn build_with_store_internal(
 			if e.kind() == std::io::ErrorKind::NotFound {
 				Arc::new(EventQueue::new(Arc::clone(&kv_store), Arc::clone(&logger)))
 			} else {
+				log_error!(logger, "Failed to read event queue from store: {}", e);
 				return Err(BuildError::ReadFailed);
 			}
 		},
@@ -1700,6 +1706,7 @@ fn build_with_store_internal(
 			if e.kind() == std::io::ErrorKind::NotFound {
 				Arc::new(PeerStore::new(Arc::clone(&kv_store), Arc::clone(&logger)))
 			} else {
+				log_error!(logger, "Failed to read peer data from store: {}", e);
 				return Err(BuildError::ReadFailed);
 			}
 		},