Eugene

I won't have time to work on this unfortunately

I think this is because honggfuzz ASAN doesn't have an instrumented strncmp in some scenarios. I am not sure *why* this occurs, but I am also facing this issue. I...

rebased @wpaulino

I think it would be best if this wasn't merged in until #3219 was. Then I can update the code to properly check for node announcements.

Rebased + added correct test checks since #3219 was merged in

@halseth #3219 makes sure each node gets the node announcement. There are two problems present in the current `testGraphTopologyNotifications` itest. 1. It only waits for 2 updates OR 2 node...

@wpaulino needed because Alice and Bob disconnect with each other here https://github.com/lightningnetwork/lnd/blob/2711b7c7f622d879c5991968c4be207c55c5e13a/lntest/itest/lnd_test.go#L9021-L9023

This isn't necessary since the problem is with btcd which can instead be fixed

This should be fixed by https://github.com/btcsuite/btcd/pull/1876 and upgrading to point to the newer btcd version upon merge. Also fixes cases where users advertise a hostname that only has an ipv6...

Would you mind sharing logs for this? Either your node or the watchtower has data loss or a bug