A stab at #2487

informalsystems · Aug 4, 2022 · 7ad5084 · 7ad5084
1 parent 6fc3a28
commit 7ad5084
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/relayer/src/link/relay_path.rs b/relayer/src/link/relay_path.rs
@@ -619,7 +619,7 @@ impl<ChainA: ChainHandle, ChainB: ChainHandle> RelayPath<ChainA, ChainB> {
             match self.send_from_operational_data::<S>(&odata) {
                 Ok(reply) => {
                     // Done with this op. data
-                    info!("success");
+                    info!("done");
                     telemetry!({
                         let (chain, counterparty, channel_id, port_id) =
                             self.target_info(odata.target);

diff --git a/relayer/src/link/relay_sender.rs b/relayer/src/link/relay_sender.rs
@@ -93,7 +93,11 @@ impl Submit for AsyncSender {
             .send_messages_and_wait_check_tx(msgs)
             .map_err(LinkError::relayer)?;
         let reply = AsyncReply { responses: a };
-        info!("[Async~>{}] {}\n", target.id(), reply);
+
+        // Note: There may be errors in the reply, for example:
+        // `Response { code: Err(11), data: Data([]), log: Log("Too much gas wanted: 35000000, maximum is 25000000: out of gas")`
+        // The runtime deliberately did not catch or retry on such errors.
+        info!(target_chain = %target.id(), "{}", reply);
 
         Ok(reply)
     }