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
next prev parent 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).