[lxc-devel] [PATCH] add travis-ci support to LXC github repo
Serge Hallyn
serge.hallyn at ubuntu.com
Mon Dec 23 19:19:32 UTC 2013
I'll admit knowing in advance whether compilation would break could be
nice. But switching to purely git pull based model would be quite a
change for us.
Quoting Ranjib Dey (dey.ranjib at gmail.com):
> we'll have several other advantages with travis. the build status can be
> reflected on the project page easily, downstream clients (language specific
> bindings or other lxc dependent libraries) can reuse the build status to
> trigger their own builds , which we'll allow then to catch any breakage
> early on.
>
> but the whole model is very tied to github and pull request based
> development, so unless lxc is aopting that, travis usage might not yield a
> lot of value,
>
>
>
>
> On Mon, Dec 23, 2013 at 2:03 PM, S.Çağlar Onur <caglar at 10ur.org> wrote:
>
> > Hi Serge,
> >
> > On Mon, Dec 23, 2013 at 10:42 AM, Serge Hallyn <serge.hallyn at ubuntu.com>
> > wrote:
> > > Is this something you've discussed with Stéphane? A set of tests is
> > > already being run on his own server, so I'm not sure this is needed.
> > > There is the "the more testing the better" point of view, but I've seen
> > > more time wasted trying to debug test "breakages" that were actually
> > > host system errors (at least on one of the test harnasses we were using
> > > for lxc) than real errors found, so I'm a bit hesitant.
> >
> > No, I haven't discussed this with Stéphane. IMHO this only ensures
> > that master builds fine after a commit. We don't run test cases as
> > travis-ci vms not capable of creating LXC containers for now. One
> > advantage of travis over others is its integration with github. When
> > someone sends a pull request, travis builds that changeset on top of
> > master so that one can see whether merging the pull request is OK or
> > not.
> >
> > But if you think that's unnecessary, or think that this duplicates
> > what qa.linuxcontainers.org does then please feel free to drop this :)
> >
> > > I'd be more interested in new tests for specific use cases - like a
> > > unprivileged container create/start - which aren't currently being
> > > tested, or cleanups to the existing tests cases to make them more
> > > robust and useful.
> > >
> > > thanks,
> > > -serge
> > >
> > > Quoting Ranjib Dey (dey.ranjib at gmail.com):
> > >> +1 . too bad travis run on openvz :-( . else we could have run the
> > tests .
> > >>
> > >>
> > >> On Sun, Dec 22, 2013 at 11:58 PM, S.Çağlar Onur <caglar at 10ur.org>
> > wrote:
> > >>
> > >> > Travis is a free hosted CI platform for the open source community. It
> > >> > integrates
> > >> > well with github and enables continous builds/tests for both
> > repository
> > >> > itself
> > >> > and all the pull requests so that one can quickly see the result of
> > the
> > >> > possible
> > >> > merge.
> > >> >
> > >> > This yml file is one of the few required steps to enable travis-ci
> > support
> > >> > for
> > >> > LXC github repo. One of you guys still need to sign in travis-ci
> > through
> > >> > GitHub OAuth
> > >> > and enable travis support from its profile page
> > >> > https://travis-ci.org/profile
> > >> >
> > >> > As an example
> > >> > https://travis-ci.org/caglar10ur/lxc-upstream/builds/15872074 can be
> > seen
> > >> >
> > >> > Signed-off-by: S.Çağlar Onur <caglar at 10ur.org>
> > >> > ---
> > >> > .travis.yml | 9 +++++++++
> > >> > 1 file changed, 9 insertions(+)
> > >> > create mode 100644 .travis.yml
> > >> >
> > >> > diff --git a/.travis.yml b/.travis.yml
> > >> > new file mode 100644
> > >> > index 0000000..e7e3776
> > >> > --- /dev/null
> > >> > +++ b/.travis.yml
> > >> > @@ -0,0 +1,9 @@
> > >> > +language: c
> > >> > +compiler:
> > >> > + - gcc
> > >> > +# failing with python
> > >> > +# - clang
> > >> > +before_install:
> > >> > + - sudo apt-get update -qq
> > >> > + - sudo apt-get install -qq libapparmor-dev libcap-dev libseccomp-dev
> > >> > python3-dev
> > >> > +script: ./autogen.sh && CFLAGS="-g -O2 -fstack-protector
> > >> > --param=ssp-buffer-size=4 -Wformat -Werror=format-security"
> > >> > CPPFLAGS="-D_FORTIFY_SOURCE=2" CXXFLAGS="-g -O2 -fstack-protector
> > >> > --param=ssp-buffer-size=4 -Wformat -Werror=format-security"
> > FFLAGS="-g -O2"
> > >> > LDFLAGS="-Wl,-Bsymbolic-functions -Wl,-z,relro" ./configure
> > --prefix=/usr
> > >> > --sysconfdir=/etc --bindir=/usr/bin
> > --libdir=/usr/lib/x86_64-linux-gnu/
> > >> > --localstatedir=/var --includedir=/usr/include --disable-rpath
> > >> > --enable-python --enable-tests --enable-apparmor --with-distro=ubuntu
> > &&
> > >> > make -j4
> > >> > --
> > >> > 1.8.3.2
> > >> >
> > >> > _______________________________________________
> > >> > lxc-devel mailing list
> > >> > lxc-devel at lists.linuxcontainers.org
> > >> > http://lists.linuxcontainers.org/listinfo/lxc-devel
> > >> >
> > >
> > >> _______________________________________________
> > >> lxc-devel mailing list
> > >> lxc-devel at lists.linuxcontainers.org
> > >> http://lists.linuxcontainers.org/listinfo/lxc-devel
> > >
> > > _______________________________________________
> > > lxc-devel mailing list
> > > lxc-devel at lists.linuxcontainers.org
> > > http://lists.linuxcontainers.org/listinfo/lxc-devel
> >
> >
> >
> > --
> > S.Çağlar Onur <caglar at 10ur.org>
> > _______________________________________________
> > lxc-devel mailing list
> > lxc-devel at lists.linuxcontainers.org
> > http://lists.linuxcontainers.org/listinfo/lxc-devel
> >
> _______________________________________________
> lxc-devel mailing list
> lxc-devel at lists.linuxcontainers.org
> http://lists.linuxcontainers.org/listinfo/lxc-devel
More information about the lxc-devel
mailing list