[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [OSSTEST PATCH 1/7] ts-hosts-allocate-Executive: Fix broken call to $duration_estimator
The debug subref is passed to the constructor (and indeed we do that). The final argument to the actual estimator is $uptoincl_testid (but we didn't say $will_uptoincl_testid, so it is ignored). The code was wrong, but with no effect. So no functional change. Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx> --- ts-hosts-allocate-Executive | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index 3bc38102..e5a6fbfe 100755 --- a/ts-hosts-allocate-Executive +++ b/ts-hosts-allocate-Executive @@ -516,10 +516,7 @@ sub find_recent_duration ($$) { ($candrow->{Duration}, $candrow->{MostRecentStarted}, $candrow->{MostRecentStatus}) = - $duration_estimator->($job, $hid->{Ident}, $candrow->{resname}, - sub { -# print DEBUG "$dbg DUR-EST @_\n"; - }); + $duration_estimator->($job, $hid->{Ident}, $candrow->{resname}); } -- 2.11.0
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |