Commit c04c9828 authored by Leigh B Stoller's avatar Leigh B Stoller

Do not run the precheck on empty testbed, if the reason for failure is

failure to allocate nodes; obviously the top mapped.
parent 744143ef
...@@ -400,7 +400,13 @@ sub AssignLoop() ...@@ -400,7 +400,13 @@ sub AssignLoop()
# #
# When precheck is off, we do a precheck run if the first try fails # When precheck is off, we do a precheck run if the first try fails
# to find a solution. This avoids looping on an unsolvable topology. # to find a solution. This avoids looping on an unsolvable topology.
# But, if the reason we are here is cause we could not allocate nodes,
# then we found a solution, and so trying on an empty testbed is
# pointless; it will obviously find a solution again.
# #
if (!$precheck && !$tried_precheck && ($retval == 2 || $retval == 3)) {
$tried_precheck = 1;
}
if (!$precheck && !$tried_precheck) { if (!$precheck && !$tried_precheck) {
chat("Trying assign on an empty testbed to verify mapability.\n"); chat("Trying assign on an empty testbed to verify mapability.\n");
my $save_impotent = $impotent; my $save_impotent = $impotent;
......
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