From 698d8d5600fe6d68310ed44491ec63b85adbc008 Mon Sep 17 00:00:00 2001
From: Svyatoslav Nikolsky <svyatonik@gmail.com>
Date: Tue, 24 Nov 2020 19:27:23 +0300
Subject: [PATCH] added more relay and message-lane pallet logs (#527)

---
 bridges/modules/message-lane/src/instant_payments.rs | 12 +++++++++---
 bridges/relays/substrate/src/headers_pipeline.rs     |  7 +++++++
 .../substrate/src/millau_messages_to_rialto.rs       |  6 ++++++
 .../substrate/src/rialto_messages_to_millau.rs       |  6 ++++++
 4 files changed, 28 insertions(+), 3 deletions(-)

diff --git a/bridges/modules/message-lane/src/instant_payments.rs b/bridges/modules/message-lane/src/instant_payments.rs
index 3c11d263ce..7db2429e07 100644
--- a/bridges/modules/message-lane/src/instant_payments.rs
+++ b/bridges/modules/message-lane/src/instant_payments.rs
@@ -56,14 +56,20 @@ where
 		);
 
 		// we can't actually do anything here, because rewards are paid as a part of unrelated transaction
-		if let Err(error) = pay_result {
-			frame_support::debug::trace!(
+		match pay_result {
+			Ok(_) => frame_support::debug::trace!(
+				target: "runtime",
+				"Rewarded relayer {:?} with {:?}",
+				relayer,
+				reward,
+			),
+			Err(error) => frame_support::debug::trace!(
 				target: "runtime",
 				"Failed to pay relayer {:?} reward {:?}: {:?}",
 				relayer,
 				reward,
 				error,
-			);
+			),
 		}
 	}
 }
diff --git a/bridges/relays/substrate/src/headers_pipeline.rs b/bridges/relays/substrate/src/headers_pipeline.rs
index a60f8fb0ad..8ad6fc50b9 100644
--- a/bridges/relays/substrate/src/headers_pipeline.rs
+++ b/bridges/relays/substrate/src/headers_pipeline.rs
@@ -159,6 +159,13 @@ pub async fn run<SourceChain, TargetChain, P>(
 		source_justifications,
 	);
 
+	log::info!(
+		target: "bridge",
+		"Starting {} -> {} headers relay",
+		SourceChain::NAME,
+		TargetChain::NAME,
+	);
+
 	headers_relay::sync_loop::run(
 		HeadersSource::new(source_client),
 		SourceChain::AVERAGE_BLOCK_INTERVAL,
diff --git a/bridges/relays/substrate/src/millau_messages_to_rialto.rs b/bridges/relays/substrate/src/millau_messages_to_rialto.rs
index 132d43813b..fe0e19fa03 100644
--- a/bridges/relays/substrate/src/millau_messages_to_rialto.rs
+++ b/bridges/relays/substrate/src/millau_messages_to_rialto.rs
@@ -112,6 +112,12 @@ pub fn run(
 		relayer_id_at_source: relayer_id_at_millau,
 	};
 
+	log::info!(
+		target: "bridge",
+		"Starting Millau -> Rialto messages relay. Millau relayer account id: {:?}",
+		lane.relayer_id_at_source,
+	);
+
 	messages_relay::message_lane_loop::run(
 		messages_relay::message_lane_loop::Params {
 			lane: lane_id,
diff --git a/bridges/relays/substrate/src/rialto_messages_to_millau.rs b/bridges/relays/substrate/src/rialto_messages_to_millau.rs
index f772973728..1fc724921d 100644
--- a/bridges/relays/substrate/src/rialto_messages_to_millau.rs
+++ b/bridges/relays/substrate/src/rialto_messages_to_millau.rs
@@ -112,6 +112,12 @@ pub fn run(
 		relayer_id_at_source: relayer_id_at_rialto,
 	};
 
+	log::info!(
+		target: "bridge",
+		"Starting Rialto -> Millau messages relay. Rialto relayer account id: {:?}",
+		lane.relayer_id_at_source,
+	);
+
 	messages_relay::message_lane_loop::run(
 		messages_relay::message_lane_loop::Params {
 			lane: lane_id,