diff mbox series

[OSSTEST,20/82] ts-hosts-allocate-Executive: Fix handling of failed preps for same sharing

Message ID 20201007180024.7932-21-iwj@xenproject.org (mailing list archive)
State New, archived
Headers show
Series Reuse test hosts | expand

Commit Message

Ian Jackson Oct. 7, 2020, 5:59 p.m. UTC 
From: Ian Jackson <ian.jackson@eu.citrix.com>

This code was previously unreachable.  It ought to be executed when
all the shares are allocatable or prep: in that case, we can unshare
and re-share the host.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 ts-hosts-allocate-Executive | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)
diff mbox series

Patch

diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive
index 78b94c6d..2c18a739 100755
--- a/ts-hosts-allocate-Executive
+++ b/ts-hosts-allocate-Executive
@@ -1044,12 +1044,11 @@  sub actual_allocation ($) {
 		# now-dead task.  If so that share will become allocatable
 		# (and therefore not be counted in `ntasks') in due course.
 		return undef;
-
-                # someone was preparing it but they aren't any more
-                push @allocwarnings,
-                    "previous preparation apparently abandoned";
-                $allocatable= 1;
-            }
+	    }
+	    # someone was preparing it but they aren't any more
+	    push @allocwarnings,
+		"previous preparation apparently abandoned";
+	    $allocatable= 1;
         }
     }