[Buildroot] [PATCH 23/27] autobuild-run: create reason file on build failures

[Arnout Vandecappelle]

On 29/06/2019 07:02, Atharva Lele wrote:
> When a build fails, we calculate the reason of failure on
> the server side as well as client side. To solve this redundancy
> we only calculate on the client side and submit a file which
> contains the reason of failure.
> 
> Signed-off-by: Atharva Lele <itsatharva at gmail.com>

Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>

 Regards,
 Arnout

> ---
> Changes from reason-file-v2 series:
>   - Move to builder-class series
> ---
>  scripts/autobuild-run | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/autobuild-run b/scripts/autobuild-run
> index 3e29f1f..d26594a 100755
> --- a/scripts/autobuild-run
> +++ b/scripts/autobuild-run
> @@ -595,6 +595,11 @@ class Builder:
>              # not found
>              return None
>  
> +        reason = get_failure_reason()
> +        if reason:
> +            with open(os.path.join(self.resultdir, "reason"), "w+") as reasonf:
> +                reasonf.write("-".join(reason))
> +
>          def extract_end_log(resultfile):
>              """Save the last part of the build log, starting from the failed package"""
>  
> @@ -603,7 +608,6 @@ class Builder:
>                                  (os.path.join(self.outputdir, "logfile"), resultfile)],
>                                  shell=True)
>  
> -            reason = get_failure_reason()
>              if not reason:
>                  extract_last_500_lines()
>              else:
> @@ -627,7 +631,6 @@ class Builder:
>          def copy_config_log_files():
>              """Recursively copy any config.log files from the failing package"""
>  
> -            reason = get_failure_reason()
>              if not reason:
>                  return
>  
>