[PATCH OSSTEST v1] make-flight: Only test ovmf guests on ovmf flight

[PATCH OSSTEST v1] make-flight: Only test ovmf guests on ovmf flight

[Ian Campbell]

Specifically drop all of the following test jobs from the ovmf flight:

-test-amd64-amd64-xl-qemuu-debianhvm-amd64
-test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm
-test-amd64-amd64-xl-qemuu-win7-amd64
-test-amd64-amd64-xl-qemuu-winxpsp3
-test-amd64-i386-qemuu-rhel6hvm-amd
-test-amd64-i386-qemuu-rhel6hvm-intel
-test-amd64-i386-xl-qemuu-debianhvm-amd64
-test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm
-test-amd64-i386-xl-qemuu-win7-amd64
-test-amd64-i386-xl-qemuu-winxpsp3
-test-amd64-i386-xl-qemuu-winxpsp3-vcpus1

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
 make-flight | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/make-flight b/make-flight
index c763ce9..3b31939 100755
--- a/make-flight
+++ b/make-flight
@@ -92,6 +92,10 @@ job_create_test_filter_callback () {
           *-qemuu-*) ;;
           *) return 1;;
       esac
+      case " $* " in
+        *" bios=ovmf "*) ;;
+        *) return 1;;
+      esac
       ;;
     *)
       case "$job" in
-- 
2.1.4

Re: [PATCH OSSTEST v1] make-flight: Only test ovmf guests on ovmf flight

[Ian Jackson]

Ian Campbell writes ("[PATCH OSSTEST v1] make-flight: Only test ovmf guests on ovmf flight"):
> Specifically drop all of the following test jobs from the ovmf flight:
> 
> -test-amd64-amd64-xl-qemuu-debianhvm-amd64
> -test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm
> -test-amd64-amd64-xl-qemuu-win7-amd64
> -test-amd64-amd64-xl-qemuu-winxpsp3
> -test-amd64-i386-qemuu-rhel6hvm-amd
> -test-amd64-i386-qemuu-rhel6hvm-intel
> -test-amd64-i386-xl-qemuu-debianhvm-amd64
> -test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm
> -test-amd64-i386-xl-qemuu-win7-amd64
> -test-amd64-i386-xl-qemuu-winxpsp3
> -test-amd64-i386-xl-qemuu-winxpsp3-vcpus1

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

(I note that the patch context is rather vague so when rebasing this
care should be taken that it doesn't misapply.)

Ian.

Re: [PATCH OSSTEST v1] make-flight: Only test ovmf guests on ovmf flight

[Wei Liu]

On Tue, Jul 07, 2015 at 05:26:45PM +0100, Ian Campbell wrote:
> Specifically drop all of the following test jobs from the ovmf flight:
> 
> -test-amd64-amd64-xl-qemuu-debianhvm-amd64
> -test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm
> -test-amd64-amd64-xl-qemuu-win7-amd64
> -test-amd64-amd64-xl-qemuu-winxpsp3
> -test-amd64-i386-qemuu-rhel6hvm-amd
> -test-amd64-i386-qemuu-rhel6hvm-intel
> -test-amd64-i386-xl-qemuu-debianhvm-amd64
> -test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm
> -test-amd64-i386-xl-qemuu-win7-amd64
> -test-amd64-i386-xl-qemuu-winxpsp3
> -test-amd64-i386-xl-qemuu-winxpsp3-vcpus1
> 
> Signed-off-by: Ian Campbell <ian.campbell@citrix.com>

In fact you can also drop all PV test cases.

But that's one for another day.

> ---
>  make-flight | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/make-flight b/make-flight
> index c763ce9..3b31939 100755
> --- a/make-flight
> +++ b/make-flight
> @@ -92,6 +92,10 @@ job_create_test_filter_callback () {
>            *-qemuu-*) ;;
>            *) return 1;;
>        esac
> +      case " $* " in
> +        *" bios=ovmf "*) ;;
> +        *) return 1;;
> +      esac
>        ;;
>      *)
>        case "$job" in
> -- 
> 2.1.4

Re: [PATCH OSSTEST v1] make-flight: Only test ovmf guests on ovmf flight

[Ian Campbell]

On Tue, 2015-07-07 at 17:31 +0100, Wei Liu wrote:
> On Tue, Jul 07, 2015 at 05:26:45PM +0100, Ian Campbell wrote:
> > Specifically drop all of the following test jobs from the ovmf flight:
> > 
> > -test-amd64-amd64-xl-qemuu-debianhvm-amd64
> > -test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm
> > -test-amd64-amd64-xl-qemuu-win7-amd64
> > -test-amd64-amd64-xl-qemuu-winxpsp3
> > -test-amd64-i386-qemuu-rhel6hvm-amd
> > -test-amd64-i386-qemuu-rhel6hvm-intel
> > -test-amd64-i386-xl-qemuu-debianhvm-amd64
> > -test-amd64-i386-xl-qemuu-debianhvm-amd64-xsm
> > -test-amd64-i386-xl-qemuu-win7-amd64
> > -test-amd64-i386-xl-qemuu-winxpsp3
> > -test-amd64-i386-xl-qemuu-winxpsp3-vcpus1
> > 
> > Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
> 
> In fact you can also drop all PV test cases.

I think they are already taken care of. I think due to only testing jobs
with "-qemuu" in the name:

After this change the full set of jobs is:
        build-amd64
        build-amd64-libvirt
        build-amd64-pvops
        build-amd64-xsm
        build-i386
        build-i386-libvirt
        build-i386-pvops
        build-i386-xsm
        test-amd64-amd64-xl-qemuu-ovmf-amd64
        test-amd64-i386-xl-qemuu-ovmf-amd64

> But that's one for another day.

One some time ago I think ;-)