Commit 62c1618d authored by David Anderson's avatar David Anderson
Browse files

Last-minute tweaks to error ranges (wider latency tolerance due to occasional...

Last-minute tweaks to error ranges (wider latency tolerance due to occasional errors when running at low bandwidth, plus wider Pathrate tolerance to avoid false positives while waiting for tuning data).
Added more info when aborting run_linktest.pl
Added more time to prevent a race condition between collector startup and test during the loss and bandwidth tests.
parent 9200ed76
......@@ -33,8 +33,8 @@ use constant LIMIT_BW_LOSS => 0;
# A hack to make sure that we don't get bogged down in being too
# accurate! Make sure the
# error is a certain significance before we start reporting it.
use constant INSIGNIFICANT_LAT_ERROR => 0.25; # ms
use constant INSIGNIFICANT_BW_ERROR => 0.25; # mb
use constant INSIGNIFICANT_LAT_ERROR => 0.40; # ms (quite wide)
use constant INSIGNIFICANT_BW_ERROR => 1.00; # mb (quite wide)
# latency must be corrected for xmit delay under this speed.
use constant LAT_LOW_BW => 10000000;
......@@ -802,6 +802,9 @@ sub init {
if(&dotest(TEST_BW)){
&my_system_initonly(PATH_PATHRATE_SND,"-i","-q");
}
if(&dotest(TEST_LOSS) || &dotest(TEST_BW)) {
sleep(1); # give collectors time to start up (potential race)
}
# synserver host reads the ns file early to get the node count.
# not using ready count, in practice sometimes didn't get a ready
......
......@@ -28,8 +28,8 @@ if (! $EUID) {
# signal handler in case the process is killed.
$SIG{INT} = sub {
print "Aborted. Linktest continues on nodes.\n";
exit(1);
print "Aborted. Linktest continues on nodes.\nErrors so far:\n";
exit &analyze;
};
my $server;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment