diff mbox series

[OSSTEST,15/16] host reuse fixes: Do not break host-reuse if no host allocated

Message ID 20201022164506.1552-16-iwj@xenproject.org (mailing list archive)
State New, archived
Headers show
Series Bugfixes | expand

Commit Message

Ian Jackson Oct. 22, 2020, 4:45 p.m. UTC 
If host allocation failed, or our dependency jobs failed, then we
won't have allocated a host.  The host runvar will not be set.
In this case, we want to do nothing.

But we forgot to pass $noneok to selecthost.

Signed-off-by: Ian Jackson <iwj@xenproject.org>
---
 ts-host-reuse | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/ts-host-reuse b/ts-host-reuse
index e2498bb6..b885a3e6 100755
--- a/ts-host-reuse
+++ b/ts-host-reuse
@@ -165,7 +165,7 @@  sub act_start_test () {
 
 sub act_final () {
     if (!@ARGV) {
-	$ho = selecthost($whhost);
+	$ho = selecthost($whhost, 1);
 	return unless $ho;
 	host_update_lifecycle_info($ho, 'final');
     } elsif ("@ARGV" eq "--post-test-ok") {