validator-discovery: some extra logs (#3157)

* remove duplicate logging

* validator-discovery: add extra logging
This commit is contained in:
Andronik Ordian
2021-06-03 16:17:24 +02:00
committed by GitHub
parent 53de6871e0
commit bd66121324
3 changed files with 14 additions and 8 deletions
@@ -89,6 +89,7 @@ impl<N: Network, AD: AuthorityDiscovery> Service<N, AD> {
// collect multiaddress of validators
let mut failed_to_resolve: usize = 0;
let mut newly_requested = HashSet::new();
let requested = validator_ids.len();
for authority in validator_ids.into_iter() {
let result = authority_discovery_service.get_addresses_by_authority_id(authority.clone()).await;
if let Some(addresses) = result {
@@ -101,15 +102,24 @@ impl<N: Network, AD: AuthorityDiscovery> Service<N, AD> {
let state = &mut self.state[peer_set];
// clean up revoked requests
let multiaddr_to_remove = state.previously_requested
let multiaddr_to_remove: HashSet<_> = state.previously_requested
.difference(&newly_requested)
.cloned()
.collect();
let multiaddr_to_add = newly_requested.difference(&state.previously_requested)
let multiaddr_to_add: HashSet<_> = newly_requested.difference(&state.previously_requested)
.cloned()
.collect();
state.previously_requested = newly_requested;
tracing::debug!(
target: LOG_TARGET,
?peer_set,
?requested,
added = multiaddr_to_add.len(),
removed = multiaddr_to_remove.len(),
?failed_to_resolve,
"New ConnectToValidators request",
);
// ask the network to connect to these nodes and not disconnect
// from them until removed from the set
if let Err(e) = network_service.add_to_peers_set(