Re: [Xen-devel] [PATCH OSSTEST v4 3/3] Create a flight to test OpenStack with xen-unstable and libvirt

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Fri, 2015-11-20 at 14:55 +0000, Anthony PERARD wrote:

I think this will also add a devstack job to most other flights? That's a
good thing, I think.

I wonder if the flight ought to be called openstack-nova, to leave open the
possibility of other openstack-$foo flights in the future?

> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> 
> ---
> Change in V4:
> - also skip build-*-oldkern in make flight

Those should already be caught by cr-daily-branch
settingÂREVISION_LINUX_OLD=disable for all but the xen-unstable flight
(arguably that default ought to be inverted, such that make-flight by
default doesn't make such flights unless asked to, but that's not your Yakk
I think)

> - fix select_xenbranch
> - set revision_*=$REVISION_OPENSTACK_* in make-flight
> Â (was revision_*=master before)
> Â only REVISION_OPENSTACK_NOVA is set, the others are unset.
> Â empty revision_* runvar would clone the default branch, which should
> Â be master for every openstack repos

The sort of info in this last bullet would be useful in the commit message,
I think.

> diff --git a/make-flight b/make-flight
> index 8523995..5a4fc0c 100755
> --- a/make-flight
> +++ b/make-flight
> @@ -54,6 +54,12 @@ job_create_build_filter_callback () {
> ÂÂÂÂÂÂÂÂÂ*)                   return 1 ;;
> ÂÂÂÂÂÂÂesac
> ÂÂÂÂÂ;;
> +ÂÂÂÂopenstack)
> +ÂÂÂÂÂÂcase "$job" in
> +ÂÂÂÂÂÂÂÂ*-xsm) return 1;;

I wonder, would aÂtest-$xenarch$kern-$dom0arch-devstack-xsm be a useful
think to have though?

> +ÂÂÂÂÂÂÂÂ*-oldkern) return 1;;

See above.

> diff --git a/mfi-common b/mfi-common
> index 5fbe195..f11302b 100644
> --- a/mfi-common
> +++ b/mfi-common
> @@ -59,6 +59,7 @@ xenbranch_xsm_variants () {
> ÂÂÂÂÂxen-4.3-testing) echo "false";;
> ÂÂÂÂÂxen-4.4-testing) echo "false";;
> ÂÂÂÂÂxen-4.5-testing) echo "false";;
> +ÂÂÂÂopenstack)ÂÂÂÂÂÂÂecho "false";;
> ÂÂÂÂÂ*) echo "false true";
> ÂÂÂÂÂesac
> Â}
> @@ -102,6 +103,7 @@ create_build_jobs () {
> ÂÂÂÂÂÂÂrumpuserxen) continue;;
> ÂÂÂÂÂÂÂseabios) continue;;
> ÂÂÂÂÂÂÂovmf) continue;;
> +ÂÂÂÂÂÂopenstack) continue;;
> ÂÂÂÂÂÂÂesac
> ÂÂÂÂÂÂÂcase "$xenbranch" in
> ÂÂÂÂÂÂÂxen-3.*-testing) continue;;
> @@ -127,6 +129,9 @@ create_build_jobs () {
> ÂÂÂÂÂÂÂ"
> ÂÂÂÂÂÂÂ;;
> ÂÂÂÂÂesac
> +ÂÂÂÂif [ "$arch" = i386 ] && [ "$branch" = openstack ]; then
> +ÂÂÂÂÂÂcontinue

This accepts ARM, but I think you filter the test cases for that? The
filtering of build vs. test jobs in make-flight/mfi-common is a bit of a
mess, recently e21625f79d33 "make-flight: move in-lined branch vs arch
filtering into callbacks" make it a bit less bad, which might be useful to
you here?

> +ÂÂÂÂfi
> Â
> ÂÂÂÂÂcase "$arch" in
> ÂÂÂÂÂ*)ÂÂÂÂÂsuite=$defsuite;;

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