diff mbox

[OSSTEST] make-flight: XTF: honour $bfi (ie build flight)

Message ID 1475779305-31777-1-git-send-email-ian.jackson@eu.citrix.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ian Jackson Oct. 6, 2016, 6:41 p.m. UTC
If make-flight is run with a $buildflight argument, it does not create
any build jobs.  The test jobs are supposed to refer to the build jobs.

This was not done correctly for the XTF tests.  Add the missing ${bfi}.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
---
 make-flight | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wei Liu Oct. 6, 2016, 8:14 p.m. UTC | #1
On Thu, Oct 06, 2016 at 07:41:45PM +0100, Ian Jackson wrote:
> If make-flight is run with a $buildflight argument, it does not create
> any build jobs.  The test jobs are supposed to refer to the build jobs.
> 
> This was not done correctly for the XTF tests.  Add the missing ${bfi}.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>

Reviewed-by: Wei Liu <wei.liu2@citrix.com>

> ---
>  make-flight | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/make-flight b/make-flight
> index 9b47999..3a95cfa 100755
> --- a/make-flight
> +++ b/make-flight
> @@ -454,7 +454,7 @@ do_xtf_tests () {
>    for i in `seq 1 5`; do
>        job_create_test test-xtf-$xenarch-$dom0arch-$i                 \
>         test-xtf xl $xenarch $dom0arch                                \
> -            xtfbuildjob=build-$xenarch-xtf                           \
> +            xtfbuildjob=${bfi}build-$xenarch-xtf                           \
>              xen_boot_append='hvm_fep=1'                              \
>              all_hostflags=$most_hostflags,diverse-xtf-x86
>    done
> -- 
> 2.1.4
>
diff mbox

Patch

diff --git a/make-flight b/make-flight
index 9b47999..3a95cfa 100755
--- a/make-flight
+++ b/make-flight
@@ -454,7 +454,7 @@  do_xtf_tests () {
   for i in `seq 1 5`; do
       job_create_test test-xtf-$xenarch-$dom0arch-$i                 \
        test-xtf xl $xenarch $dom0arch                                \
-            xtfbuildjob=build-$xenarch-xtf                           \
+            xtfbuildjob=${bfi}build-$xenarch-xtf                           \
             xen_boot_append='hvm_fep=1'                              \
             all_hostflags=$most_hostflags,diverse-xtf-x86
   done