|
[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 |