[Xen-devel] [OSSTEST PATCH 05/10] sg-run-job: testid generation: Move spawn-step-begin

[Ian Jackson <ian.jackson@xxxxxxxxxxxxx>]

Run spawn-step-begin until after the loop over $args, and after the
computation of the basic deftestid.

No functional change: nothing in that loop looks at stepno.

Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
---
 sg-run-job |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sg-run-job b/sg-run-job
index 2cf810a..68d6c65 100755
--- a/sg-run-job
+++ b/sg-run-job
@@ -115,8 +115,6 @@ proc spawn-ts {iffail testid ts args} {
 
     if {![string compare . $iffail]} { set iffail fail }
 
-    jobdb::spawn-step-begin $flight $jobinfo(job) $ts stepno
-
     set real_args {}
     set adding 1
     set host_testid_suffix {}
@@ -132,6 +130,8 @@ proc spawn-ts {iffail testid ts args} {
     regsub {^ts-} $ts {} deftestid
     append deftestid /@
 
+    jobdb::spawn-step-begin $flight $jobinfo(job) $ts stepno
+
     if {[string match =* $testid]} {
         set testid "$deftestid[string range $testid 1 end]"
     } elseif {![string compare $testid *]} {
-- 
1.7.10.4


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