Re: [OSSTEST PATCH v3 2/3] Testing cpupools: recipe for it and job definition

[Ian Campbell <ian.campbell@citrix.com>]

On Sat, 2015-10-03 at 02:39 +0200, Dario Faggioli wrote:
> Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
> ---
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: Juergen Gross <jgross@suse.com>

This looks correct to me as it stands, but I think it will be impacted by
the changes relating to host flags for numbers of cpus etc.

> ---
> Changes from v2:
>  * restrict test generation to xl only.
> 
> Changes from v1:
>  * added invocation to ts-guest-stop in the recipe to kill
>    leak-check complaints (which went unnoticed during v1
>    testing, sorry)
>  * moved the test before the "ARM cutoff", and remove the
>    per-arch filtering, so that the test can run on ARM
>    hardware too
> ---
>  make-flight |   12 ++++++++++++
>  sg-run-job  |    7 +++++++
>  2 files changed, 19 insertions(+)
> 
> diff --git a/make-flight b/make-flight
> index 8c75a9c..d27a02c 100755
> --- a/make-flight
> +++ b/make-flight
> @@ -373,6 +373,16 @@ do_multivcpu_tests () {
>                      $debian_runvars all_hostflags=$most_hostflags
>  }
>  
> +do_cpupools_tests () {
> +  if [ x$toolstack != xxl -a $xenarch != $dom0arch ]; then
> +    return
> +  fi
> +
> +  job_create_test test-$xenarch$kern-$dom0arch-xl-cpupools            \
> +                    test-cpupools xl $xenarch $dom0arch               \
> +                    $debian_runvars all_hostflags=$most_hostflags
> +}
> +
>  do_passthrough_tests () {
>    if [ $xenarch != amd64 -o $dom0arch != amd64 -o "$kern" != "" ]; then
>      return
> @@ -498,6 +508,8 @@ test_matrix_do_one () {
>    do_rtds_tests
>    do_credit2_tests
>  
> +  do_cpupools_tests
> +
>    # No further arm tests at the moment
>    if [ $dom0arch = armhf ]; then
>        return
> diff --git a/sg-run-job b/sg-run-job
> index 66145b8..ea48a03 100755
> --- a/sg-run-job
> +++ b/sg-run-job
> @@ -296,6 +296,13 @@ proc run-job/test-debianhvm {} {
>      test-guest debianhvm
>  }
>  
> +proc need-hosts/test-cpupools {} { return host }
> +proc run-job/test-cpupools {} {
> +    install-guest-debian
> +    run-ts . = ts-cpupools + host debian
> +    run-ts . = ts-guest-stop + host debian
> +}
> +
>  proc setup-test-pair {} {
>      run-ts . =              ts-debian-install      dst_host
>      run-ts . =              ts-debian-fixup        dst_host          +
> debian
>