[Xen-devel] [OSSTEST PATCH 09/15] sg-run-job: Break out allocate-build-host, and reorder

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

We're going to want to start the syslog server after host allocation
but before installation, so we need to split these steps up.

A side effect is that a build job becomes `running' when it starts its
host install, not when it starts the actual build.

Signed-off-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
---
v2: New patch
---
 sg-run-job | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/sg-run-job b/sg-run-job
index ea3ea41..620d89a 100755
--- a/sg-run-job
+++ b/sg-run-job
@@ -69,13 +69,16 @@ proc run-job {job} {
         eval run-ts broken  =             ts-hosts-allocate + $need_xen_hosts
     }
 
-    if {$need_build_host} { catching-otherwise broken prepare-build-host }
+    if {$need_build_host} { catching-otherwise broken allocate-build-host }
 
     if {$ok} { setstatus running                                          }
 
     if {$ok && ![catch { info args prep-job/$jobinfo(recipe) }]} \
         {      catching-otherwise fail     prep-job/$jobinfo(recipe)      }
 
+    if {$ok && $need_build_host} \
+        { catching-otherwise broken prepare-build-host }
+
     per-host-ts broken  host-install/@(*) ts-host-install-twice
 
     per-host-prep
@@ -689,9 +692,12 @@ proc run-job/build-xtf {} {
     run-ts . = ts-xtf-build
 }
 
-proc prepare-build-host {} {
+proc allocate-build-host {} {
     global jobinfo
     run-ts broken = ts-hosts-allocate + host
+}
+proc prepare-build-host {} {
+    global jobinfo
     run-ts broken host-install(*) ts-host-install-twice
     run-ts . host-build-prep ts-xen-build-prep
 }
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel