xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [OSSTEST PATCH] make-flight: XTF: honour $bfi (ie build flight)
@ 2016-10-06 18:41 Ian Jackson
  2016-10-06 20:14 ` Wei Liu
  0 siblings, 1 reply; 2+ messages in thread
From: Ian Jackson @ 2016-10-06 18:41 UTC (permalink / raw)
  To: xen-devel; +Cc: Wei Liu, Ian Jackson

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(-)

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


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

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [OSSTEST PATCH] make-flight: XTF: honour $bfi (ie build flight)
  2016-10-06 18:41 [OSSTEST PATCH] make-flight: XTF: honour $bfi (ie build flight) Ian Jackson
@ 2016-10-06 20:14 ` Wei Liu
  0 siblings, 0 replies; 2+ messages in thread
From: Wei Liu @ 2016-10-06 20:14 UTC (permalink / raw)
  To: Ian Jackson; +Cc: xen-devel, Wei Liu

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
> 

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-10-06 20:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-06 18:41 [OSSTEST PATCH] make-flight: XTF: honour $bfi (ie build flight) Ian Jackson
2016-10-06 20:14 ` Wei Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).