| Message ID | 1495713067-19757-2-git-send-email-ian.jackson@eu.citrix.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index 7494291..975a8b9 100755 --- a/ts-hosts-allocate-Executive +++ b/ts-hosts-allocate-Executive @@ -624,6 +624,7 @@ sub hid_recurse ($$) { if ($jobinfo->{recipe} =~ m/build/) { $variation_age= 0; $duration_for_cost= $duration + $duration_rightaway_adjust; + $prevfail_bonus //= 0; } my $log_variation_age = log(1+$variation_age/86400);
Builds (jobs whose recipe contains `build') are not supposed to contain host-specific tests. We already don't care about varying which host they run on, to hunt for host-specific bugs. We should also not make build failures sticky to the host they last failed on. Fix this by defaulting $prevfail_bonus to 0 in that case, before the week-long later default is applied. This means that a build job runvar would override the built-in default whether that's 7 days or 0. Reported-by: Roger Pau Monné <roger.pau@citrix.com> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com> --- ts-hosts-allocate-Executive | 1 + 1 file changed, 1 insertion(+)