diff mbox series

[v2,18/23] travis: enable travis_retry for check phase

Message ID 20190717134335.15351-19-alex.bennee@linaro.org
State Superseded
Headers show
Series testing/next for 4.1-rc2 (win, travis, iotests) | expand

Commit Message

Alex Bennée July 17, 2019, 1:43 p.m. UTC
We have some flaky tests and usually the test passes on a retry.
Enable travis_retry for the test phase and see if that helps keep
things green.

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

---
 .travis.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.20.1

Comments

Philippe Mathieu-Daudé July 22, 2019, 8:01 p.m. UTC | #1
Hi Alex,

On 7/17/19 3:43 PM, Alex Bennée wrote:
> We have some flaky tests and usually the test passes on a retry.

> Enable travis_retry for the test phase and see if that helps keep

> things green.

> 

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

> ---

>  .travis.yml | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/.travis.yml b/.travis.yml

> index 5d3d6ee1d32..caf0a1f8faf 100644

> --- a/.travis.yml

> +++ b/.travis.yml

> @@ -75,7 +75,7 @@ before_script:

>    - mkdir -p ${BUILD_DIR} && cd ${BUILD_DIR}

>    - ${SRC_DIR}/configure ${BASE_CONFIG} ${CONFIG} || { cat config.log && exit 1; }

>  script:

> -  - make -j3 && ${TEST_CMD}

> +  - make -j3 && travis_retry ${TEST_CMD}

>  

>  

>  matrix:

> 


Does it help?

IIRC if TEST_CMD takes >20min the job will likely fails, anyway at best
it succeeds :)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
diff mbox series

Patch

diff --git a/.travis.yml b/.travis.yml
index 5d3d6ee1d32..caf0a1f8faf 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -75,7 +75,7 @@  before_script:
   - mkdir -p ${BUILD_DIR} && cd ${BUILD_DIR}
   - ${SRC_DIR}/configure ${BASE_CONFIG} ${CONFIG} || { cat config.log && exit 1; }
 script:
-  - make -j3 && ${TEST_CMD}
+  - make -j3 && travis_retry ${TEST_CMD}
 
 
 matrix: