gaspersic/solana
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Better logs

Browse Source
This commit is contained in:
Anatoly Yakovenko
2018-07-03 15:09:35 -07:00
committed by Greg Fitzgerald
parent 800c2dd370
commit e771d36278
3 changed files with 76 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/crdt.rs
View File

@ -479,7 +479,7 @@ impl Crdt {
let errs: Vec<_> = orders let errs: Vec<_> = orders
.par_iter() .par_iter()
.map(|v| { .map(|v| {
debug!( info!(
"{:x}: retransmit blob {} to {:x}", "{:x}: retransmit blob {} to {:x}",
me.debug_id(), me.debug_id(),
rblob.get_index().unwrap(), rblob.get_index().unwrap(),
@ -766,8 +766,11 @@ impl Crdt {
} else { } else {
assert!(window.read().unwrap()[pos].is_none()); assert!(window.read().unwrap()[pos].is_none());
info!( info!(
"failed RequestWindowIndex {} {} {}", "{:x}: failed RequestWindowIndex {:x} {} {}",
ix, pos, from.repair_addr me.debug_id(),
from.debug_id(),
ix,
pos,
); );
} }
@ -840,10 +843,10 @@ impl Crdt {
obj.write().unwrap().insert(&from); obj.write().unwrap().insert(&from);
let me = obj.read().unwrap().my_data().clone(); let me = obj.read().unwrap().my_data().clone();
trace!( trace!(
"received RequestWindowIndex {} {} myaddr {}", "{:x}:received RequestWindowIndex {:x} {} ",
me.debug_id(),
from.debug_id(),
ix, ix,
from.repair_addr,
me.repair_addr
); );
assert_ne!(from.repair_addr, me.repair_addr); assert_ne!(from.repair_addr, me.repair_addr);
Self::run_window_request(&window, &me, &from, ix, blob_recycler) Self::run_window_request(&window, &me, &from, ix, blob_recycler)

21
src/streamer.rs
View File

@ -253,6 +253,13 @@ fn recv_window(
while let Ok(mut nq) = r.try_recv() { while let Ok(mut nq) = r.try_recv() {
dq.append(&mut nq) dq.append(&mut nq)
} }
info!(
"{:x}: RECV_WINDOW {} {}: got packets {}",
debug_id,
*consumed,
*received,
dq.len(),
);
{ {
//retransmit all leader blocks //retransmit all leader blocks
let mut retransmitq = VecDeque::new(); let mut retransmitq = VecDeque::new();
@ -291,6 +298,13 @@ fn recv_window(
warn!("{:x}: no leader to retransmit from", debug_id); warn!("{:x}: no leader to retransmit from", debug_id);
} }
if !retransmitq.is_empty() { if !retransmitq.is_empty() {
info!(
"{:x}: RECV_WINDOW {} {}: retransmit {}",
debug_id,
*consumed,
*received,
retransmitq.len(),
);
retransmit.send(retransmitq)?; retransmit.send(retransmitq)?;
} }
} }
@ -398,6 +412,13 @@ fn recv_window(
print_window(locked_window, *consumed); print_window(locked_window, *consumed);
trace!("sending contq.len: {}", contq.len()); trace!("sending contq.len: {}", contq.len());
if !contq.is_empty() { if !contq.is_empty() {
info!(
"{:x}: RECV_WINDOW {} {}: forwarding contq {}",
debug_id,
*consumed,
*received,
contq.len(),
);
trace!("sending contq.len: {}", contq.len()); trace!("sending contq.len: {}", contq.len());
s.send(contq)?; s.send(contq)?;
} }

71
tests/multinode.rs
View File

@ -126,7 +126,7 @@ fn test_multi_node_validator_catchup_from_zero() {
//verify validator has the same balance //verify validator has the same balance
let mut success = 0usize; let mut success = 0usize;
for server in servers.iter() { for server in servers.iter() {
info!("0server: {:?}", server.id[0]); info!("0server: {:x}", server.debug_id());
let mut client = mk_client(server); let mut client = mk_client(server);
if let Ok(bal) = client.poll_get_balance(&bob_pubkey) { if let Ok(bal) = client.poll_get_balance(&bob_pubkey) {
info!("validator balance {}", bal); info!("validator balance {}", bal);
@ -166,7 +166,7 @@ fn test_multi_node_validator_catchup_from_zero() {
for server in servers.iter() { for server in servers.iter() {
let mut client = mk_client(server); let mut client = mk_client(server);
info!("1server: {:?}", server.id[0]); info!("1server: {:x}", server.debug_id());
for _ in 0..10 { for _ in 0..10 {
if let Ok(bal) = client.poll_get_balance(&bob_pubkey) { if let Ok(bal) = client.poll_get_balance(&bob_pubkey) {
info!("validator balance {}", bal); info!("validator balance {}", bal);
@ -294,7 +294,7 @@ fn test_boot_validator_from_file() {
#[test] #[test]
fn test_multi_node_dynamic_network() { fn test_multi_node_dynamic_network() {
logger::setup(); logger::setup();
const N: usize = 3; const N: usize = 25;
let leader = TestNode::new(); let leader = TestNode::new();
let bob_pubkey = KeyPair::new().pubkey(); let bob_pubkey = KeyPair::new().pubkey();
let exit = Arc::new(AtomicBool::new(false)); let exit = Arc::new(AtomicBool::new(false));
@ -305,7 +305,7 @@ fn test_multi_node_dynamic_network() {
true, true,
InFile::Path(ledger_path.clone()), InFile::Path(ledger_path.clone()),
None, None,
None, Some(OutFile::Path(ledger_path.clone())),
exit.clone(), exit.clone(),
); );
let threads = server.thread_hdls; let threads = server.thread_hdls;
@ -316,7 +316,7 @@ fn test_multi_node_dynamic_network() {
send_tx_and_retry_get_balance(&leader_data, &alice, &bob_pubkey, Some(1000)).unwrap(); send_tx_and_retry_get_balance(&leader_data, &alice, &bob_pubkey, Some(1000)).unwrap();
assert_eq!(leader_balance, 1000); assert_eq!(leader_balance, 1000);
let mut vals: Vec<(ReplicatedData, Arc<AtomicBool>, FullNode)> = (0..N) let mut validators: Vec<(ReplicatedData, Arc<AtomicBool>, FullNode)> = (0..N)
.into_iter() .into_iter()
.map(|_| { .map(|_| {
let exit = Arc::new(AtomicBool::new(false)); let exit = Arc::new(AtomicBool::new(false));
@ -327,7 +327,7 @@ fn test_multi_node_dynamic_network() {
false, false,
InFile::Path(ledger_path.clone()), InFile::Path(ledger_path.clone()),
Some(leader_data.gossip_addr), Some(leader_data.gossip_addr),
None, Some(OutFile::Path(ledger_path.clone())),
exit.clone(), exit.clone(),
); );
(rd, exit, val) (rd, exit, val)
@ -341,30 +341,50 @@ fn test_multi_node_dynamic_network() {
.unwrap(); .unwrap();
assert_eq!(leader_balance, expected); assert_eq!(leader_balance, expected);
//verify all validators have the same balance //verify all validators have the same balance
let mut success = 0usize; for i in 0..10 {
for server in vals.iter() { let mut success = 0usize;
let mut client = mk_client(&server.0); let mut distance = 0i64;
let getbal = retry_get_balance(&mut client, &bob_pubkey, Some(expected)); for server in validators.iter() {
info!("{:x} {} get_balance: {:?}", server.0.debug_id(), i, getbal); let mut client = mk_client(&server.0);
if let Some(bal) = getbal { let getbal = retry_get_balance(&mut client, &bob_pubkey, Some(expected));
if bal == leader_balance { trace!(
success += 1; "{:x} {} get_balance: {:?} expected: {}",
server.0.debug_id(),
i,
getbal,
expected
);
let bal = getbal.unwrap_or(0);
distance += (expected - bal) / 500;
if let Some(bal) = getbal {
if bal == leader_balance {
success += 1;
}
} }
} }
if success == validators.len() {
break;
}
sleep(Duration::from_millis(i * 100));
info!(
"SUCCESS {} out of {} distance: {}",
success,
validators.len(),
distance
);
} }
info!("SUCCESS {} out of {}", success, vals.len()); // this should be almost true, or at least validators.len() - 1 while the other node catches up
// this should be almost true, or at least vals.len() - 1 while the other node catches up //assert!(success == validators.len());
//assert!(success == vals.len());
//kill a validator //kill a validator
vals[i].1.store(true, Ordering::Relaxed); validators[i].1.store(true, Ordering::Relaxed);
let mut ts = vec![]; let mut ts = vec![];
ts.append(&mut vals[i].2.thread_hdls); ts.append(&mut validators[i].2.thread_hdls);
for t in ts.into_iter() { for t in ts.into_iter() {
t.join().unwrap(); t.join().unwrap();
} }
info!("{:x} KILLED", vals[i].0.debug_id()); info!("{:x} KILLED", validators[i].0.debug_id());
//add a new one //add a new one
vals[i] = { validators[i] = {
let exit = Arc::new(AtomicBool::new(false)); let exit = Arc::new(AtomicBool::new(false));
let validator = TestNode::new(); let validator = TestNode::new();
let rd = validator.data.clone(); let rd = validator.data.clone();
@ -373,14 +393,14 @@ fn test_multi_node_dynamic_network() {
false, false,
InFile::Path(ledger_path.clone()), InFile::Path(ledger_path.clone()),
Some(leader_data.gossip_addr), Some(leader_data.gossip_addr),
None, Some(OutFile::Path(ledger_path.clone())),
exit.clone(), exit.clone(),
); );
info!("{:x} ADDED", rd.debug_id()); info!("{:x} ADDED", rd.debug_id());
(rd, exit, val) (rd, exit, val)
}; };
} }
for (_, exit, val) in vals.into_iter() { for (_, exit, val) in validators.into_iter() {
exit.store(true, Ordering::Relaxed); exit.store(true, Ordering::Relaxed);
for t in val.thread_hdls { for t in val.thread_hdls {
t.join().unwrap(); t.join().unwrap();
@ -415,9 +435,10 @@ fn retry_get_balance(
bob_pubkey: &PublicKey, bob_pubkey: &PublicKey,
expected: Option<i64>, expected: Option<i64>,
) -> Option<i64> { ) -> Option<i64> {
for _ in 0..10 { const LAST: usize = 9;
for run in 0..(LAST + 1) {
let out = client.poll_get_balance(bob_pubkey); let out = client.poll_get_balance(bob_pubkey);
if expected.is_none() { if expected.is_none() || run == LAST {
return out.ok().clone(); return out.ok().clone();
} }
if let (Some(e), Ok(o)) = (expected, out) { if let (Some(e), Ok(o)) = (expected, out) {