db-snapshot: propagate any node snapshot error through RPC call

In particular, it means that "garage meta snapshot --all" will get an exit
code of 1 if any node fails to snapshot.

This makes sure that any external tool trying to snapshot nodes (e.g. from
cron) will be aware of the failure.

Fix #920

src/garage/admin/mod.rs

@@ -484,7 +484,7 @@ impl AdminRpcHandler {
 							AdminRpc::MetaOperation(MetaOperation::Snapshot { all: false }),
 							PRIO_NORMAL,
 						)
-						.await
+						.await?
 				}))
 				.await;
 
@@ -497,8 +497,12 @@ impl AdminRpcHandler {
 					ret.push(format!("{:?}\t{}", to, res_str));
 				}
 
+				if resps.iter().any(Result::is_err) {
+					Err(GarageError::Message(format_table_to_string(ret)).into())
+				} else {
 					Ok(AdminRpc::Ok(format_table_to_string(ret)))
 				}
+			}
 			MetaOperation::Snapshot { all: false } => {
 				garage_model::snapshot::async_snapshot_metadata(&self.garage).await?;
 				Ok(AdminRpc::Ok("Snapshot has been saved.".into()))