Closed
Bug 793800
Opened 12 years ago
Closed 12 years ago
TBPL should recognise hg errors ("^abort: ")
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
(Blocks 1 open bug)
Details
(Whiteboard: [sheriff-want])
Attachments
(1 file)
1.47 KB,
patch
|
Swatinem
:
review+
|
Details | Diff | Splinter Review |
Broken out from bug 793641. Whilst hg failures are set to RETRY, it would still be nice if TBPL's annotated summary showed the reason for the retry, without having to open the log: { ========= Started clone build tools failed (results: 5, elapsed: 1 secs) (at 2012-09-24 03:17:20.662054) ========= hg clone http://hg.mozilla.org/build/tools tools in dir /Users/cltbld/talos-slave/test/. (timeout 1320 secs) watching logfiles {} argv: ['hg', 'clone', 'http://hg.mozilla.org/build/tools', 'tools'] environment: Apple_PubSub_Socket_Render=/tmp/launch-0HWyTr/Render CVS_RSH=ssh DISPLAY=/tmp/launch-OkShAP/org.x:0 HOME=/Users/cltbld LOGNAME=cltbld PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/X11/bin PWD=/Users/cltbld/talos-slave/test PYTHONPATH=/Library/Python/2.5/site-packages SHELL=/bin/bash SSH_AUTH_SOCK=/tmp/launch-BiBKpA/Listeners TMPDIR=/var/folders/Hs/HsDn6a9SG8idoIya6p9mtE+++TI/-Tmp-/ USER=cltbld VERSIONER_PYTHON_PREFER_32_BIT=no VERSIONER_PYTHON_VERSION=2.6 __CF_USER_TEXT_ENCODING=0x1F5:0:0 using PTY: False abort: HTTP Error 500: Internal Server Error program finished with exit code 255 elapsedTime=1.266496 ========= Finished clone build tools failed (results: 5, elapsed: 1 secs) (at 2012-09-24 03:17:21.955880) ========= }
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #664153 -
Flags: review?(arpad.borsos)
Updated•12 years ago
|
Attachment #664153 -
Flags: review?(arpad.borsos) → review+
Assignee | ||
Comment 2•12 years ago
|
||
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/9282a188e3a3
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 3•12 years ago
|
||
The law of unexpected consequences says: https://tbpl.mozilla.org/php/getParsedLog.php?id=15575462&tree=Mozilla-Inbound
Assignee | ||
Comment 4•12 years ago
|
||
Yeah I'd seen those prior to landing. Whilst not the root cause of that failure case, I still think they add some clarity to the failure in the annotated summary (at least for people who know what it implies). We can: * Add regex to support "rm: cannot lstat `.*': Permission denied" and then either: a) accept that we'll have a double failure entry (the "rm:.*" and the "abort:.*") for this failure case b) Make the clobber step fatal, so we don't blindly carry on c) Replace the generic "^abort:" with something more specific from http://hg.mozilla.org/build/buildbotcustom/file/802f2ec88379/status/errors.py#l12
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 5•12 years ago
|
||
b') - haltOnFailure + RETRY to make the bad thing go away
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•9 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•