Hi Serge, On Mon, Dec 23, 2013 at 10:42 AM, Serge Hallyn <[email protected]> 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 ([email protected]): >> +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 <[email protected]> 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 <[email protected]> >> > --- >> > .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 >> > [email protected] >> > http://lists.linuxcontainers.org/listinfo/lxc-devel >> > > >> _______________________________________________ >> lxc-devel mailing list >> [email protected] >> http://lists.linuxcontainers.org/listinfo/lxc-devel > > _______________________________________________ > lxc-devel mailing list > [email protected] > http://lists.linuxcontainers.org/listinfo/lxc-devel -- S.Çağlar Onur <[email protected]> _______________________________________________ lxc-devel mailing list [email protected] http://lists.linuxcontainers.org/listinfo/lxc-devel
