From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [OSSTEST PATCH 29/33] make-flight: Provide xen-unstable-smoke branch Date: Fri, 18 Sep 2015 10:50:46 +0100 Message-ID: <1442569846.18856.282.camel@citrix.com> References: <1442511480-32587-1-git-send-email-ian.jackson@eu.citrix.com> <1442511480-32587-30-git-send-email-ian.jackson@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1ZcsJf-0003Gx-0X for xen-devel@lists.xenproject.org; Fri, 18 Sep 2015 09:50:51 +0000 In-Reply-To: <1442511480-32587-30-git-send-email-ian.jackson@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Jackson , xen-devel@lists.xenproject.org List-Id: xen-devel@lists.xenproject.org On Thu, 2015-09-17 at 18:37 +0100, Ian Jackson wrote: > This contains a very limited set of jobs > test-amd64-amd64-libvirt > test-amd64-amd64-xl-qemuu-debianhvm-i386 > test-armhf-armhf-xl > and the builds they depend on. > > The debianhvm job exists only in this flight, and is generated by > having branch_debianhvm_arch return i386 instead of amd64. This is so > that this branch contains a 32-bit x86 guest as well as a 64-bit one. > > We override host allocator parameters to make this flight not care > about host stickiness: it just takes whatever comes to hand. These > runvars are marked `synth' so that cs-bisection-step and > cs-adjust-flight do not copy them, as discussed in previous patches. Didn't you drop it from those previous patches with the intention of moving it here? Everything else looks good. > Later we will arrange to reuse previous builds for the build artefacts > which aren't intended subjects of the smoke test. > > (Deployment note: This needs images/debian-7.2.0-i386-CD-1.iso which I > have already placed in the Cambridge and Xen Project instances.) > > Signed-off-by: Ian Jackson > --- > v2: Generate all the jobs that this flight's tests use, and add > note about this to the commit message. > Mention `synth'-ness of hostalloc runvars in commit message. > Image is in Xen Project test colo too. > --- > make-flight | 31 ++++++++++++++++++++++++++++++- > 1 file changed, 30 insertions(+), 1 deletion(-) > > diff --git a/make-flight b/make-flight > index 13674b8..c1f40b9 100755 > --- a/make-flight > +++ b/make-flight > @@ -34,8 +34,28 @@ flight=`./cs-flight-create $blessing $branch` > defsuite=`getconfig DebianSuite` > defguestsuite=`getconfig GuestDebianSuite` > > +case "$branch" in > +xen-unstable-smoke) > + global_runvars+=" hostalloc_maxbonus_variation~=0 " > + global_runvars+=" hostalloc_bonus_previousfail~=0 " > + ;; > +esac > + > job_create_build_filter_callback () { > - : > + local job=$1; shift > + case "$branch" in > + xen-unstable-smoke) > + case "$job" in > + build-amd64) ;; > + build-amd64-pvops) ;; > + build-amd64-libvirt) ;; > + build-armhf) ;; > + build-armhf-pvops) ;; > + *) return 1 ;; > + esac > + ;; > + esac > + return 0 > } > > if [ x$buildflight = x ]; then > @@ -56,6 +76,14 @@ job_create_test_filter_callback () { > local dom0arch=$1; shift > > case "$branch" in > + xen-unstable-smoke) > + case "$job" in > + test-amd64-amd64-libvirt) return 0 ;; > + test-armhf-armhf-xl) return 0 ;; > + test-amd64-amd64-xl-qemuu-debianhvm-i386) return 0 ;; > + *) return 1 ;; > + esac > + ;; > qemu-upstream-4.2-testing) > case " $* " in > *" device_model_version=qemu-xen "*) > @@ -222,6 +250,7 @@ do_hvm_win7_x64_tests () { > > branch_debianhvm_arch () { > case $branch in > + xen-unstable-smoke) echo i386;; > *) echo amd64;; > esac > }