summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorbst-marge-bot <marge-bot@buildstream.build>2019-08-12 12:18:30 +0000
committerbst-marge-bot <marge-bot@buildstream.build>2019-08-12 12:18:30 +0000
commit05d673ff2881e0f8a177622489ead9c9158d6032 (patch)
tree653d44aa86c71ed120939ac583544a80de066d6b
parentf7e56788bca1204b1a76ffb357db92b5994db22c (diff)
parentcded5549756cedf9620c89571b3864da963328dc (diff)
downloadbuildstream-05d673ff2881e0f8a177622489ead9c9158d6032.tar.gz
Merge branch 'tmewett/report-weird-return-codes' into 'master'
Report error when job process unexpectedly dies Closes #1089 See merge request BuildStream/buildstream!1531
-rw-r--r--src/buildstream/_scheduler/jobs/job.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/src/buildstream/_scheduler/jobs/job.py b/src/buildstream/_scheduler/jobs/job.py
index 32c5559fb..7cbde2d39 100644
--- a/src/buildstream/_scheduler/jobs/job.py
+++ b/src/buildstream/_scheduler/jobs/job.py
@@ -449,7 +449,11 @@ class Job():
try:
returncode = _ReturnCode(returncode)
- except ValueError: # An unexpected return code was returned, let's fail permanently
+ except ValueError:
+ # An unexpected return code was returned; fail permanently and report
+ self.message(MessageType.ERROR,
+ "Internal job process unexpectedly died with exit code {}".format(returncode),
+ logfile=self._logfile)
returncode = _ReturnCode.PERM_FAIL
# We don't want to retry if we got OK or a permanent fail.