qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
Cc: qemu-devel@nongnu.org, Fam Zheng <famz@redhat.com>,
	Peter Maydell <peter.maydell@linaro.org>
Subject: Re: [Qemu-devel] [PATCH 06/11] travis: retry if llvm.org timeouts
Date: Mon, 19 Jun 2017 12:00:55 +0100	[thread overview]
Message-ID: <8737awutqw.fsf@linaro.org> (raw)
In-Reply-To: <20170616161334.7492-7-f4bug@amsat.org>


Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> example of failure: https://travis-ci.org/qemu/qemu/jobs/243232857
>
>     $ sudo apt-get update -qq
>     W: Failed to fetch http://llvm.org/apt/trusty/dists/llvm-toolchain-trusty-3.9/Release.gpg  Connection failed
>     E: Some index files failed to download. They have been ignored, or old ones used instead.
>     The command "sudo apt-get update -qq" failed and exited with 100 during .
>     Your build has been stopped.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

This is good - I suspect there are other places we could use travis_retry.

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

> ---
>  .travis.yml | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index dbbb11617d..69a960b714 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -123,10 +123,10 @@ matrix:
>          - COMPILER_NAME=clang CXX=clang++-3.9 CC=clang-3.9
>          - CONFIG="--disable-linux-user --cc=clang-3.9 --cxx=clang++-3.9"
>        before_install:
> -        - wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
> -        - sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
> -        - sudo apt-get update -qq
> -        - sudo apt-get install -qq -y clang-3.9
> +        - travis_retry wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
> +        - travis_retry sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
> +        - travis_retry sudo apt-get update -qq
> +        - travis_retry sudo apt-get install -qq -y clang-3.9
>          - sudo apt-get build-dep -qq qemu
>          - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
>          - git submodule update --init --recursive
> @@ -142,10 +142,10 @@ matrix:
>          - COMPILER_NAME=clang CXX=clang++-3.9 CC=clang-3.9
>          - CONFIG="--disable-system --cc=clang-3.9 --cxx=clang++-3.9"
>        before_install:
> -        - wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
> -        - sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
> -        - sudo apt-get update -qq
> -        - sudo apt-get install -qq -y clang-3.9
> +        - travis_retry wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
> +        - travis_retry sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
> +        - travis_retry sudo apt-get update -qq
> +        - travis_retry sudo apt-get install -qq -y clang-3.9
>          - sudo apt-get build-dep -qq qemu
>          - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
>          - git submodule update --init --recursive


--
Alex Bennée

  reply	other threads:[~2017-06-19 11:00 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-16 16:13 [Qemu-devel] [PATCH 00/11] travis: try to reduce failures Philippe Mathieu-Daudé
2017-06-16 16:13 ` [Qemu-devel] [PATCH 01/11] travis: build using all available cpus Philippe Mathieu-Daudé
2017-06-19 10:48   ` Alex Bennée
2017-06-16 16:13 ` [Qemu-devel] [PATCH 02/11] travis: split building/testing to have finer elapsed time Philippe Mathieu-Daudé
2017-06-19 10:49   ` Alex Bennée
2017-06-16 16:13 ` [Qemu-devel] [PATCH 03/11] travis: use yes/no variable to enable/disable tests Philippe Mathieu-Daudé
2017-06-17  8:06   ` Alex Bennée
2017-06-20 17:08     ` Philippe Mathieu-Daudé
2017-06-16 16:13 ` [Qemu-devel] [PATCH 04/11] tests: add rule to compile many objects used by tests Philippe Mathieu-Daudé
2017-06-19 10:56   ` Alex Bennée
2017-06-16 16:13 ` [Qemu-devel] [PATCH 05/11] travis: build tests objects in parallel, then run tests sequentially Philippe Mathieu-Daudé
2017-06-19 10:58   ` Alex Bennée
2017-06-19 11:12     ` Peter Maydell
2017-06-23 18:22       ` Philippe Mathieu-Daudé
2017-06-16 16:13 ` [Qemu-devel] [PATCH 06/11] travis: retry if llvm.org timeouts Philippe Mathieu-Daudé
2017-06-19 11:00   ` Alex Bennée [this message]
2017-06-16 16:13 ` [Qemu-devel] [PATCH 07/11] travis: install more library dependencies Philippe Mathieu-Daudé
2017-06-16 16:13 ` [Qemu-devel] [PATCH 08/11] " Philippe Mathieu-Daudé
2017-06-19 11:01   ` Alex Bennée
2017-06-16 16:13 ` [Qemu-devel] [PATCH 09/11] scripts/run-coverity-scan: Script to run Coverity Scan build Philippe Mathieu-Daudé
2017-06-16 16:13 ` [Qemu-devel] [PATCH 10/11] travis: Add config to do a Coverity Scan upload Philippe Mathieu-Daudé
2017-06-16 16:13 ` [Qemu-devel] [PATCH 11/11] MAINTAINERS: self-appoint me as reviewer in build/test automation Philippe Mathieu-Daudé

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8737awutqw.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=f4bug@amsat.org \
    --cc=famz@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).