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

Re: [Xen-devel] [PATCH v3] OSSTEST: introduce a raisin build test



On Mon, 11 May 2015, Ian Campbell wrote:
> On Fri, 2015-05-08 at 13:36 +0100, Stefano Stabellini wrote:
> > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > 
> > ---
> > Changes in v3:
> > - use $raisindir throughout ts-raisin-build
> > - do not specify ENABLED_COMPONENTS so that empty REVISION variables can
> > be used to disable building a raisin component
> 
> Is this existing raisin behaviour, or is there a patch pending for that?

It is existing behaviour.


> > Changes in v2:
> > - set revision_* variables in mfi-common;
> > - in ts-raisin-build set the *_REVISION config options based on the
> >   revision_* variables;
> > - in ts-raisin-build, call store_revision appropriately;
> > - divide the output in an hypervisor and a tools tarball.
> > ---
> >  ap-common       |    5 ++
> >  mfi-common      |   25 +++++++++
> >  sg-run-job      |    5 ++
> >  ts-raisin-build |  161 
> > +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  4 files changed, 196 insertions(+)
> >  create mode 100755 ts-raisin-build
> > 
> > diff --git a/ap-common b/ap-common
> > index 64749e3..985eeec 100644
> > --- a/ap-common
> > +++ b/ap-common
> > @@ -47,13 +47,18 @@
> >  # rumpsrc-related runvars needed only for old rumpuser-xen
> >  # (ie ones which need $bodges=1 in ts-rumpuserxen-build)
> >  
> > +: ${TREE_RAISIN:=git://xenbits.xen.org/people/sstabellini/raisin.git}
> > +: ${DEFAULT_REVISION_RAISIN:=master}
> > +
> >  : ${TREE_SEABIOS_UPSTREAM:=git://git.seabios.org/seabios.git}
> >  : ${PUSH_TREE_SEABIOS:=$XENBITS:/home/xen/git/osstest/seabios.git}
> >  : ${BASE_TREE_SEABIOS:=git://xenbits.xen.org/osstest/seabios.git}
> > +: ${TREE_SEABIOS:=$TREE_SEABIOS_UPSTREAM}
> >  
> >  : ${TREE_OVMF_UPSTREAM:=https://github.com/tianocore/edk2.git}
> >  : ${PUSH_TREE_OVMF:=$XENBITS:/home/xen/git/osstest/ovmf.git}
> >  : ${BASE_TREE_OVMF:=git://xenbits.xen.org/osstest/ovmf.git}
> > +: ${TREE_OVMF:=$TREE_OVMF_UPSTREAM}
> 
> What are these two doing? If it's something legitimate then it really
> needs to be mentioned in the commit log, but using
> git://git.seabios.org/seabios.git here seems wrong, and I think we
> change things for older branches and the existing ts-xen-build job.

I thought it was wrong that all the other components provide a TREE_
variable, but OVMF and SEABIOS. Running in stand-alone mode, their
absence would break raisin compilation. From a look at mfi-common, it
seems to me that other jobs could break too for the same reason.


> I have a feeling that these relate to trees which we previously allowed
> xen.git:Config.mk to choose, which means we need to have a little think
> about the behaviour of cr-daily-branch vs this script here.
> 
> Perhaps you should do for TREE_{SEABIOS,OVMF} as you have done for the
> revision, i.e. use ${TREE_SEABIOS:-<SOMEDEFAULT>}
>
> Where <SOMEDEFAULT> is whatever is in xen.git Config.mk today.

Yes, that would work too. Do you mean here, or under ./cs-job-create in
mfi-common?


> 
> Or maybe raisin has some way to say "whatever you think is best"?
> 
> The question here is, I suppose, where should the replacement for the
> Config.mk supplied defaults live now, in osstest.git or raisin.git?

The defaults are available in raisin to users via the default config
(see defconfig), that we are not using because we want to pass specific
settings to raisin, in particular we want to specify a REVISION for
each component. As we are down to this level of control, it seems
natural to me to also specify the trees we want to use.


> >  : ${TREE_LINUXFIRMWARE:=git://xenbits.xen.org/osstest/linux-firmware.git}
> >  : ${PUSH_TREE_LINUXFIRMWARE:=$XENBITS:/home/osstest/ext/linux-firmware.git}
> > diff --git a/mfi-common b/mfi-common
> > index 16fc8c5..57bbf1c 100644
> > --- a/mfi-common
> > +++ b/mfi-common
> > @@ -215,6 +215,31 @@ create_build_jobs () {
> >  
> >      fi
> >  
> > +    if [ "x$REVISION_RAISIN" != xdisable ]; then
> > +
> > +    ./cs-job-create $flight build-$arch-raisin build-raisin                
> >   \
> > +                arch=$arch                                                 
> >   \
> > +        tree_xen=$TREE_XEN                                                 
> >   \
> > +                $RUNVARS $BUILD_RUNVARS $BUILD_RAISIN_RUNVARS 
> > $arch_runvars \
> > +                $suite_runvars                                             
> >   \
> > +                host_hostflags=$build_hostflags                            
> >   \
> > +                buildjob=${bfi}build-$arch                                 
> >   \
> > +                tree_raisin=$TREE_RAISIN                                   
> >   \
> > +                tree_qemuu=$TREE_QEMU_UPSTREAM                             
> >   \
> > +                tree_qemu=$TREE_QEMU                                       
> >   \
> > +                tree_seabios=$TREE_SEABIOS                                 
> >   \
> > +                tree_libvirt=$TREE_LIBVIRT                                 
> >   \
> > +                tree_ovmf=$TREE_OVMF                                       
> >   \
> > +                revision_xen=${REVISION_XEN:-master}                       
> >   \
> > +                revision_qemu=${REVISION_QEMU:-master}                     
> >   \
> > +                revision_qemuu=${REVISION_QEMU_UPSTREAM:-master}           
> >   \
> > +                revision_seabios=${REVISION_SEABIOS:-master}               
> >   \
> > +                revision_ovmf=${REVISION_OVMF:-master}                     
> >   \
> > +                revision_libvirt=${REVISION_LIBVIRT:-master}               
> >   \
> 
> I'm not sure about the fallback to master here.
> 
> In the current xen.git build stuff revision_foo='' means "use
> Config.mk".
> 
> > +                
> > revision_raisin=${REVISION_RAISIN:-${DEFAULT_REVISION_RAISIN}}\
> > +
> > +    fi
> > +
> >      if branch_wants_rumpkernel_tests; then
> >  
> >      case $arch in
> > diff --git a/sg-run-job b/sg-run-job
> > index eae159d..449118d 100755
> > --- a/sg-run-job
> > +++ b/sg-run-job
> > @@ -346,6 +346,7 @@ proc need-hosts/build {} { return BUILD }
> >  proc need-hosts/build-kern {} { return BUILD }
> >  proc need-hosts/build-libvirt {} { return BUILD }
> >  proc need-hosts/build-rumpuserxen {} { return BUILD }
> > +proc need-hosts/build-raisin {} { return BUILD }
> >  
> >  proc run-job/build {} {
> >      run-ts . = ts-xen-build
> > @@ -364,6 +365,10 @@ proc run-job/build-rumpuserxen {} {
> >      run-ts . = ts-xen-build + host tools
> >  }
> >  
> > +proc run-job/build-raisin {} {
> > +    run-ts . = ts-raisin-build
> > +}
> > +
> >  proc prepare-build-host {} {
> >      global jobinfo
> >      run-ts broken = ts-hosts-allocate + host
> > diff --git a/ts-raisin-build b/ts-raisin-build
> > new file mode 100755
> > index 0000000..94c6447
> > --- /dev/null
> > +++ b/ts-raisin-build
> > @@ -0,0 +1,161 @@
> > +#!/usr/bin/perl -w
> > +# This is part of "osstest", an automated testing framework for Xen.
> > +# Copyright (C) 2009-2013 Citrix Inc.
> > +# 
> > +# This program is free software: you can redistribute it and/or modify
> > +# it under the terms of the GNU Affero General Public License as published 
> > by
> > +# the Free Software Foundation, either version 3 of the License, or
> > +# (at your option) any later version.
> > +# 
> > +# This program is distributed in the hope that it will be useful,
> > +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> > +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > +# GNU Affero General Public License for more details.
> > +# 
> > +# You should have received a copy of the GNU Affero General Public License
> > +# along with this program.  If not, see <http://www.gnu.org/licenses/>.
> > +
> > +use strict qw(vars);
> > +use DBI;
> > +use Osstest;
> > +use File::Path;
> > +use POSIX;
> > +use Osstest::TestSupport;
> > +use Osstest::BuildSupport;
> > +
> > +tsreadconfig();
> > +selectbuildhost(\@ARGV);
> > +# remaining arguments are passed as targets to "make"
> > +builddirsprops();
> > +
> > +my $raisindir="$builddir/raisin";
> > +
> > +sub checkout () {
> > +    prepbuilddirs();
> > +    build_clone($ho, 'raisin', $builddir, 'raisin');
> > +
> > +    target_cmd_build($ho, 100, $builddir, <<END);
> > +        cd $raisindir
> > +   >config
> > +
> > +    echo >>config MAKE=\\"make $makeflags\\"
> > +    echo >>config PREFIX=\\"/usr\\"
> > +    echo >>config DESTDIR=dist
> > +
> > +    echo >>config XEN_URL=\\"$r{tree_xen}\\"
> > +    echo >>config QEMU_URL=\\"$r{tree_qemuu}\\"
> > +    echo >>config QEMU_TRADITIONAL_URL=\\"$r{tree_qemu}\\"
> > +    echo >>config SEABIOS_URL=\\"$r{tree_seabios}\\"
> > +    echo >>config LIBVIRT_URL=\\"$r{tree_libvirt}\\"
> > +    echo >>config OVMF_URL=\\"$r{tree_ovmf}\\"
> > +
> > +    echo >>config XEN_REVISION=\\"$r{revision_xen}\\"
> > +    echo >>config QEMU_REVISION=\\"$r{revision_qemuu}\\"
> > +    echo >>config QEMU_TRADITIONAL_REVISION=\\"$r{revision_qemu}\\"
> > +    echo >>config SEABIOS_REVISION=\\"$r{revision_seabios}\\"
> > +    echo >>config LIBVIRT_REVISION=\\"$r{revision_libvirt}\\"
> > +    echo >>config OVMF_REVISION=\\"$r{revision_ovmf}\\"
> > +END
> > +#/;
> > +}
> > +
> > +sub build () {
> > +    target_cmd_root($ho, <<END);
> > +        cd $raisindir
> > +        ./raise -y install-builddep
> > +END
> > +#/;
> > +    target_cmd_build($ho, 9000, $builddir, <<END);
> > +        cd $raisindir
> > +        ./raise -n build
> > +END
> > +#/;
> > +}
> > +
> > +sub checkoutput () {
> > +    target_cmd_build($ho, 100, $builddir, <<END);
> > +            cd $raisindir/dist
> > +            ls boot/xen.gz
> > +            ls usr/sbin/xenstored
> > +            ls usr/sbin/xenconsoled
> > +            ls usr/lib/libxenlight.so
> > +            ls usr/sbin/xl
> > +            ls usr/lib/xen/boot/hvmloader
> > +            ls usr/lib/xen/bin/qemu-dm
> > +            ls usr/lib/xen/bin/qemu-system-i386
> > +            ls usr/sbin/libvirtd
> > +END
> > +#/;
> > +}
> > +
> > +sub collectversions () {
> > +    store_revision($ho, 'xen', "$raisindir/xen-dir", 1);
> > +    store_revision($ho, 'qemu', "$raisindir/qemu-traditional-dir", 1);
> > +    store_revision($ho, 'qemuu', "$raisindir/qemu-dir", 1);
> > +    store_revision($ho, 'seabios', "$raisindir/seabios-dir", 1);
> > +    store_revision($ho, 'ovmf', "$raisindir/ovmf-dir", 1);
> > +    store_revision($ho, 'libvirt', "$raisindir/libvirt-dir", 1);
> > +}
> > +
> > +sub divide () {
> > +    # Only move hv to xeninstall, so that we can have
> > +    # xenpolicy in tools tarball.
> > +    #
> > +    # The files inside boot/ after `make dist' are
> > +    # xen-$XEN_VERSION: Xen binary
> > +    # xen.gz/xen: symlink to xen-$XEN_VERSION
> > +    # xen-$MAJOR: symlink to xen-$XEN_VERSION
> > +    # xen-$MAJOR.$MINOR: symlink to xen-$XEN_VERSION
> > +    # xen-sym-$XEN_VERSION: Xen symbol
> > +    # xenpolicy-$XEN_VERSION: flask policy binary if xsm is enabled
> > +    #
> > +    # So the following snippet will leave xenpolicy* in
> > +    # install/boot and get packaged to tools tarball.
> > +    target_cmd_build($ho, 100, $builddir, <<END);
> > +        cd raisin
> > +        mkdir xendist
> > +        for f in *dist; do
> > +            mkdir -p \$f/lib
> > +        done
> > +        if test -d dist/boot; then
> > +            if test -f dist/boot/xen.gz || test -f dist/boot/xen; then
> > +                mkdir xendist/boot
> > +                mvfiles=`find dist/boot -name 'xen[a-z]*' -prune -o -name 
> > 'xen*' -print`
> > +                mv \$mvfiles xendist/boot/.
> > +            fi
> > +        fi
> > +END
> > +}
> > +
> > +sub stash () {
> > +    foreach my $part ('', 'xen') {
> > +        built_stash($ho, $builddir,
> > +                    "raisin/${part}dist",
> > +                    "${part}dist");
> > +    }
> > +    built_stash_file($ho, $builddir, "xen-syms", 
> > "raisin/xen-dir/xen/xen-syms", 1);
> > +    built_stash_file($ho, $builddir, "raisin-config", "raisin/config", 1);
> > +    built_stash_file($ho, $builddir, "seabios-config",
> > +             "raisin/seabios-dir/.config", 1);
> > +    built_compress_stashed("xen-syms");
> > +}
> > +
> > +
> > +our @probs;
> > +
> > +sub trapping ($) {
> > +    my ($sub) = @_;
> > +    my $tok= eval { $sub->(); 1; };
> > +    if (!$tok) { push @probs, $@; print STDERR "failure (trapped): $@\n"; }
> > +}
> > +
> > +checkout();
> > +trapping(\&build);
> > +trapping(\&checkoutput);
> > +trapping(\&collectversions);
> > +divide();
> > +stash();
> > +
> > +die "*** something failed:\n\n".(join "\n\n",@probs)."\n** something 
> > failed"
> > +    if @probs;
> > +
> 
> 

_______________________________________________
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®.