Re: [PATCH 2/2] scripts: return errors for CI

[Henning Schild <henning.schild@siemens.com>]

I just saw something similar on the list.

Henning

Am Fri, 9 Mar 2018 09:21:28 +0100
schrieb Henning Schild <henning.schild@siemens.com>:

> I received mail from Jenkins looking like that:
> 
> > Test: FAIL  
> ...
> > Test: FAIL  
> ...
> > POST BUILD TASK : SUCCESS  
> 
> This patch actually collects the number of failing qemus and returns
> it, so the caller can react on it.
> 
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
> ---
>  scripts/vm_smoke_test | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/scripts/vm_smoke_test b/scripts/vm_smoke_test
> index 3bcfd5a..f1bd411 100755
> --- a/scripts/vm_smoke_test
> +++ b/scripts/vm_smoke_test
> @@ -6,6 +6,8 @@
>  CONSOLE_OUTPUT=/tmp/isar_console
>  PID_FILE=/tmp/qemu.pid
>  
> +ret=0
> +
>  check_output() {
>      str=$(tail -1 $CONSOLE_OUTPUT)
>  
> @@ -13,6 +15,7 @@ check_output() {
>          echo "Test: PASSED"
>      else
>          echo "Test: FAIL"
> +	ret=`expr $ret + 1`
>      fi
>  }
>  
> @@ -48,3 +51,5 @@ run_test amd64 stretch
>  # i386 machine
>  run_test i386 jessie
>  run_test i386 stretch
> +
> +exit $ret