subd: Remove overly verbose log when receiving a message

These tend to spam the logs, so I removed them :-)

Signed-off-by: Christian Decker <decker.christian@gmail.com>
This commit is contained in:
Christian Decker 2019-05-22 21:36:11 +02:00
parent ced2ff8aa7
commit 8feb05aef4
3 changed files with 8 additions and 8 deletions

View File

@ -469,8 +469,6 @@ static struct io_plan *sd_msg_read(struct io_conn *conn, struct subd *sd)
} }
} }
log_debug(sd->log, "UPDATE %s", sd->msgname(type));
/* Might free sd (if returns negative); save/restore sd->conn */ /* Might free sd (if returns negative); save/restore sd->conn */
sd->conn = NULL; sd->conn = NULL;
tal_add_destructor2(sd, mark_freed, &freed); tal_add_destructor2(sd, mark_freed, &freed);

View File

@ -308,7 +308,7 @@ def test_penalty_inhtlc(node_factory, bitcoind, executor):
l2.daemon.wait_for_log('=WIRE_COMMITMENT_SIGNED-nocommit') l2.daemon.wait_for_log('=WIRE_COMMITMENT_SIGNED-nocommit')
# Make sure l1 got l2's commitment to the HTLC, and sent to master. # Make sure l1 got l2's commitment to the HTLC, and sent to master.
l1.daemon.wait_for_log('UPDATE WIRE_CHANNEL_GOT_COMMITSIG') l1.daemon.wait_for_log('got commitsig')
# Take our snapshot. # Take our snapshot.
tx = l1.rpc.dev_sign_last_tx(l2.info['id'])['tx'] tx = l1.rpc.dev_sign_last_tx(l2.info['id'])['tx']

View File

@ -364,8 +364,10 @@ def test_reconnect_openingd(node_factory):
l1.rpc.fundchannel(l2.info['id'], 20000) l1.rpc.fundchannel(l2.info['id'], 20000)
l1.daemon.wait_for_log('sendrawtx exit 0') l1.daemon.wait_for_log('sendrawtx exit 0')
# Just to be sure, second openingd hand over to channeld. l1.bitcoin.generate_block(3)
l2.daemon.wait_for_log('lightning_openingd.*UPDATE WIRE_OPENING_FUNDEE')
# Just to be sure, second openingd hand over to channeld. This log line is about channeld being started
l2.daemon.wait_for_log(r'lightning_channeld-[0-9a-f]{66} chan #[0-9]: pid [0-9]+, msgfd [0-9]+')
@unittest.skipIf(not DEVELOPER, "needs DEVELOPER=1") @unittest.skipIf(not DEVELOPER, "needs DEVELOPER=1")
@ -1124,8 +1126,8 @@ def test_peerinfo(node_factory, bitcoind):
wait_for(lambda: not only_one(l2.rpc.listpeers(l1.info['id'])['peers'])['connected']) wait_for(lambda: not only_one(l2.rpc.listpeers(l1.info['id'])['peers'])['connected'])
bitcoind.generate_block(100) bitcoind.generate_block(100)
l1.daemon.wait_for_log('WIRE_ONCHAIN_ALL_IRREVOCABLY_RESOLVED') l1.daemon.wait_for_log('onchaind complete, forgetting peer')
l2.daemon.wait_for_log('WIRE_ONCHAIN_ALL_IRREVOCABLY_RESOLVED') l2.daemon.wait_for_log('onchaind complete, forgetting peer')
# The only channel was closed, everybody should have forgotten the nodes # The only channel was closed, everybody should have forgotten the nodes
assert l1.rpc.listnodes()['nodes'] == [] assert l1.rpc.listnodes()['nodes'] == []
@ -1418,7 +1420,7 @@ def test_dataloss_protection(node_factory, bitcoind):
l2.daemon.wait_for_log("ERROR: Unknown commitment #[0-9], recovering our funds!") l2.daemon.wait_for_log("ERROR: Unknown commitment #[0-9], recovering our funds!")
bitcoind.generate_block(100) bitcoind.generate_block(100)
l2.daemon.wait_for_log('WIRE_ONCHAIN_ALL_IRREVOCABLY_RESOLVED') l2.daemon.wait_for_log('onchaind complete, forgetting peer')
# l2 should have it in wallet. # l2 should have it in wallet.
assert (closetxid, "confirmed") in set([(o['txid'], o['status']) for o in l2.rpc.listfunds()['outputs']]) assert (closetxid, "confirmed") in set([(o['txid'], o['status']) for o in l2.rpc.listfunds()['outputs']])