[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [OSSTEST PATCH v4 8/9] make-flight: factor out do_pv_debian_tests



On Wed, 2014-12-10 at 22:21 +0000, Wei Liu wrote:
> Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> ---
>  make-flight |   21 ++++++++++++++-------
>  1 file changed, 14 insertions(+), 7 deletions(-)
> 
> diff --git a/make-flight b/make-flight
> index 9963a46..35904be 100755
> --- a/make-flight
> +++ b/make-flight
> @@ -281,17 +281,24 @@ do_passthrough_tests () {
>    done
>  }
>  
> -test_matrix_do_one () {
> -
> -  # Basic PV Linux test with xl
> +do_pv_debian_test_one () {
> +  toolstack=$1
>  
> -  job_create_test test-$xenarch$kern-$dom0arch-xl test-debian xl \
> +  job_create_test test-$xenarch$kern-$dom0arch-$toolstack test-debian 
> $toolstack \
>              $xenarch $dom0arch                                   \
>              $debian_runvars all_hostflags=$most_hostflags
> +}
>  
> -  job_create_test test-$xenarch$kern-$dom0arch-libvirt test-debian libvirt \
> -            $xenarch $dom0arch                                       \
> -            $debian_runvars all_hostflags=$most_hostflags
> +do_pv_debian_tests () {
> +  for toolstack in xl libvirt; do
> +    do_pv_debian_test_one $toolstack
> +  done
> +}
> +
> +test_matrix_do_one () {
> +
> +  # Basic PV Linux test with xl

This comment isn't true, since libvirt is included too.

Did you confirm no changes to runvars after this change? If so then
please say so in the commit log, then:
Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

> +  do_pv_debian_tests
>  
>    # No further arm tests at the moment
>    if [ $dom0arch = armhf ]; then



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.