[v3,24/31] autobuild-run: create reason file on build failures
diff mbox series

Message ID 20190707052831.9469-24-itsatharva@gmail.com
State Superseded
Headers show
  • [v3,01/31] autobuild-run: introduce Builder class
Related show

Commit Message

Atharva Lele July 7, 2019, 5:28 a.m. UTC
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@gmail.com>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Changes from reason-file-v2 series:
  - Move to builder-class series
 scripts/autobuild-run | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff mbox series

diff --git a/scripts/autobuild-run b/scripts/autobuild-run
index 678bac5..8f60e89 100755
--- a/scripts/autobuild-run
+++ b/scripts/autobuild-run
@@ -598,6 +598,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"""
@@ -606,7 +611,6 @@  class Builder:
                                  (os.path.join(self.outputdir, "logfile"), resultfile)],
-            reason = get_failure_reason()
             if not reason:
@@ -630,7 +634,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: