mbox series

[v1,00/23] Current Travis Queue

Message ID 20170717144848.11793-1-alex.bennee@linaro.org
Headers show
Series Current Travis Queue | expand

Message

Alex Bennée July 17, 2017, 2:48 p.m. UTC
Hi,

This is the current status of the travis/next patch queue. The
includes updates from Paolo to allow parallelism while testing in the
docker environment. I've extended the travis image so we can actually
run our travis.py script in the Travis image.

There are also a number of updates from Phillipe which add a bunch of
additional cross compile targets to our shippable setup. The cachinfo
patch is temporary and won't make the pull as it is already queued in
Richard's tcg-next.

I'm currently trying to catch one of our Travis hangs in the act
(postcopy-test) but it seems to be very much a heavy load race
condition which annoyingly stops happening once you try and get
debugging tools on it. This is the reason I've updated the travis
docker image to include the debug tools ;-)

As long as there are no screams of outrage I'll roll a pullreq for
softfreeze tomorrow.

Alex.

Alex Bennée (3):
  docker: include python-yaml in travis.docker
  docker: add debug tools to travis.docker
  docker: add debian/powerpc based on Jessie

Paolo Bonzini (2):
  docker: allow customizing Travis global_env variables
  travis: move make -j flag out of script

Philippe Mathieu-Daudé (18):
  docker: rename debian stable -> 9 (Stretch)
  docker: add pkg-config and python-minimal packages to debian base
  docker: debian/s390x no more in unstable, now available in Stretch
  docker: remove packages now dependent of qemu in Stretch
  docker: add debian/armel based on Stretch
  docker: add debian/ppc64el based on Stretch
  docker: add debian/amd64 based on Stretch
  docker: enable virgl to extend code coverage on amd64
  docker: enable netmap to extend code coverage on amd64
  docker: enable gcrypt to extend code coverage on amd64
  docker: enable nettle to extend code coverage on arm64
  shippable: add armel targets
  shippable: add ppc64el targets
  shippable: add x86_64 targets
  util/cacheinfo: add missing include (TEMP)
  docker: add 'apt-fake' script which generate fake debian packages
  docker: add qemu:debian-jessie based on outdated jessie release
  shippable: add powerpc target

 .shippable.yml                                     |  8 ++++
 .travis.yml                                        |  3 +-
 tests/docker/Makefile.include                      | 13 ++++--
 tests/docker/dockerfiles/debian-amd64.docker       | 39 ++++++++++++++++++
 tests/docker/dockerfiles/debian-apt-fake.sh        | 46 ++++++++++++++++++++++
 tests/docker/dockerfiles/debian-arm64-cross.docker | 10 ++---
 tests/docker/dockerfiles/debian-armel-cross.docker | 24 +++++++++++
 tests/docker/dockerfiles/debian-armhf-cross.docker |  7 +---
 .../docker/dockerfiles/debian-mipsel-cross.docker  |  7 +---
 .../docker/dockerfiles/debian-powerpc-cross.docker | 40 +++++++++++++++++++
 .../docker/dockerfiles/debian-ppc64el-cross.docker | 24 +++++++++++
 tests/docker/dockerfiles/debian-s390x-cross.docker | 23 +++++------
 tests/docker/dockerfiles/debian8.docker            | 31 +++++++++++++++
 .../dockerfiles/{debian.docker => debian9.docker}  | 15 +++++--
 tests/docker/dockerfiles/travis.docker             |  2 +-
 tests/docker/travis.py                             |  6 +--
 util/cacheinfo.c                                   |  1 +
 17 files changed, 259 insertions(+), 40 deletions(-)
 create mode 100644 tests/docker/dockerfiles/debian-amd64.docker
 create mode 100755 tests/docker/dockerfiles/debian-apt-fake.sh
 create mode 100644 tests/docker/dockerfiles/debian-armel-cross.docker
 create mode 100644 tests/docker/dockerfiles/debian-powerpc-cross.docker
 create mode 100644 tests/docker/dockerfiles/debian-ppc64el-cross.docker
 create mode 100644 tests/docker/dockerfiles/debian8.docker
 rename tests/docker/dockerfiles/{debian.docker => debian9.docker} (59%)

-- 
2.13.0

Comments

Fam Zheng July 17, 2017, 11 p.m. UTC | #1
On Mon, 07/17 15:48, Alex Bennée wrote:
> Hi,

> 

> This is the current status of the travis/next patch queue. The

> includes updates from Paolo to allow parallelism while testing in the

> docker environment. I've extended the travis image so we can actually

> run our travis.py script in the Travis image.

> 

> There are also a number of updates from Phillipe which add a bunch of

> additional cross compile targets to our shippable setup. The cachinfo

> patch is temporary and won't make the pull as it is already queued in

> Richard's tcg-next.

> 

> I'm currently trying to catch one of our Travis hangs in the act

> (postcopy-test) but it seems to be very much a heavy load race

> condition which annoyingly stops happening once you try and get

> debugging tools on it. This is the reason I've updated the travis

> docker image to include the debug tools ;-)

> 

> As long as there are no screams of outrage I'll roll a pullreq for

> softfreeze tomorrow.


Renaming of debian to debian9 seems backward incompatible. We don't care that
much, but I'd like to see a commit message justifying the change. Otherwise it
looks good to go (minus the cachinfo patch, of course).

Fam
Alex Bennée July 18, 2017, 8:32 a.m. UTC | #2
Fam Zheng <famz@redhat.com> writes:

> On Mon, 07/17 15:48, Alex Bennée wrote:

>> Hi,

>>

>> This is the current status of the travis/next patch queue. The

>> includes updates from Paolo to allow parallelism while testing in the

>> docker environment. I've extended the travis image so we can actually

>> run our travis.py script in the Travis image.

>>

>> There are also a number of updates from Phillipe which add a bunch of

>> additional cross compile targets to our shippable setup. The cachinfo

>> patch is temporary and won't make the pull as it is already queued in

>> Richard's tcg-next.

>>

>> I'm currently trying to catch one of our Travis hangs in the act

>> (postcopy-test) but it seems to be very much a heavy load race

>> condition which annoyingly stops happening once you try and get

>> debugging tools on it. This is the reason I've updated the travis

>> docker image to include the debug tools ;-)

>>

>> As long as there are no screams of outrage I'll roll a pullreq for

>> softfreeze tomorrow.

>

> Renaming of debian to debian9 seems backward incompatible. We don't care that

> much, but I'd like to see a commit message justifying the change. Otherwise it

> looks good to go (minus the cachinfo patch, of course).


Well originally we were basing off debian-stable but we got caught out
by the new release. With this series we have most things off the latest
stable (Debian 9 - aka Stretch) but one or two builds are based of old
stable (Debian 8 - aka Jessie) and using the embedian hacks we had
before.

I think for this it's mostly a matter of clarity.

--
Alex Bennée
Fam Zheng July 20, 2017, 7:59 a.m. UTC | #3
On Mon, 07/17 23:47, Philippe Mathieu-Daudé wrote:
> Sorry I messed previous series... just updated last 5 patches of v2.


Please do full respins and don't post as in-reply-to previous versions, this
doesn't make things any easier for maintainers.

Fam