testing/next (with build fixes, finally GREEN!)

[PATCH v3 15/17] .travis.yml: drop the travis_retry from tests

Posted by Alex Bennée 5 years, 9 months ago

This was a crutch when we introduced it - however it does have the
disadvantage of causing tests to timeout with large amounts of logs.
Lets drop it and see if the stability has improved since.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .travis.yml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index 022462f6981..875671325c8 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -103,7 +103,12 @@ script:
     else
         $(exit $BUILD_RC);
     fi
-  - if [ "$BUILD_RC" -eq 0 ] ; then travis_retry ${TEST_CMD} ; else $(exit $BUILD_RC); fi
+  - |
+    if [ "$BUILD_RC" -eq 0 ] ; then
+        ${TEST_CMD} ;
+    else
+        $(exit $BUILD_RC);
+    fi
 after_script:
   - if command -v ccache ; then ccache --show-stats ; fi
 
-- 
2.20.1

Re: [PATCH v3 15/17] .travis.yml: drop the travis_retry from tests

Posted by Philippe Mathieu-Daudé 5 years, 9 months ago

On 2/3/20 10:09 AM, Alex Bennée wrote:
> This was a crutch when we introduced it - however it does have the
> disadvantage of causing tests to timeout with large amounts of logs.
> Lets drop it and see if the stability has improved since.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   .travis.yml | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 022462f6981..875671325c8 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -103,7 +103,12 @@ script:
>       else
>           $(exit $BUILD_RC);
>       fi
> -  - if [ "$BUILD_RC" -eq 0 ] ; then travis_retry ${TEST_CMD} ; else $(exit $BUILD_RC); fi
> +  - |
> +    if [ "$BUILD_RC" -eq 0 ] ; then
> +        ${TEST_CMD} ;
> +    else
> +        $(exit $BUILD_RC);
> +    fi
>   after_script:
>     - if command -v ccache ; then ccache --show-stats ; fi
>   
> 

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>